WMDE-Fisch has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/364414 )
Change subject: Move data pos attr to PointerView
......................................................................
Move data pos attr to PointerView
Change-Id: I46c3dceaddae84b279625caec3349aa10d931048
---
M modules/ext.RevisionSlider.Pointer.js
M modules/ext.RevisionSlider.PointerView.js
M modules/ext.RevisionSlider.SliderView.js
3 files changed, 10 insertions(+), 20 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/RevisionSlider
refs/changes/14/364414/1
diff --git a/modules/ext.RevisionSlider.Pointer.js
b/modules/ext.RevisionSlider.Pointer.js
index 3f53e88..8388e3a 100644
--- a/modules/ext.RevisionSlider.Pointer.js
+++ b/modules/ext.RevisionSlider.Pointer.js
@@ -31,6 +31,7 @@
*/
setPosition: function ( p ) {
this.position = p;
+ this.getView().setDataPositionAttribute( p );
},
/**
diff --git a/modules/ext.RevisionSlider.PointerView.js
b/modules/ext.RevisionSlider.PointerView.js
index 6e2db5f..fad6efb 100644
--- a/modules/ext.RevisionSlider.PointerView.js
+++ b/modules/ext.RevisionSlider.PointerView.js
@@ -75,6 +75,15 @@
},
/**
+ * Sets the HTML attribute for the position
+ *
+ * @param {number} pos
+ */
+ setDataPositionAttribute: function ( pos ) {
+ this.getElement().attr( 'data-pos', pos );
+ },
+
+ /**
* Moves the pointer to a position
*
* @param {number} posInPx
diff --git a/modules/ext.RevisionSlider.SliderView.js
b/modules/ext.RevisionSlider.SliderView.js
index 923a9aa..f70da59 100644
--- a/modules/ext.RevisionSlider.SliderView.js
+++ b/modules/ext.RevisionSlider.SliderView.js
@@ -294,7 +294,6 @@
$clickedRev = this.getRevElementAtPosition( $revisions,
pos );
pointerMoved.setPosition( pos );
- this.updatePointerPositionAttributes();
if ( $line.hasClass(
'mw-revslider-pointer-container-newer' ) ) {
this.refreshRevisions(
$clickedRev.attr( 'data-revid' ),
@@ -341,14 +340,12 @@
self.removePointerDragCursor();
if ( self.escapePressed ) {
-
self.updatePointerPositionAttributes();
self.resetSliderLines();
return;
}
mw.track(
'counter.MediaWiki.RevisionSlider.event.pointerMove' );
pointer.setPosition(
self.slider.getOldestVisibleRevisionIndex() + relativeIndex );
- self.updatePointerPositionAttributes();
self.resetSliderLines();
self.resetRevisionStylesBasedOnPointerPosition( $revisions );
@@ -478,7 +475,6 @@
this.resetRevisionStylesBasedOnPointerPosition(
this.$element.find(
'div.mw-revslider-revisions' )
);
- this.updatePointerPositionAttributes();
this.refreshRevisions(
$( '.mw-revslider-revision[data-pos="' +
this.getNewerPointerPos() + '"]' ).attr( 'data-revid' ),
$( '.mw-revslider-revision[data-pos="' +
this.getOlderPointerPos() + '"]' ).attr( 'data-revid' )
@@ -535,7 +531,6 @@
}
this.setNewerPointerPos( $newRevElement.data( 'pos' ) );
this.resetSliderLines();
- this.updatePointerPositionAttributes();
},
/**
@@ -617,21 +612,6 @@
this.getRevElementAtPosition( $revisions,
positionIndex ).addClass( 'mw-revslider-revision-intermediate' );
positionIndex++;
}
- },
-
- /**
- * Updates value of pointers' position data attribute
- */
- updatePointerPositionAttributes: function () {
- this.pointerNewer.getView().getElement().attr(
- 'data-pos',
- this.getNewerPointerPos()
- );
-
- this.pointerOlder.getView().getElement().attr(
- 'data-pos',
- this.getOlderPointerPos()
- );
},
/**
--
To view, visit https://gerrit.wikimedia.org/r/364414
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I46c3dceaddae84b279625caec3349aa10d931048
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/RevisionSlider
Gerrit-Branch: master
Gerrit-Owner: WMDE-Fisch <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
