Problem statement
During the latest additions and fixes to the revision slider something must have broken the line order
mock
they used to look like this:
Acceptance criteria
- both lines should be in the same placement as seen in the mock
Charlie_WMDE | |
Oct 29 2018, 4:48 PM |
F26707615: Bildschirmfoto 2018-10-22 um 11.54.22.png | |
Oct 29 2018, 4:48 PM |
F26962669: Screenshot_20181029_174735.png | |
Oct 29 2018, 4:48 PM |
Problem statement
During the latest additions and fixes to the revision slider something must have broken the line order
mock
they used to look like this:
Acceptance criteria
Change 471541 had a related patch set uploaded (by Zoranzoki21; owner: Vijay9908):
[mediawiki/extensions/RevisionSlider@master] Reversed the colour of sliders
Change 471776 had a related patch set uploaded (by Vijay9908; owner: Vijay9908):
[mediawiki/extensions/RevisionSlider@master] Bug: T208238 Reversed the colors of the sliders
Change 473217 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/RevisionSlider@master] Fix reversed blue and yellow lines
Change 471776 abandoned by Thiemo Kreuz (WMDE):
Bug: T208238 Reversed the colors of the sliders
Reason:
This patch is empty. I created I0c03e47 as a fix for the bug mentioned. Please have a look.
I tracked the issue down to the following change in the patch about T162119: Add keyboard shortcuts to move between revisions: https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/RevisionSlider/ /459678/5/modules/ext.RevisionSlider.SliderView.js. I am not sure, but it looks to me like this particular change might have been nothing but a mistake. @SrishtiSethi, can you have a look, please?
Change 473217 merged by jenkins-bot:
[mediawiki/extensions/RevisionSlider@master] Fix (accidentally?) reversed blue and yellow lines
Change 473710 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/RevisionSlider@wmf/1.33.0-wmf.4] Fix (accidentally?) reversed blue and yellow lines
Change 473711 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/RevisionSlider@wmf/1.33.0-wmf.3] Fix (accidentally?) reversed blue and yellow lines
Change 473711 merged by jenkins-bot:
[mediawiki/extensions/RevisionSlider@wmf/1.33.0-wmf.3] Fix (accidentally?) reversed blue and yellow lines
Change 473710 merged by jenkins-bot:
[mediawiki/extensions/RevisionSlider@wmf/1.33.0-wmf.4] Fix (accidentally?) reversed blue and yellow lines
Mentioned in SAL (#wikimedia-operations) [2018-11-15T12:37:06Z] <lucaswerkmeister-wmde@deploy1001> Started scap: php-1.33.0-wmf.3/extensions/RevisionSlider/modules/ext.RevisionSlider.SliderView.js [[gerrit:473710|Fix (accidentally?) reversed blue and yellow lines (T162119, T208238)]]
Mentioned in SAL (#wikimedia-operations) [2018-11-15T12:37:17Z] <lucaswerkmeister-wmde@deploy1001> sync aborted: php-1.33.0-wmf.3/extensions/RevisionSlider/modules/ext.RevisionSlider.SliderView.js [[gerrit:473710|Fix (accidentally?) reversed blue and yellow lines (T162119, T208238)]] (duration: 00m 11s)
Mentioned in SAL (#wikimedia-operations) [2018-11-15T12:38:33Z] <lucaswerkmeister-wmde@deploy1001> Synchronized php-1.33.0-wmf.3/extensions/RevisionSlider/modules/ext.RevisionSlider.SliderView.js: [[gerrit:473710|Fix (accidentally?) reversed blue and yellow lines (T162119, T208238)]] (duration: 00m 54s)
Mentioned in SAL (#wikimedia-operations) [2018-11-15T12:59:47Z] <tarrow@deploy1001> Synchronized php-1.33.0-wmf.4/extensions/RevisionSlider/modules/ext.RevisionSlider.SliderView.js: [[gerrit:473710]] Fix (accidentally?) reversed blue and yellow lines SWAT T208238 T162119 again (duration: 00m 55s)