⚓ T357166 Categories appear in light theme
Page MenuHomePhabricator

Categories appear in light theme
Closed, ResolvedPublic0 Estimated Story Points

Description

This should be automatically fixed once T334790 is resolved.

In advanced mode visit https://en.m.wikipedia.org/wiki/Spain?minervanightmode=1 and scroll to bottom

Screenshot 2024-02-09 at 10.29.55 AM.png (644×2 px, 126 KB)

Expected: No color contrast issues. Should appear blue links on dark background.

QA Results - Beta

ACStatusDetails
1T357166#9685923

QA Results - Prod

ACStatusDetails
1T357166#9702684

Event Timeline

Jdlrobson renamed this task from [Tracking] Categories appear in light theme to Categories appear in light theme.Feb 23 2024, 3:21 PM
Jdlrobson added a project: Web-Team-Backlog.

Change 997978 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Skin: Use design tokens rather than hardcoded values

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

Jdlrobson triaged this task as Medium priority.Feb 29 2024, 4:26 PM

Change 1010317 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Skin: Use design tokens rather than hardcoded values

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

Change 997978 abandoned by Jdlrobson:

[mediawiki/core@master] Skin: Use LESS variable for design tokens rather than CSS variables

Reason:

Merged into https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1010317/1?usp=related-change

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

Change 1010317 merged by jenkins-bot:

[mediawiki/core@master] Skin: Use design tokens rather than hardcoded values

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

Jdlrobson set the point value for this task to 0.
Jdlrobson moved this task from Incoming to QA on the Web-Team-Backlog (FY2023-24 Q3 Sprint 6) board.

This can be QAed as part of sprint 6.

Change #1015657 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Define background-color-neutral-subtle in dark mode palette

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

This does not appear to be working. --background-color-neutral-subtle only seems to be defined for light mode theme. https://gerrit.wikimedia.org/r/c/mediawiki/skins/MinervaNeue/+/1015657 fixes it.

Change #1015657 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Define background-color-neutral-subtle in dark mode palette

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

Edtadros subscribed.

Test Result - Beta

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

Test Artifact(s):

QA Steps

In advanced mode visit https://en.m.wikipedia.org/wiki/Spain?minervanightmode=1 and scroll to bottom
✅ AC1: Expected: No color contrast issues. Should appear blue links on dark background.

screenshot 166.png (826×1 px, 231 KB)

screenshot 165.png (826×1 px, 231 KB)

Test Result - Prod

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

Test Artifact(s):

QA Steps

In advanced mode visit https://en.m.wikipedia.org/wiki/Spain?minervanightmode=1 and scroll to bottom
✅ AC1: Expected: No color contrast issues. Should appear blue links on dark background.

screenshot 195.png (1×1 px, 372 KB)

screenshot 196.png (1×1 px, 459 KB)