[go: up one dir, main page]

Page MenuHomePhabricator

Special:NewPage has unreadable text in the dark mode
Closed, ResolvedPublicBUG REPORT

Assigned To
Authored By
Ebrahim
Sep 7 2024, 7:45 AM
Referenced Files
F57573484: image.png
Sep 30 2024, 9:16 PM
Restricted File
Sep 16 2024, 5:49 PM
F57475153: image.png
Sep 7 2024, 10:24 AM
F57475150: image.png
Sep 7 2024, 10:24 AM
F57475145: image.png
Sep 7 2024, 10:24 AM
F57475142: image.png
Sep 7 2024, 10:24 AM
F57474577: image.png
Sep 7 2024, 7:45 AM

Description

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

Have patrol rights on fawiki

https://fa.wikipedia.org/wiki/Special:NewPages?safemode=1&useskin=vector-2022&vectornightmode=1

What happens?:
White text on yellow background

image.png (720×1 px, 201 KB)

I guess that needs some right to be seen.

What should have happened instead?:
Something other than light yellow

Event Timeline

Change #1071291 had a related patch set uploaded (by Ebrahim; author: Ebrahim):

[mediawiki/core@master] Fix Special:NewPages unpatrolled items color in dark mode

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

Ammarpad updated the task description. (Show Details)
Restricted Application added a subscriber: Huji. · View Herald TranscriptSep 7 2024, 9:17 AM

The patch in it's current state https://gerrit.wikimedia.org/r/1071291 turns

image.png (470×974 px, 174 KB)

image.png (446×1 px, 162 KB)

To

image.png (508×1 px, 242 KB)

image.png (440×1 px, 202 KB)

As it uses the background-color-warning-subtle token, not sure if that is ideal, maybe we can just have some arbitrary code for the dark mode (if so let me know about your suggested color code for the dark mode fit), maybe it could be better if we had the non subtle version of warning background color or maybe it's fine.

Change #1071291 merged by jenkins-bot:

[mediawiki/core@master] Fix Special:NewPages unpatrolled items color in dark mode

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

Ebrahim closed this task as Resolved.EditedSep 7 2024, 1:18 PM
Ebrahim claimed this task.

The patch is now merged, if you got any feedback and have a request for a specific custom color code I'm here to upload a patch for that. Thanks

I don't think these changes are actually an improvement in light mode. I've heard and read from several people of the Dutch Wikipedia they find the new background color of unpatrolled new pages too light. Concerns about this background color were already raised in T370423, this task is a duplicate of that one.

Nemoralis reopened this task as Open.EditedSep 16 2024, 5:38 PM
Nemoralis subscribed.

Color change should be reconsidered. New color is too light, you can barely see it.
{F57515674}

No problem, as said give me the color code you want both for the light (perhaps the previous color) and the dark mode and I'll send a patch and a reviewer will decide (if you don't want to upload the patch yourself). All I did was going with one Codex color which apparently wasn't enough so a custom color is needed.

It has turned from

(unreadable white text on yellow background, which this file was about)

image.png (720×1 px, 201 KB)

to this

image.png (628×774 px, 165 KB)

so if it's not satisfying, both in the light and dark colors, please file the issue in a separate task with screenshot and your suggestions for the colors and add me as a CC to the task. Thanks