Page MenuHomePhabricator

Edit-links are shown in reverse order in sticky header from standard tabs in article
Closed, ResolvedPublic3 Estimated Story PointsBUG REPORT

Description

  • When using the Preference for "Show me both editor tabs"
  • The edit | edit source links in Vector-2022's sticky header, are in the reverse-order from the standard links.

image.png (146×863 px, 28 KB)

Most easily seen with my personal user-css that replaces the in-page text-links with icons, and floats them right.
image.png (208×578 px, 22 KB)

QA Results - Beta

ACStatusDetails
1T318447#9265927

QA Results - Prod

ACStatusDetails
1T318447#9270928

Event Timeline

Jdlrobson renamed this task from Edit-links are shown in reverse order from standard to Edit-links are shown in reverse order in sticky header from standard tabs in article.Sep 27 2022, 10:32 PM

Note that the order of these tabs is controlled by the config VisualEditorTabPosition, which can be before (default) or after. In WMF config it is set to after if wmgVisualEditorIsSecondaryEditor is set on a non-single-edit-tab wiki, which is the case on 2 wikis: eswiki and hewiki.

Adding the visual edit button should be done in the VisualEditor extension, so an API should be exposed from Vector to allow extensions to do this.

VE can then place the button in the correct order, depending on VisualEditorTabPosition.

Do we need to consult VisualEditorTabsPosition or could we work out the order of these buttons by inspecting the DOM and seeing the order of the links there ?

bwang set the point value for this task to 3.Jul 20 2023, 5:57 PM

Do we need to consult VisualEditorTabsPosition or could we work out the order of these buttons by inspecting the DOM and seeing the order of the links there ?

You probably could, but doing it properly is going to be more robust in the long term.

I think it's more important to let the code live in VE, at which point it becomes trivial to use the config.

Change 957817 had a related patch set uploaded (by Mabualruz; author: Mabualruz):

[mediawiki/skins/Vector@master] Vector sticky header dynamic edit icons order

https://gerrit.wikimedia.org/r/957817

Moving back to incoming since team didn't commit yet to finishing this during the sprint

Mabualruz updated Other Assignee, added: Mabualruz.
Mabualruz subscribed.

Change 957817 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Vector sticky header dynamic edit icons order

https://gerrit.wikimedia.org/r/957817

Edtadros subscribed.

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

When using the Preference for "Show me both editor tabs"
✅ AC1: The edit | edit source links in Vector-2022's sticky header, are in the same order as the standard links.

screenshot 134.png (657×1 px, 214 KB)

Test Result - Prod

Status: ✅ PASS
Environment: enwiki
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

When using the Preference for "Show me both editor tabs"
✅ AC1: The edit | edit source links in Vector-2022's sticky header, are in the same order as the standard links.

screenshot 167.png (433×1 px, 179 KB)