Page MenuHomePhabricator

[Config change] Enable night theme on pages which use data tables
Closed, ResolvedPublic1 Estimated Story Points

Description

NOTE: 20th June is the earliest this can currently be deployed (when OOUI update will hopefully be enabled on all projects)

Background

Night theme doesn't work on Special:TrackingCategories

User story

As a user I want all my pages in dark mode.

Acceptance criteria

Derived Requirement

Enable the night theme on pages that use data tables, ensuring consistency and readability across different themes.

BDD

Feature: Enable Night Theme on Data Table Pages

Scenario: Display night theme on data table pages

Given a user has enabled the night theme
When the user views a page containing data tables
Then the page should display using the night theme
And the data tables should be styled consistently with the night theme
Test Steps

Test Case 1: Verify Night Theme on Data Table Pages

Enable the night theme on the site.
Navigate to a page containing data tables.
✅❓❌⬜ AC1: Confirm the page displays using the night theme and verify the data tables are styled consistently.

QA

Check that none of the pages have:

  1. AA color contrast issues for text that is not disabled (these are okay)
  2. icons that are the same color as the background.

Communication criteria - does this need an announcement or discussion?

N/A

Rollback plan

N/A

This task was created by Version 1.0.0 of the Web team task template using phabulous

QA Results - PROD

ACStatusDetails
1T366373#9983202

Event Timeline

Jdlrobson renamed this task from Enable night theme on Special:TrackingCategories to Enable night theme on pages which use data tables.May 31 2024, 4:14 PM
Jdlrobson updated the task description. (Show Details)
Jdlrobson triaged this task as Medium priority.May 31 2024, 4:49 PM

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

[mediawiki/core@master] Fix data tables in night mode

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

Change #1037837 merged by jenkins-bot:

[mediawiki/core@master] Pager: Fix data tables in night mode

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

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

[mediawiki/core@master] DataTables: Fix theme in os mode

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

Change #1038374 abandoned by Jdlrobson:

[mediawiki/core@master] DataTables: Fix theme in os mode

Reason:

Fixed in https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1037837/3/resources/src/mediawiki.pager.styles/DataTable.less

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

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

[operations/mediawiki-config@master] Enable dark mode on data table pages

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

Jdlrobson changed the task status from Open to Stalled.Jun 10 2024, 11:12 PM
Jdlrobson updated the task description. (Show Details)

Patch can be backported 20th at the latest (as the OOUI release is required)

I reviewed all these pages, and it looks like none of them have color contrast issues. The earliest this can happen is June 20th.

Jdlrobson renamed this task from Enable night theme on pages which use data tables to [Config change] Enable night theme on pages which use data tables.Jun 12 2024, 9:18 PM
Jdlrobson changed the task status from Stalled to Open.
Jdlrobson updated the task description. (Show Details)
Jdlrobson set the point value for this task to 1.Jun 13 2024, 12:13 AM

Adding 1 as this is a configuration change.

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

[operations/mediawiki-config@master] Enable dark mode on more pages

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

Change #1042431 abandoned by Jdlrobson:

[operations/mediawiki-config@master] Enable dark mode on more pages

Reason:

Folded into https://gerrit.wikimedia.org/r/c/operations/mediawiki-config/+/1041250

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

Change #1041250 merged by jenkins-bot:

[operations/mediawiki-config@master] Enable dark mode on more pages

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

Mentioned in SAL (#wikimedia-operations) [2024-06-25T20:19:01Z] <cjming@deploy1002> jdlrobson, cjming: Backport for [[gerrit:1041250|Enable dark mode on more pages (T366378 T367374 T366373 T366520 T366373)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2024-06-25T20:31:25Z] <cjming@deploy1002> Finished scap: Backport for [[gerrit:1041250|Enable dark mode on more pages (T366378 T367374 T366373 T366520 T366373)]] (duration: 15m 04s)

@Jdlrobson Please review issues below, thanks!

Test Result - PROD

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

Test Artifact(s):

Test Steps

Test Case 1: Verify Night Theme on Data Table Pages

  1. Enable the night theme on the site.
  2. Navigate to a page containing data tables. Night mode is enabled on https://en.wikipedia.org/wiki/Special:TrackingCategories Night mode is enabled on https://en.wikipedia.org/w/index.php?title=Special:AutoblockList Night mode is enabled on https://en.wikipedia.org/wiki/Special:BlockList?safemode=1 Night mode is enabled on https://en.wikipedia.org/w/index.php?title=Special:AbuseLog Night mode is enabled on https://en.wikipedia.org/w/index.php?title=Special:AbuseFilter Night mode is enabled on https://en.wikipedia.org/wiki/Special:BlockedExternalDomains https://en.wikipedia.org/w/index.php?title=Special:Tags https://en.wikipedia.org/w/index.php?title=Special:TopicSubscriptions https://en.wikipedia.org/w/index.php?title=Special:ListFiles Night mode is enabled on https://en.m.wikipedia.org/wiki/Special:SecurePoll Night mode is enabled on https://en.wikipedia.org/wiki/Special:PageAssessments?project=&namespace=0&page_title=San+Francisco

❌ AC1: Confirm the page displays using the night theme.

image.png (44×26 px, 4 KB)
"Search" contrast failed & table is off & overlaps
2024-07-15_12-33-57.png (1×2 px, 413 KB)
"Search" contrast failed
2024-07-15_12-37-13.png (1×2 px, 436 KB)
2024-07-15_12-38-33.png (1×2 px, 759 KB)
Some are grey for some reason like "by", "Public",etc
2024-07-15_12-39-56.png (1×3 px, 579 KB)
2024-07-15_12-41-56.png (1×3 px, 569 KB)
Table is off and overlaps
2024-07-15_12-42-35.png (1×3 px, 482 KB)
2024-07-15_12-43-27.png (1×3 px, 298 KB)
"Show file list" contrast failed & table is off & overlaps
2024-07-15_12-44-09.png (1×3 px, 586 KB)
2024-07-15_12-44-55.png (1×3 px, 548 KB)
"Search" contrast failed
2024-07-15_12-45-29.png (1×3 px, 403 KB)

Thanks!

"Search" contrast failed & table is off & overlaps {FThis is now fixed on test.wikipedia.org (Was reported in another bug). Overlap reported in T370188
Some are grey for some reason like "by", "Public",etc
2024-07-15_12-39-56.png (1×3 px, 579 KB)
Tracked in T370191
Table is off and overlaps
2024-07-15_12-42-35.png (1×3 px, 482 KB)
Overlap reported in T370188
"Show file list" contrast failed & table is off & overlaps
2024-07-15_12-44-09.png (1×3 px, 586 KB)
This is now fixed on test.wikipedia.org (Was reported in another bug). Overlap reported in T370188
"Search" contrast failed
2024-07-15_12-45-29.png (1×3 px, 403 KB)
This is now fixed on test.wikipedia.org (Was reported in another bug)