WordPress.org

Make WordPress Core

Opened 19 months ago

Last modified 2 months ago

#23042 closed defect (bug)

Post edit page may become almost unusable and textarea resize not saved — at Initial Version

Reported by: kinderr@… Owned by:
Milestone: 3.5.1 Priority: normal
Severity: normal Version: 3.5
Component: Editor Keywords: has-patch
Focuses: Cc:

Description

Follow-up testing to Ticket #22708 and patches http://core.trac.wordpress.org/changeset/23016
http://core.trac.wordpress.org/changeset/23017

I installed the new versions of:
/wp-admin/js/post.js (23016) and
/wp-includes/class-wp-editor.php (23017)

on my Wordpress 3.5 server. The problem still exists that the post editor window is forced to 5000 pixels tall because the wp_usermeta -> wp_user-settings -> ed_size value is being set to ed_size=310566666&editor=tinymce&mfold=f or other extremely large value with FireFox 17.0.1 and Chrome 23.0.1271.97 m. The ed_size bad cookie value triggers the patch code logic to set the post editor to 5000 lines.

My Win7 screen resolution is 1920w x 1080h which means the editor is around 5 screens tall. This is almost as bad as the practically infinite bug value. Manually resizing the post editor height by dragging the lower right pane control will not survive a post update and the post editor resets to 5000 pixels high. This makes the editor tool bar practically inaccessible and a big drag on usability. See http://wordpress.org/support/topic/the-option-to-change-the-size-of-editor-is-missing-how-to-change-the-size

As a temporary workaround, I modified line 704 in post.js to 500 pixels:
if ( height > 50 && height < 500 && height != getUserSetting( 'ed_size' ) )
setUserSetting( 'ed_size', height );
}

and lines 74 & 75 in class-wp-editor.php to 500 pixels:
if ( $seteditor_height? < 50 )

$seteditor_height? = 50;

elseif ( $seteditor_height? > 500 )

$seteditor_height? = 500;

which is a comfortable post editor size (my personal preference) that is "sticky".

In summary, the 5000 pixel sanity logic for the post editor size doesn't fix the underlying ed_size blowup.

Change History (0)

Note: See TracTickets for help on using tickets.