[wp-trac] [WordPress Trac] #23497: Revisions Rewrite using JS/Backbone

WordPress Trac noreply at wordpress.org
Fri Mar 1 05:35:15 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 adamsilverstein):

 '''changes/corrections'''

  * correct variable misname: REVAPP.right_model.loading ->
 REVAPP.right_model_loading
  * correct variable name convention across code in
 revisions.js/revisions.php/ajax-actions.php/pluggable.php
  * didn't change css names yet, change IDs to classes? cleanup help
 appreciated.
  * remove extraneous <br clear> above wrap
  * changed checks for booleans to ! empty()
  * indent foreach block
  * added translations for Next/Previous buttons
  * added _post_restored_from meta when restoring revision, currently a
 single meta entry (should this be one per restore for full audit trail?)
  * added display of _post_restored_from in post revisions meta box, at
 bottom of list, if present
  * change restore button to include revision ID, as in "Restore Revision
 ID xxx"
  * other minor bug fixes - if you still see failures please report
  * still need to optimize returned data

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


More information about the wp-trac mailing list