jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/344419 )
Change subject: Factor out draggable options
......................................................................
Factor out draggable options
Change-Id: I64cca7a7547310bee64ad95f500b722c362bbb45
---
M modules/ext.RevisionSlider.SliderView.js
1 file changed, 34 insertions(+), 12 deletions(-)
Approvals:
jenkins-bot: Verified
Andrew-WMDE: Looks good to me, approved
diff --git a/modules/ext.RevisionSlider.SliderView.js
b/modules/ext.RevisionSlider.SliderView.js
index 5bef724..f5a9419 100644
--- a/modules/ext.RevisionSlider.SliderView.js
+++ b/modules/ext.RevisionSlider.SliderView.js
@@ -178,9 +178,7 @@
*/
initPointers: function( $revisions ) {
var $pointers,
- lastValidLeftPos,
- escapePressed = false,
- self = this;
+ escapePressed = false;
$pointers = this.$element.find( '.mw-revslider-pointer'
);
@@ -191,12 +189,27 @@
}
} );
- $pointers.draggable( {
+ $pointers.draggable( this.buildDraggableOptions(
escapePressed, $revisions ) );
+ },
+
+ /**
+ * Build options for the draggable
+ *
+ * @param {boolean} escapePressed
+ * @param {jQuery} $revisions
+ * @return {Object}
+ */
+ buildDraggableOptions: function( escapePressed, $revisions ) {
+ var lastValidLeftPos,
+ self = this;
+
+ return {
axis: 'x',
grid: [ this.revisionWidth, null ],
containment: '.mw-revslider-pointer-container',
start: function() {
- $( '.mw-revslider-revision-wrapper'
).addClass( 'mw-revslider-pointer-cursor' );
+ $( '.mw-revslider-revision-wrapper' )
+ .addClass(
'mw-revslider-pointer-cursor' );
escapePressed = false;
},
stop: function() {
@@ -221,21 +234,26 @@
self.resetPointerStylesBasedOnPosition();
self.resetRevisionStylesBasedOnPointerPosition( $revisions );
- revId1 = self.getRevElementAtPosition(
$revisions, self.pointerOlder.getPosition() ).data( 'revid' );
+ revId1 = self.getRevElementAtPosition(
+ $revisions,
self.pointerOlder.getPosition()
+ ).data( 'revid' );
- revId2 = self.getRevElementAtPosition(
$revisions, self.pointerNewer.getPosition() ).data( 'revid' );
+ revId2 = self.getRevElementAtPosition(
+ $revisions,
self.pointerNewer.getPosition()
+ ).data( 'revid' );
self.refreshRevisions( revId1, revId2 );
self.redrawPointerLines();
-
},
drag: function( event, ui ) {
var olderLeftPos, newerLeftPos,
- isNew = $( this ).hasClass(
'mw-revslider-pointer-newer' ),
- newestVisibleRevisionLeftPos =
$( '.mw-revslider-revisions-container' ).width() - self.revisionWidth;
+ isNew = $( this ).hasClass(
'mw-revslider-pointer-newer' );
- ui.position.left = Math.min(
ui.position.left, newestVisibleRevisionLeftPos );
+ ui.position.left = Math.min(
+ ui.position.left,
+
self.getNewestVisibleRevisonLeftPos()
+ );
olderLeftPos =
self.pointerOlder.getView().getElement().position().left;
newerLeftPos =
self.pointerNewer.getView().getElement().position().left;
@@ -254,7 +272,11 @@
revert: function() {
return escapePressed;
}
- } );
+ };
+ },
+
+ getNewestVisibleRevisonLeftPos: function() {
+ return $( '.mw-revslider-revisions-container' ).width()
- this.revisionWidth;
},
/**
--
To view, visit https://gerrit.wikimedia.org/r/344419
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I64cca7a7547310bee64ad95f500b722c362bbb45
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/RevisionSlider
Gerrit-Branch: master
Gerrit-Owner: WMDE-Fisch <christoph.jau...@wikimedia.de>
Gerrit-Reviewer: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: Andrew-WMDE <andrew.kos...@wikimedia.de>
Gerrit-Reviewer: Tobias Gritschacher <tobias.gritschac...@wikimedia.de>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
