[wp-trac] [WordPress Trac] #23497: Revisions Rewrite using JS/Backbone
WordPress Trac
noreply at wordpress.org
Thu Feb 28 22:55:48 UTC 2013
#23497: Revisions Rewrite using JS/Backbone
---------------------------------------+------------------------
Reporter: adamsilverstein | Owner: westi
Type: enhancement | Status: reviewing
Priority: normal | Milestone: 3.6
Component: Revisions | Version: 3.5.1
Severity: normal | Resolution:
Keywords: needs-patch needs-testing |
---------------------------------------+------------------------
Comment (by helen):
Definitely need to do a review of HTML and CSS naming and structure -
there's a lot of ID usage going on the CSS that I'm not super convinced
about as well. ocean90 - do you want to look at that?
Other notes I had:
* The clearing break at the top pushes the whole wrap down. Does that div
+ br really need to be there? If something appears there (which I haven't
managed to trigger), it would make the page jumpy, which is also not
great. I don't think anything should come before or really outside of the
wrap at all, anyway.
* The "Restore" button doesn't tell me what exactly is being restored,
especially in compare-two mode. Not sure what a good way is to handle
that, especially given translations, but perhaps adding the revision
number would be helpful.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/23497#comment:39>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list