Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the dark skin more customizable #136

Open
starlightsparks opened this issue Sep 17, 2024 · 3 comments
Open

Make the dark skin more customizable #136

starlightsparks opened this issue Sep 17, 2024 · 3 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@starlightsparks
Copy link

starlightsparks commented Sep 17, 2024

I propose we make the dark skin more customizable—Being able to change the header(which is grey).

@Kenny2github
Copy link
Member

If by "accent color" you mean that of links etc., customizing that is not something we will support. It's too tightly tied to the rest of the design, and it's not customizable on light mode anyway.

The header color is not customizable on dark mode mainly to avoid users setting a header color in light mode, then switching to dark mode and finding that the header is unreadable due to the inverted text color. I'm open to allowing dark theme header color customization as long as it a) does not involve adding a separate color setting, b) guarantees that the theme itself and the change to/from it cannot cause unreadable headers, and c) isn't any more complicated than the current setting. If you have specific suggestions on that front, please feel free to share.

@starlightsparks
Copy link
Author

starlightsparks commented Sep 21, 2024

Forget the accent color, I have removed it from OP due to your given reasons.

And as for the header;
a) May you clarify?

b) Could there be a system where it automatically inverts text color on the header if the color is above or below a certain lightness/darkness? (I presume, if this was implemented, it’d be in light mode as well.) I believe this is possible, as I recall an extension for Scratch (not Scratch Wiki, but I’m using this as an example since code is code) does it—But I’m unsure if it’s hard or not. But regardless; Is it not the user’s own fault if they deliberately choose to keep the color an unreadable one? I mean, I feel like most people would connect the dots that “Hey, the text color is different in dark mode, so I could change the header color to a different shade for contrast”, including most children over the age of 8. However, the former (making it so that text color automatically inverts depending on the header color) is still a better solution then to leave it be cause of the latter reason, since it has other benefits, if possible.

c) Yeah, it isn’t unless the thing I proposed in my answer to b) is implemented. So I do still stand with my latter point in b), if my former point in b) is too complicated.

@Kenny2github
Copy link
Member

a) By this I mean we will not be adding another field to Special:Preferences - the user must not be required to do anything new, such as having to choose the text color separately.
b) I'd be open to a system that does the inversion you describe in compliance with WCAG guidelines. I'm marking this issue "help wanted" if anyone feels like trying their hand at implementing that.

@Kenny2github Kenny2github added help wanted Extra attention is needed good first issue Good for newcomers labels Oct 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants