Page MenuHomePhabricator

Two column edit conflict needs dark mode
Open, Needs TriagePublicBUG REPORT

Description

Ending up in an edit conflict with two column merge turned on needs a dark mode treatment.

Some of the issues are:

  • The edit buttons are completely invisible (white on white).
  • Invisible text (black on black) in the help popups.
  • Some places with white text on bright backgrounds.
  • … possibly more?

Screenshot from 2024-10-09 09-57-56_.png (953×825 px, 147 KB)

For comparison, this is how diff and editor should look like (screenshot from the normal 2010 wikitext editor):

Screenshot from 2024-10-09 10-02-03.png (942×490 px, 68 KB)

Event Timeline

thiemowmde changed the subtype of this task from "Task" to "Bug Report".

Change #1078905 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/TwoColConflict@master] [WIP] Full fix for TwoColConflict in dark mode

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

Change #1078906 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/TwoColConflict@master] Make use of more (non-color) LESS variables

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

Change #1078910 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/TwoColConflict@master] Replace most problematic hard-coded colors with skin colors

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

Change #1078911 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/TwoColConflict@master] Add notheme hack to make edit buttons visible in dark mode

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

Change #1078906 merged by jenkins-bot:

[mediawiki/extensions/TwoColConflict@master] Make use of more (non-color) LESS variables

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

Change #1078910 merged by jenkins-bot:

[mediawiki/extensions/TwoColConflict@master] Replace most problematic hard-coded colors with skin colors

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

Here is a comparison for what the quick, temporary notheme hack in https://gerrit.wikimedia.org/r/1078911 does:

Screenshot from 2024-10-11 09-44-45.png (693×830 px, 127 KB)

Screenshot from 2024-10-11 09-44-50.png (693×830 px, 128 KB)

I need help with the final patch https://gerrit.wikimedia.org/r/1078905. I can't get the LESS functions working. Why is that?

Change #1078911 merged by jenkins-bot:

[mediawiki/extensions/TwoColConflict@master] Add notheme hack to make edit buttons visible in dark mode

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