Page MenuHomePhabricator

Various buttons on Vector 2022 acquired unexpected link styling when hovered
Closed, ResolvedPublic1 Estimated Story PointsBUG REPORT

Assigned To
Authored By
matmarex
Jun 18 2024, 2:39 AM
Referenced Files
F56716538: screenshot 13.mov.gif
Jul 27 2024, 2:44 AM
F56716537: screenshot 11.mov.gif
Jul 27 2024, 2:44 AM
F56716536: screenshot 12.mov.gif
Jul 27 2024, 2:44 AM
F55442641: 2024-06-18_11-59-10.png
Jun 18 2024, 7:14 PM
F55442627: 2024-06-18_11-55-11.png
Jun 18 2024, 7:14 PM
F55442622: 2024-06-18_11-50-43.png
Jun 18 2024, 7:14 PM
F55442613: 2024-06-18_11-49-33.png
Jun 18 2024, 7:14 PM
F55442611: 2024-06-18_11-47-43.png
Jun 18 2024, 7:14 PM

Description

Various buttons on Vector 2022 acquired unexpected link styling when hovered, changing the text color to light blue or gaining underlines.

For example: (testing on https://patchdemo.wmflabs.org/wikis/a61a6b72be/wiki/Main_Page)

image.png (2×3 px, 393 KB)

image.png (2×3 px, 368 KB)

image.png (2×3 px, 367 KB)

image.png (2×3 px, 379 KB)

image.png (2×3 px, 370 KB)

image.png (2×3 px, 368 KB)

image.png (2×3 px, 386 KB)

image.png (2×3 px, 381 KB)

Requirement

Ensure that various buttons on the Vector 2022 skin do not acquire unexpected link styling when hovered, maintaining consistent styling with no changes to text color or added underlines.

BDD

Feature: Button Styling in Vector 2022

  Scenario: Verify button styling on hover
    Given the user is using the Vector 2022 skin
    When the user hovers over various buttons
    Then the buttons should not change text color
    And the buttons should not gain underlines

Test Steps

Test Case 1: Verify Button Styling on Hover in Vector 2022

  1. Navigate to the Vector 2022 skin.
  2. Hover over various buttons.
  3. AC1: Confirm that the buttons do not change text color when hovered.
  4. AC2: Confirm that the buttons do not gain underlines when hovered.

QA Results - Beta

ACStatusDetails
1T367844#9904775

QA Results - Prod

ACStatusDetails
1T367844#10020084
2T367844#10020084

Event Timeline

Change #1047120 had a related patch set uploaded (by Bernard Wang; author: Bernard Wang):

[mediawiki/skins/Vector@master] Fix codex link styles overriding other link styles

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

ovasileva lowered the priority of this task from Unbreak Now! to High.Jun 18 2024, 4:08 PM

Change #1047120 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Fix codex link styles overriding other link styles

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

Change #1047155 had a related patch set uploaded (by Jdlrobson; author: Bernard Wang):

[mediawiki/skins/Vector@wmf/1.43.0-wmf.10] Fix codex link styles overriding other link styles

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

@Jdlrobson Please reviews issues below and let me know if you have any questions, thanks!

Test Result - Beta

Status: ❌ FAIL
Environment: Beta
OS: macOS Sonoma 14.5
Browser: Chrome 126
Device: MBA
Emulated Device: NA

Test Artifact(s):

Test Steps

❌ AC1: unexpected link styling when hovered, changing the text color to light blue or gaining underlines.

StatusLogin
Changes to lighter blue when hovering
2024-06-18_11-53-21.png (1×2 px, 447 KB)
StatusPublish Changes Grey
No blue underline
2024-06-18_11-46-06.png (666×2 px, 242 KB)
StatusPublish Changes Blue
No underline
2024-06-18_11-47-43.png (443×1 px, 88 KB)
StatusPublish Changes Blue
First name is not bold
2024-06-18_11-49-33.png (624×1 px, 187 KB)
StatusSearch for pages containing
No blue underline
2024-06-18_11-50-43.png (719×1 px, 217 KB)
StatusNotices1
Mark all as read is missing
2024-06-18_11-55-11.png (465×1 px, 195 KB)
StatusNotices2
No blue underline
2024-06-18_11-50-43.png (719×1 px, 217 KB)
StatusNotices3
No blue underline
2024-06-18_11-59-10.png (970×1 px, 475 KB)
Jdlrobson added a subscriber: GMikesell-WMF.

❌ AC1: unexpected link styling when hovered, changing the text color to light blue or gaining underlines.

I checked with Justin and this expected. It's part of the deliverable of T366515.

First name is not bold

Tracked in T367453 and unrelated.

Mark all as read is missing

Out of scope for this ticket - I think this relates to Echo behaviour - and only shows if notifications are unread (excluding the more notices line).

Change #1047155 merged by jenkins-bot:

[mediawiki/skins/Vector@wmf/1.43.0-wmf.10] Fix codex link styles overriding other link styles

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

Mentioned in SAL (#wikimedia-operations) [2024-06-18T21:03:07Z] <jdrewniak@deploy1002> Started scap: Backport for [[gerrit:1046790|Improve responsive images and avoid for inline (T367463)]], [[gerrit:1047155|Fix codex link styles overriding other link styles (T367844)]]

Mentioned in SAL (#wikimedia-operations) [2024-06-18T21:07:47Z] <jdrewniak@deploy1002> jdrewniak, jdlrobson: Backport for [[gerrit:1046790|Improve responsive images and avoid for inline (T367463)]], [[gerrit:1047155|Fix codex link styles overriding other link styles (T367844)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2024-06-18T21:09:33Z] <jdrewniak@deploy1002> Started scap: Backport for [[gerrit:1046790|Improve responsive images and avoid for inline (T367463)]], [[gerrit:1047155|Fix codex link styles overriding other link styles (T367844)]]

Mentioned in SAL (#wikimedia-operations) [2024-06-18T21:13:59Z] <jdrewniak@deploy1002> jdlrobson, jdrewniak: Backport for [[gerrit:1046790|Improve responsive images and avoid for inline (T367463)]], [[gerrit:1047155|Fix codex link styles overriding other link styles (T367844)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2024-06-18T21:26:07Z] <jdrewniak@deploy1002> Finished scap: Backport for [[gerrit:1046790|Improve responsive images and avoid for inline (T367463)]], [[gerrit:1047155|Fix codex link styles overriding other link styles (T367844)]] (duration: 16m 33s)

Looks good, thank you everyone!

Edtadros added a project: Verified.
Edtadros subscribed.

Test Result - Prod

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

Test Artifact(s):

Test Steps

Test Case 1: Verify Button Styling on Hover in Vector 2022

  1. Navigate to the Vector 2022 skin.
  2. Hover over various buttons.
  3. AC1: Confirm that the buttons do not change text color when hovered.

see below

  1. AC2: Confirm that the buttons do not gain underlines when hovered.

I didn't notice any underlines or text color changes

screenshot 12.mov.gif (1×1 px, 868 KB)

screenshot 11.mov.gif (1×1 px, 1 MB)

screenshot 13.mov.gif (1×1 px, 1 MB)