[wp-trac] [WordPress Trac] #22491: Autosave / AYS on the attachment edit screen

WordPress Trac noreply at wordpress.org
Mon Nov 26 15:33:23 UTC 2012


#22491: Autosave / AYS on the attachment edit screen
--------------------------+-----------------------
 Reporter:  helenyhou     |       Owner:  azaozz
     Type:  defect (bug)  |      Status:  reopened
 Priority:  normal        |   Milestone:  3.5
Component:  Media         |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+-----------------------

Comment (by helenyhou):

 Can we just go back to the original plan of not having autosave on the
 attachment edit screen (or if the post type doesn't support editor)? The
 original autosave.js has problems because it's specifically looking for
 #post #content to match the post content, which in this case, it won't if
 there's anything in the post content (#post #content being empty here). It
 may autosave the title, but you'll still get the AYS as well as an
 autosave with empty content that will give you the newer autosave admin
 warning when the content wasn't originally empty.

 Using the ID of content creates issues with styling, since it's not a
 TinyMCE instance but there's a bunch of JS that looks for #content and
 does special things to it that are specific to matching the textarea to
 TinyMCE and the editor size. Seems like that should be dealt with in a
 better way (like only doing that special stuff if #content is associated
 with a TinyMCE instance), but I don't think that's a wise thing to pursue
 in the RC tage.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/22491#comment:28>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list