Page MenuHomePhabricator

Enable night mode on talk page namespaces
Closed, ResolvedPublic

Description

Background

Currently night theme is not available on talk namespace, due to issues with OOUI in dark mode and Flow. Since talk pages are an important part of the wiki experience night theme should be made available.

User story

AS a user I want to view talk pages in night theme.

Requirements

  • All subtasks resolved.
  • A configuration change is made to enable night theme on talk pages.

Communication criteria - does this need an announcement or discussion?

  • Add communication criteria

Rollback plan

We have a configuration flag for controlling rollout that can be safely modified if necessary (see wmgMinervaNightModeExcludeNamespaces)

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

QA Results - Beta

ACStatusDetails
1T365509#9965955

Related Objects

StatusSubtypeAssignedTask
Resolvedovasileva
Resolvedovasileva
Resolvedovasileva
Resolvedovasileva
Resolvedovasileva
Öffnen SieBUG REPORTNone
ResolvedJdlrobson
DeclinedFeatureegardner
DuplicateNone
Resolvedovasileva
DuplicateBUG REPORTNone
DeclinedFeatureegardner
DuplicateCCiufo-WMF
Resolvedovasileva
ResolvedNBaca-WMF
DeclinedVolker_E
Resolvedovasileva
ResolvedBUG REPORTovasileva
Resolvedovasileva
DuplicateBUG REPORTNone
ResolvedBUG REPORTovasileva
Resolvedovasileva

Event Timeline

Jdlrobson renamed this task from Talk pages using DiscussionTools should appear in night mode to [GOAL] Talk pages using DiscussionTools should appear in night mode.May 21 2024, 5:12 PM
Jdlrobson renamed this task from [GOAL] Talk pages using DiscussionTools should appear in night mode to [GOAL] Talk page namespace should not be excluded from night theme.
ovasileva moved this task from Incoming to Epics/Goals on the Web-Team-Backlog board.
Jdlrobson renamed this task from [GOAL] Talk page namespace should not be excluded from night theme to [GOAL] Talk page namespaces should not be excluded from night theme.Jun 6 2024, 2:59 PM

Fixing VisualEditor (T366737) is a blocker for fixing discussion tools, as many of the rules we have there apply to talk pages too. Moving this out of the sprint board, as the subtasks now reflect the work involved here.

Jdlrobson renamed this task from [GOAL] Talk page namespaces should not be excluded from night theme to Enable night mode on talk page namespaces.Jul 1 2024, 10:03 PM
Jdlrobson removed ovasileva as the assignee of this task.
Jdlrobson assigned this task to ovasileva.
Jdlrobson added a subscriber: ovasileva.

@ovasileva I am assuming T357600 is not a blocker for enabling dark mode on talk pages? If so this is ready to go! Let me know!

Jdlrobson lowered the priority of this task from High to Medium.Jul 4 2024, 1:42 AM
ovasileva raised the priority of this task from Medium to High.Mon, Jul 8, 10:53 AM

@Edtadros - when we're ready for QA, could you retest the bugs from the table in T360691: QA Editing-related OOUI updates (mobile + desktop) as a part of this ticket?

@Jdlrobson @ovasileva Please review below, thanks!

Test Result - Beta

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

Test Artifact(s):

