Page MenuHomePhabricator

Suggested Edits: unstructured tasks onboarding elements not readable in Dark Mode
Closed, ResolvedPublicBUG REPORT

Description

Background

The Web team is releasing dark mode more widely. To prepare for this, some Growth maintained features need improvements.

Related task

T371901: Provide Add a link on-boarding assets without embedded background colors

User story

  • As a new editor who uses dark mode, I want the ability to use dark mode when viewing the Suggested Edit onboarding (of a unstructured task)
Suggested Edit onboarding - many elements are non-readable
Screen Shot 2024-06-22 at 4.27.09 PM.png (1×830 px, 152 KB)
Acceptance Criteria:

Given I'm completing a Suggested Edit,
When I'm viewing the onboarding in dark mode,
Then all elements are readable and pass WCAG AA contrast requirements

Event Timeline

KStoller-WMF created this task.
KStoller-WMF moved this task from Inbox to Up Next on the Growth-Team board.
Sgs renamed this task from Suggested Edits: some onboarding elements not readable in Dark Mode to Suggested Edits: unstructured tasks onboarding elements not readable in Dark Mode .Aug 6 2024, 11:57 AM

There are a couple of issues impeding us to use the Codex background color token that should apply here (@background-color-progressive-subtle, #eaf3ff) to fix the contrast issues in the help panel header and content body header. See T357740: Less color functions not compatible with CSS custom properties.

Issue (1) is, in light mode, both elements use the progressive subtle color faded by a 50% using Less's fade() function see source.

Issue (2) is, in light mode, the header element uses Less's line-gradient function, adding some fancy styles including an animation of the back-ground color.

The proposal to move forward, as mentioned in T357740, would be to have an extended color palette using CSS hsl/hsla colors until Codex provides it. That should solve the fading issues. As per the linear-gradient , the refactor to CSS is not direct but should be doable. In worst case we could drop the color gradation until Codex has figured this out. cc @Michael

Change #1062065 had a related patch set uploaded (by Sergio Gimeno; author: Sergio Gimeno):

[mediawiki/extensions/GrowthExperiments@master] refactor(AddLinkOnboardingContent): introduce mixin for mark highlight styles

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

Change #1062066 had a related patch set uploaded (by Sergio Gimeno; author: Sergio Gimeno):

[mediawiki/extensions/GrowthExperiments@master] SuggestedEdits: fix dark mode contrast issues in higlighted text

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

Change #1067925 had a related patch set uploaded (by Michael Große; author: Michael Große):

[mediawiki/extensions/GrowthExperiments@master] fix: dark-mode: Make highlights in onboarding dialogues readable

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

Change #1062065 abandoned by Sergio Gimeno:

[mediawiki/extensions/GrowthExperiments@master] refactor(AddLinkOnboardingContent): introduce mixin for mark highlight styles

Reason:

Drop in favor of Ic34f415f3efc79579690316e1146034ff5fd3521

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

Change #1062066 abandoned by Sergio Gimeno:

[mediawiki/extensions/GrowthExperiments@master] SuggestedEdits: fix dark mode contrast issues in higlighted text

Reason:

Drop in favor of Ic34f415f3efc79579690316e1146034ff5fd3521

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

Change #1067925 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] fix: dark-mode: Make highlights in onboarding dialogues readable

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

Checked in betalabs - the text is readable in dark mode

Screen Shot 2024-09-19 at 5.10.57 PM.png (974×838 px, 109 KB)
Screen Shot 2024-09-19 at 5.06.44 PM.png (1×854 px, 143 KB)

The suggested-edits-panel-header elements have low contrast - filed as T375250.