Page MenuHomePhabricator

[Bug] Echo not compatible with desktop night theme
Closed, ResolvedPublic2 Estimated Story PointsBUG REPORT

Description

NOTE: Replace with beta cluster link when fix for https://phabricator.wikimedia.org/T354889 is merged.

Steps to replicate the issue (include links if applicable):

What happens?:

Screenshot 2024-04-29 at 4.14.16 PM.png (900×2 px, 1 MB)

What should have happened instead?:

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):

Notes for first-time taskers

  • you will need to install Extension:Echo to test this
  • The fix will live in the https://www.mediawiki.org/wiki/Extension:Echo
  • Adding the notheme class to any element will make the element look the same in night and day themes - this should be used here.

Event Timeline

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

[mediawiki/extensions/Echo@master] Add notheme class to Echo

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

bwang removed bwang as the assignee of this task.May 8 2024, 5:22 PM
bwang subscribed.

Change #1028880 merged by jenkins-bot:

[mediawiki/extensions/Echo@master] Add notheme class to Echo

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

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

[mediawiki/extensions/Echo@wmf/1.43.0-wmf.4] Add notheme class to Echo

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

Edtadros subscribed.

Test Result - Beta

Status: ❌ Fail
Environment: beta
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

Test Steps

Login
visit https://en.wikipedia.beta.wmflabs.org/w/index.php?title=Dog&vectornightmode=1
Load Echo
Check accessibility color contrast issues
❌ AC1: There should be no color contrast issues.
It is not clear what the expected result is, the notification dropdown is legible but has the following WCAG issues

screenshot 336.png (1×1 px, 497 KB)

screenshot 335.png (1×1 px, 497 KB)

screenshot 337.png (1×1 px, 505 KB)

Change #1030984 merged by jenkins-bot:

[mediawiki/extensions/Echo@wmf/1.43.0-wmf.4] Add notheme class to Echo

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

Mentioned in SAL (#wikimedia-operations) [2024-05-14T15:58:16Z] <logmsgbot> lucaswerkmeister-wmde@deploy1002 Started scap: Backport for [[gerrit:1030984|Add notheme class to Echo (T363779)]], [[gerrit:1031180|Convert function to arrow function to fix context (T364783)]]

Mentioned in SAL (#wikimedia-operations) [2024-05-14T16:05:33Z] <logmsgbot> lucaswerkmeister-wmde@deploy1002 jdlrobson and jforrester and lucaswerkmeister-wmde: Backport for [[gerrit:1030984|Add notheme class to Echo (T363779)]], [[gerrit:1031180|Convert function to arrow function to fix context (T364783)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2024-05-14T16:21:00Z] <logmsgbot> lucaswerkmeister-wmde@deploy1002 Finished scap: Backport for [[gerrit:1030984|Add notheme class to Echo (T363779)]], [[gerrit:1031180|Convert function to arrow function to fix context (T364783)]] (duration: 22m 43s)