Addshore has uploaded a new change for review. https://gerrit.wikimedia.org/r/288584
Change subject: Map pointer colors to diff colours ...................................................................... Map pointer colors to diff colours The need for left-pointer and right-pointer should be cleaned up at some stage. Infact It may be wise to loose the distinction between left and right pointer and simply have 2 pointers. Bug: T134994 Change-Id: I68a2159bde6fff969ca54a79587cfe03ed783454 --- M modules/ext.RevisionSlider.PointerView.js M modules/ext.RevisionSlider.SliderView.js M modules/ext.RevisionSlider.css 3 files changed, 20 insertions(+), 4 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/RevisionSlider refs/changes/84/288584/1 diff --git a/modules/ext.RevisionSlider.PointerView.js b/modules/ext.RevisionSlider.PointerView.js index aa86db4..1b2e39e 100644 --- a/modules/ext.RevisionSlider.PointerView.js +++ b/modules/ext.RevisionSlider.PointerView.js @@ -27,6 +27,11 @@ initialize: function () { this.$html = $( '<div class="pointer" />' ).addClass( this.cssClass ); + if ( this.cssClass === 'left-pointer' ) { + this.$html.addClass( 'oldid-pointer' ); + } else { + this.$html.addClass( 'newid-pointer' ); + } }, /** diff --git a/modules/ext.RevisionSlider.SliderView.js b/modules/ext.RevisionSlider.SliderView.js index 6260df9..02acb05 100644 --- a/modules/ext.RevisionSlider.SliderView.js +++ b/modules/ext.RevisionSlider.SliderView.js @@ -86,6 +86,17 @@ diffPage.refresh( revId1, revId2 ); diffPage.pushState( revId1, revId2, self ); + }, + drag: function () { + var leftPointerOffset = self.leftPointer.getView().getElement().offset(), + rightPointerOffset = self.rightPointer.getView().getElement().offset(); + if ( leftPointerOffset.left > rightPointerOffset.left ) { + self.leftPointer.getView().getElement().removeClass( 'oldid-pointer' ).addClass( 'newid-pointer' ); + self.rightPointer.getView().getElement().removeClass( 'newid-pointer' ).addClass( 'oldid-pointer' ); + } else { + self.leftPointer.getView().getElement().removeClass( 'newid-pointer' ).addClass( 'oldid-pointer' ); + self.rightPointer.getView().getElement().removeClass( 'oldid-pointer' ).addClass( 'newid-pointer' ); + } } } ); diff --git a/modules/ext.RevisionSlider.css b/modules/ext.RevisionSlider.css index b908cdb..eeb0e3b 100755 --- a/modules/ext.RevisionSlider.css +++ b/modules/ext.RevisionSlider.css @@ -69,11 +69,11 @@ margin: auto; z-index: 10; } -.left-pointer { - border-color: transparent transparent #00f transparent; +.oldid-pointer { + border-color: transparent transparent #ffe49c transparent; } -.right-pointer { - border-color: transparent transparent #f00 transparent; +.newid-pointer { + border-color: transparent transparent #a3d3ff transparent; } .pointer { position: absolute !important; -- To view, visit https://gerrit.wikimedia.org/r/288584 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I68a2159bde6fff969ca54a79587cfe03ed783454 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/RevisionSlider Gerrit-Branch: master Gerrit-Owner: Addshore <addshorew...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits