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

White and Black & Black and White #11872

Conversation

ChiefMateStarbuck
Copy link

@ChiefMateStarbuck ChiefMateStarbuck commented Oct 11, 2024

This pull request introduces two new themes, white_and_black and black_and_white, to the runtime. They are monochrome, which will help those with vision impairment.

white_and_black is a light theme.
Screenshot 2024-10-13 at 10 19 21 PM

black_and_white is a dark theme.
Screenshot 2024-10-13 at 10 19 35 PM

@gj1118
Copy link
Contributor

gj1118 commented Oct 13, 2024

I tried the dark mode. Can you please change the theme in such a way that the code when the ruler and line highlight is turned on ?

image

@the-mikedavis the-mikedavis added the A-theme Area: Theme and appearence related label Oct 14, 2024
@ChiefMateStarbuck
Copy link
Author

I tried the dark mode. Can you please change the theme in such a way that the code when the ruler and line highlight is turned on ?

image

Hello. I don't seem to understand. Should the vertical and horizontal lines be different colors? Should the cursor also be a different color than the vertical/horizontal lines?

@gj1118
Copy link
Contributor

gj1118 commented Oct 15, 2024

I tried the dark mode. Can you please change the theme in such a way that the code when the ruler and line highlight is turned on ?
image

Hello. I don't seem to understand. Should the vertical and horizontal lines be different colors? Should the cursor also be a different color than the vertical/horizontal lines?

Yes please.

@ChiefMateStarbuck
Copy link
Author

I tried the dark mode. Can you please change the theme in such a way that the code when the ruler and line highlight is turned on ?
image

Hello. I don't seem to understand. Should the vertical and horizontal lines be different colors? Should the cursor also be a different color than the vertical/horizontal lines?

Yes please.

Hello @gj1118. It's uncertain if this is possible with only 4 colors. I'm open to solutions. The vertical line has been slightly darkened.

@ChiefMateStarbuck
Copy link
Author

@archseer @gj1118 Hello.
Any feedback?
I’m very willing to keep iterating on this. Just let me know what to fix.
CC . @the-mikedavis

Copy link
Member

@the-mikedavis the-mikedavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like most of the styles are shared between both themes. Can these use inheritance instead - one theme defines all of the styles and the other makes small overrides and changes the palette? The gruvbox themes are a good example of inheritance

@ChiefMateStarbuck
Copy link
Author

hello, would like both themes to inherit from a base-theme, or for one theme to inherit from the other?

In the first scenario, would the base-theme be usable? The base-theme could be a less aggressive light theme.

@the-mikedavis
Copy link
Member

the-mikedavis commented Dec 22, 2024

I would just have one inherit the other. There isn't a way to make a base theme that doesn't show up in the :theme list

@ChiefMateStarbuck
Copy link
Author

After extensive use with these themes, I find them to be abrasive and a pain on the eyes.

The light theme is too white. It's like reading letters on the suns surface.

Reading the white text on the dark screen also cause squinting and fatigue.

I would like to reject my on PR and close this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-theme Area: Theme and appearence related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants