jenkins-bot has submitted this change and it was merged.

Change subject: Fix order when executing pointer update
......................................................................


Fix order when executing pointer update

This should always happen before updating the diff.

Change-Id: Icf6b52c3495e42e188bfa5d5307f91d0558cc7a4
---
M modules/ext.RevisionSlider.SliderView.js
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Tobias Gritschacher: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ext.RevisionSlider.SliderView.js 
b/modules/ext.RevisionSlider.SliderView.js
index 090ceac..afc8335 100644
--- a/modules/ext.RevisionSlider.SliderView.js
+++ b/modules/ext.RevisionSlider.SliderView.js
@@ -205,15 +205,15 @@
                                                revId1, revId2;
                                        mw.track( 
'counter.MediaWiki.RevisionSlider.event.pointerMove' );
                                        pointer.setPosition( 
self.slider.getFirstVisibleRevisionIndex() + relativeIndex );
+                                       self.updatePointerPositionAttributes();
                                        
self.resetPointerStylesBasedOnPosition();
                                        
self.resetRevisionStylesBasedOnPointerPosition( $revisions );
 
                                        revId1 = self.getRevElementAtPosition( 
$revisions, self.pointerOlder.getPosition() ).data( 'revid' );
+
                                        revId2 = self.getRevElementAtPosition( 
$revisions, self.pointerNewer.getPosition() ).data( 'revid' );
 
                                        self.refreshRevisions( revId1, revId2 );
-
-                                       self.updatePointerPositionAttributes();
 
                                        $( '.mw-revslider-revision-wrapper' 
).removeClass( 'mw-revslider-pointer-cursor' );
                                },

-- 
To view, visit https://gerrit.wikimedia.org/r/315672
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icf6b52c3495e42e188bfa5d5307f91d0558cc7a4
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/RevisionSlider
Gerrit-Branch: master
Gerrit-Owner: WMDE-Fisch <christoph.jau...@wikimedia.de>
Gerrit-Reviewer: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: Tobias Gritschacher <tobias.gritschac...@wikimedia.de>
Gerrit-Reviewer: WMDE-Fisch <christoph.jau...@wikimedia.de>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to