[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