Page MenuHomePhabricator

DiscussionTools source mode preview is not night mode friendly
Closed, ResolvedPublic3 Estimated Story PointsBUG REPORT

Description

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

What happens?:

  • The text in the input is not visible (note this will be addressed by T366197)
  • The preview is not visible

What should have happened instead?:
Both should be visible.

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

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

QA Results - Beta

ACStatusDetails
1T367432#9935527

QA Results - PROD

ACStatusDetails
1 ✅T367432#9971257

Event Timeline

Jdlrobson triaged this task as High priority.

Options to fix this:

  1. Add notheme to ext-discussiontools-ui-replyWidget-bodyWrapper and ext-discussiontools-ui-replyWidget-preview
  2. Add skin-invert to ext-discussiontools-ui-replyWidget-preview and update tokens in to ext-discussiontools-ui-replyWidget-bodyWrapper (it doesn't look like invert works on the input)
  3. Update to ext-discussiontools-ui-replyWidget-bodyWrapper and ext-discussiontools-ui-replyWidget-preview to use design tokens

As far as I see,

  • The visual mode isn’t better: a visual draft is also white-on-white (except for links and when selected).
  • The currently not selected tab, as well as all toolbar buttons are black-on-black (unless hovered).
Source modeVisual mode
Screenshot 2024-06-13 at 17-53-04 Talk T352930 - Wikipedia the free encyclopedia.png (551×1 px, 75 KB)
Screenshot 2024-06-13 at 17-53-28 Talk T352930 - Wikipedia the free encyclopedia.png (490×1 px, 66 KB)

As far as I see,

  • The visual mode isn’t better: a visual draft is also white-on-white (except for links and when selected).
  • The currently not selected tab, as well as all toolbar buttons are black-on-black (unless hovered).
Source modeVisual mode
Screenshot 2024-06-13 at 17-53-04 Talk T352930 - Wikipedia the free encyclopedia.png (551×1 px, 75 KB)
Screenshot 2024-06-13 at 17-53-28 Talk T352930 - Wikipedia the free encyclopedia.png (490×1 px, 66 KB)

This should be covered by T366197: VisualEditor should use Codex design tokens for custom colors

As far as I know, the source mode input box is also a VisualEditor (2017WTE) input – unless the source mode toolbar is turned off, in which case it’s an OOUI textbox, and is at least readable:

Screenshot 2024-06-14 at 09-35-41 Talk Pywikibot March Workshop_Archive 1 - Test Wikipedia.png (400×948 px, 43 KB)

So then only the preview is a task specific to DiscussionTools?

Jdlrobson renamed this task from Discussion tools source tab is not night mode friendly to Discussion tools preview source tab is not night mode friendly.Jun 14 2024, 2:51 PM
Jdlrobson updated the task description. (Show Details)

Yeh looks like it. If you add the following rule it fixes the input areas:

.ve-ui-targetWidget:not(.oo-ui-pendingElement-pending) {background: var(--background-color-base); }

Devs feel this is better estimated when VisualEditor is resolved.

Tacsipacsi renamed this task from Discussion tools preview source tab is not night mode friendly to DiscussionTools source mode preview is not night mode friendly.Jun 16 2024, 1:17 PM

I don’t know about any other task related to in-page parts of DiscussionTools, so I put some other color contrast issues I’ve just noticed here, letting y’all decide if they need a separate task(s):

Timestamp (permalink)Reply button
Screenshot from 2024-06-18 17-20-44.png (680×940 px, 120 KB)
Screenshot from 2024-06-18 17-21-23.png (680×940 px, 119 KB)

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

[mediawiki/extensions/DiscussionTools@master] Fixes the preview area on discussion tools

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

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

[mediawiki/extensions/DiscussionTools@master] Add notheme to discussion source mode preview

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

bwang removed bwang as the assignee of this task.Jun 27 2024, 2:53 PM
bwang subscribed.

Change #1049999 merged by jenkins-bot:

[mediawiki/extensions/DiscussionTools@master] Fix white on white text in source preview

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

Change #1048146 abandoned by Jdlrobson:

[mediawiki/extensions/DiscussionTools@master] Disable preview area background in dark mode

Reason:

Done in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/DiscussionTools/+/1049999

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

The change is now on the beta cluster and talk pages are suddenly looking a lot better!
https://en.wikipedia.beta.wmflabs.org/w/index.php?title=Talk:T352930&action=edit&redlink=1
Thanks for the assist @Esanders

@Jdlrobson Text input, preview, and source are now visible. I will move this to sign-off. I'm not sure if you wanted to count the date in the preview box as a contrast issue or not but I'm just letting you know about it, thanks!

Test Result - Beta

Status: ✅ PASS
Environment: Beta
OS: macOS Sonoma 14.5
Browser: Chrome 126
Device: MBA
Emulated Device: NA

✅AC1: The text in the input and preview is visible

2024-06-28_11-12-17.png (996×2 px, 313 KB)

ovasileva claimed this task.

@Jdlrobson Please review PROD when compared to Beta. It's different and the color contrast is off, thanks!

Test Result - PROD

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

❌AC1: The text in the input and preview is visible

PRODBeta
2024-07-10_11-56-37.mp4.gif (940×1 px, 2 MB)
2024-07-10_11-56-09.mp4.gif (1×1 px, 1 MB)

Hi @GMikesell-WMF these look like they are testing 2 different workflows - you need to click "reply" on English Wikipedia. Looks like that is working okay?

@Jdlrobson Got it. It was weird that Beta went straight to a reply compared to PROD. It now passes, thanks!

Test Result - PROD

Status: ✅ PASS
Environment: PROD
OS: macOS Sonoma 14.5
Browser: Chrome 126
Device: MBA
Emulated Device: NA

✅ AC1: The text in the input and preview is visible

PRODBeta
2024-07-10_13-10-29.mp4.gif (894×1 px, 1 MB)
2024-07-10_11-56-09.mp4.gif (1×1 px, 1 MB)

It was weird that Beta went straight to a reply compared to PROD.

It looks like you’ve already had a draft on beta (saved in the browser local storage), so when you opened the talk page, DiscussionTools immediately restored it (notice the notification that’s partly visible in the top right corner).