WordPress.org

Make WordPress Core

Changeset 23016


Ignore:
Timestamp:
12/04/2012 06:01:47 AM (6 years ago)
Author:
azaozz
Message:

Editor: add sanity limit for editor height of 5000px, save only when resizing the Visual or the Text editor, fixes #22708

Location:
trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/wp-admin/js/post.js

    r22653 r23016  
    700700            $(document).unbind('mousemove', dragging).unbind('mouseup', endDrag);
    701701
    702             if ( height > 83 ) {
    703                 height -= 33; // compensate for toolbars, padding...
     702            height -= 33; // compensate for toolbars, padding...
     703            // sanity check
     704            if ( height > 50 && height < 5000 && height != getUserSetting( 'ed_size' ) )
    704705                setUserSetting( 'ed_size', height );
    705             }
    706706        }
    707707
     
    730730                if ( height && !isNaN(height) && tb_height ) {
    731731                    ifr_height = (height - tb_height) + 12; // compensate for padding in the textarea
    732 
    733                     $('#content_tbl').css('height', '' );
    734                     $('#content_ifr').css('height', ifr_height + 'px' );
    735                     setUserSetting( 'ed_size', height );
     732                    // sanity check
     733                    if ( ifr_height > 50 && ifr_height < 5000 ) {
     734                        $('#content_tbl').css('height', '' );
     735                        $('#content_ifr').css('height', ifr_height + 'px' );
     736                    }
    736737                }
    737738            });
     
    741742                var height = $('#content_tbl').height();
    742743
    743                 if ( height && height > 83 ) {
     744                if ( height && height > 83 && height < 5000 ) {
    744745                    height -= 33;
    745746
    746747                    $('#content').css( 'height', height + 'px' );
    747                     setUserSetting( 'ed_size', height );
    748748                }
    749749            });
     
    756756
    757757                        height -= 33;
    758                         if ( height > 50 && height != getUserSetting( 'ed_size' ) )
     758                        // sanity check
     759                        if ( height > 50 && height < 5000 && height != getUserSetting( 'ed_size' ) )
    759760                            setUserSetting( 'ed_size', height );
    760761
  • trunk/wp-includes/class-wp-editor.php

    r22948 r23016  
    7272        if ( $set['editor_height'] < 50 )
    7373            $set['editor_height'] = 50;
     74        elseif ( $set['editor_height'] > 3000 )
     75            $set['editor_height'] = 5000;
    7476
    7577        return $set;
Note: See TracChangeset for help on using the changeset viewer.