Ticket #24425: 24425.draft.17.diff

wp-admin/js/revisions.js

@@ -350 +350 @@
                 },
 
                 nextRevision: function() {
-                        var toIndex = this.model.revisions.indexOf( this.model.get( 'to' ) );
+                        var toIndex = isRtl ? this.model.revisions.length - this.model.revisions.indexOf( this.model.get( 'to' ) ) - 1 : this.model.revisions.indexOf( this.model.get( 'to' ) );
                         toIndex     = isRtl ? toIndex - 1 : toIndex + 1;
                         this.gotoModel( toIndex );
                 },
                 
                 previousRevision: function() {
-                        var toIndex = this.model.revisions.indexOf( this.model.get('to') );
+                        var toIndex = isRtl ? this.model.revisions.length - this.model.revisions.indexOf( this.model.get( 'to' ) ) - 1 : this.model.revisions.indexOf( this.model.get( 'to' ) );
                         toIndex     = isRtl ? toIndex + 1 : toIndex - 1;
                         this.gotoModel( toIndex );
                 },
@@ -367 +367 @@
 
                 // Check to see if the Previous or Next buttons need to be disabled or enabled
                 disabledButtonCheck: function() {
-                        var maxVal   = isRtl ? 0 : this.model.revisions.length - 1,
-                                minVal   = isRtl ? this.model.revisions.length - 1 : 0,
+                        var maxVal   = this.model.revisions.length - 1,
+                                minVal   = 0,
                                 next     = $( '.revisions-next .button' ),
                                 previous = $( '.revisions-previous .button' ),
                                 val      = this.model.revisions.indexOf( this.model.get( 'to' ) );
@@ -420 +420 @@
                         this.$el.slider( this.settings.toJSON() );
                         this.settings.on( 'change', function( model, options ) {
                                 // Apply changes to slider settings here.
-                                this.$el.slider( { value: this.model.revisions.indexOf( this.model.get( 'to' ) ) } ); // Set handle to current to model
+                                if ( isRtl ) {
+                                        this.$el.slider( { // Order reversed in Rtl mode
+                                                value: this.model.revisions.length - this.model.revisions.indexOf( this.model.get( 'to' ) ) - 1
+                                        } );
+                                } else {
+                                        this.$el.slider( { // Set handle to current to model
+                                                value: this.model.revisions.indexOf( this.model.get( 'to' ) )
+                                        } );
+                                }
                         }, this );
                         // Reset to the initially selected revision
                         this.slide( '', this.settings.attributes );

wp-admin/css/wp-admin-rtl.css

@@ -975 +975 @@
         left: 6px;
 }
 
-#toggle-revision-compare-mode {
+.revision-toggle-compare-mode {
         right: auto;
         left: 0;
         padding: 9px 0 0 9px;
 }
 
-#diff-next-revision {
+.revisions-next {
         float: left;
 }
 
-#diff-previous-revision {
+.revisions-previous {
         float: right;
 }