Ticket #23899: 23899.diff

wp-admin/js/revisions.js

@@ -583 +583 @@
                                 var addHtml = this.template;
                                 this.$el.html( addHtml );
 
-                                var modelcount = REVAPP._revisions.length;
+                                var modelcount = REVAPP._revisions.length,
+                                        maxVal = modelcount - 1;
 
                                 slider = $( "#slider" );
                                 if ( 1 === REVAPP._compareOneOrTwo ) {
@@ -591 +592 @@
                                         slider.slider({
                                                 value: REVAPP._rightDiff - 1,
                                                 min: 0,
-                                                max: modelcount - 1,
+                                                max: maxVal,
                                                 step: 1,
 
+                                                create: function( event, ui ) {
+                                                        var startVal = $(this).slider('value');
 
+                                                        if ( 0 === startVal )
+                                                                $('#previous').prop('disabled', true);
+
+                                                        if ( maxVal === startVal )
+                                                                $('#next').prop('disabled', true);
+                                                },
+
                                                 // slide interactions for one handles slider
                                                 slide: function( event, ui ) {
 
                                                         REVAPP._rightDiff = ( ui.value + 1 );
                                                         REVAPP._revisionView.render();
+
+                                                        self.disabledButtonCheck(ui.value);
+
                                                         /*
                                                         $( 'a.ui-slider-handle' ).tooltip( {
                                                                 content: REVAPP._revisions.at( ui.value ).get( 'revision_date_author_short' ),
@@ -735 +748 @@
                                 return this;
                         },
 
+                        disabledButtonCheck: function(val) {
+
+                                var maxVal = this.model.length - 1,
+                                        next = $('#next'),
+                                        prev = $('#previous');
+
+                                // Disable "Next" button if you're on the last node
+                                if ( maxVal === val )
+                                        next.prop('disabled', true);
+                                else
+                                        next.prop('disabled', false);
+
+                                // Disable "Previous" button if you're on the 0 node
+                                if ( 0 === val )
+                                        prev.prop('disabled', true);
+                                else
+                                        prev.prop('disabled', false);
+                        },
+
                         // next and previous buttons, only available in compare one mode
                         events: {
                                 'click #next': 'nextRevision',
@@ -746 +778 @@
                                 if ( REVAPP._rightDiff < this.model.length ) // unless at right boundry
                                         REVAPP._rightDiff = REVAPP._rightDiff + 1 ;
 
+                                this.disabledButtonCheck(REVAPP._rightDiff - 1);
+
                                 REVAPP._revisionView.render();
 
                                 $( '#slider' ).slider( 'value', REVAPP._rightDiff - 1 ).trigger( 'slide' );
@@ -756 +790 @@
                                 if ( REVAPP._rightDiff > 1 ) // unless at left boundry
                                         REVAPP._rightDiff = REVAPP._rightDiff - 1 ;
 
+                                this.disabledButtonCheck(REVAPP._rightDiff - 1);
+
                                 REVAPP._revisionView.render();
 
                                 $( '#slider' ).slider( 'value', REVAPP._rightDiff - 1 ).trigger( 'slide' );