✅ Reply Tool toolbar not visible
2024-07-09_08-00-14.png (613×327 px, 49 KB)
Reply Tool (mobile)TBDRe-test as part of T365509
✅ Text labels within edit summary under not visible
2024-07-09_08-00-22.png (695×334 px, 56 KB)
Reply Tool (mobile)TBDRe-test as part of T365509
✅ New Topic Tool toolbar not visible
2024-07-09_08-01-41.png (617×328 px, 46 KB)
New Topic Tool (mobile)TBDRe-test as part of T365509
✅ Text labels within edit summary under not visible{
2024-07-09_08-02-05.png (714×331 px, 54 KB)
New Topic Tool Tool (mobile)TBDRe-test as part of T365509
✅ Affordance for editing/reporting a topic/section not visible:
2024-07-09_08-18-03.png (393×331 px, 32 KB)
Talk pages (mobile)TBDRe-test as part of T365509
✅ Affordance for editing/reporting a comment not visible:
2024-07-09_08-14-38.png (502×329 px, 45 KB)
Talk pages (mobile)TBDRe-test as part of T365509
❓ Highlighted texts are invisible. Mobile and Desktop highlights are different
2024-07-09_08-07-30.png (503×327 px, 45 KB)
2024-07-09_08-29-36.png (583×1 px, 164 KB)
Discussion Tool (desktop and mobile)webRe-test as part of T365509
❌ Text in <kbd /> are not visible because of the color. Icons are still white on dekstop. Is mobile supposed to have something?
2024-07-09_08-09-53.png (464×1 px, 136 KB)
2024-07-09_08-10-25.png (424×327 px, 29 KB)
Discussion Tool (desktop and mobile)webRe-test as part of T365509

❓Other Issues

Date contrast color
2024-07-08_15-02-18.png (1×3 px, 618 KB)
Desktop and Mobile
Add topic contrast color
2024-07-08_15-04-09.png (1×2 px, 310 KB)
Desktop only

I'll cut some bugs and then reassign to Olga.

❓ Highlighted texts are invisible. Mobile and Desktop highlights are different

I can't replicate this. Have reached out to George for replication steps, but it also doesn't necessarily need a bug - probably a question that can be raised in the talk-to-web channel.

❌ Text in <kbd /> are not visible because of the color. Icons are still white on dekstop. Is mobile supposed to have something?

Fixed on wiki. Documented on https://www.mediawiki.org/wiki/Recommendations_for_night_mode_compatibility_on_Wikimedia_wikis#Templates_/_modules

Date contrast color

Tracked in https://phabricator.wikimedia.org/T369667.

Add topic contrast color

Tracked in https://phabricator.wikimedia.org/T369666

@Ryasmeen - we've tested all original test cases. Is there any additional testing the editing team would like to do prior to deployment?

ppelberg subscribed.

Per what @ovasileva and I talked about offline during today's Editing - Web Team meeting, Editing is going to assume responsibility for testing talk pages to ensure the more targeted testing the Web Team completed in T365509#9965955 did not miss any issues.

With the above in mind, I'm moving this to the Editing QA board.

✅ Highlighted texts are invisible. Mobile and Desktop highlights are different

Screenshot 2024-07-27 at 05.28.10.png (746×2 px, 371 KB)

User talkTestUserEsther - Wikipedia, the free en….jpeg (2×1 px, 1 MB)

✅ Text in <kbd /> are not visible because of the color. Icons are still white on dekstop. Is mobile supposed to have something?

image.png (706×2 px, 338 KB)

@Jdlrobson @ovasileva Please review below, thanks!

Test Result - Beta

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

Test Artifact(s):

✅ Reply Tool toolbar not visible
2024-07-09_08-00-14.png (613×327 px, 49 KB)
Reply Tool (mobile)TBDRe-test as part of T365509
✅ Text labels within edit summary under not visible
2024-07-09_08-00-22.png (695×334 px, 56 KB)
Reply Tool (mobile)TBDRe-test as part of T365509
✅ New Topic Tool toolbar not visible
2024-07-09_08-01-41.png (617×328 px, 46 KB)
New Topic Tool (mobile)TBDRe-test as part of T365509
✅ Text labels within edit summary under not visible{
2024-07-09_08-02-05.png (714×331 px, 54 KB)
New Topic Tool Tool (mobile)TBDRe-test as part of T365509
✅ Affordance for editing/reporting a topic/section not visible:
2024-07-09_08-18-03.png (393×331 px, 32 KB)
Talk pages (mobile)TBDRe-test as part of T365509
✅ Affordance for editing/reporting a comment not visible:
2024-07-09_08-14-38.png (502×329 px, 45 KB)
Talk pages (mobile)TBDRe-test as part of T365509
❓ Highlighted texts are invisible. Mobile and Desktop highlights are different
2024-07-09_08-07-30.png (503×327 px, 45 KB)
2024-07-09_08-29-36.png (583×1 px, 164 KB)
Discussion Tool (desktop and mobile)webRe-test as part of T365509
❌ Text in <kbd /> are not visible because of the color. Icons are still white on dekstop. Is mobile supposed to have something?
2024-07-09_08-09-53.png (464×1 px, 136 KB)
2024-07-09_08-10-25.png (424×327 px, 29 KB)
Discussion Tool (desktop and mobile)webRe-test as part of T365509

❓Other Issues

Date contrast color
2024-07-08_15-02-18.png (1×3 px, 618 KB)
Desktop and Mobile
Add topic contrast color
2024-07-08_15-04-09.png (1×2 px, 310 KB)
Desktop only

Can this be resolved ?

With T369667 and T369666 being filed (thank you, @Jdlrobson), I think, yes.

@EAkinloose: do you agree? Are there any issues that are both NOT resolved and without Phabricator tickets?

I totally agree. We have tracking tickets already. I'll verify.

Can this be resolved ?

With T369667 and T369666 being filed (thank you, @Jdlrobson), I think, yes.

@EAkinloose: do you agree? Are there any issues that are both NOT resolved and without Phabricator tickets?

I totally agree. We have tracking tickets already. I'll verify.

Can this be resolved ?

With T369667 and T369666 being filed (thank you, @Jdlrobson), I think, yes.

@EAkinloose: do you agree? Are there any issues that are both NOT resolved and without Phabricator tickets?

Wonderful; thank you, @EAkinloose.

@Jdlrobson: this ticket is ready for you to resolve!