WordPress.org

Make WordPress Core

Changeset 23016


Ignore:
Timestamp:
12/04/12 06:01:47 (17 months 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.