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

feat: high contrast theme #256

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

feat: high contrast theme #256

wants to merge 7 commits into from

Conversation

jdev082
Copy link

@jdev082 jdev082 commented Nov 24, 2022

No description provided.

Copy link
Author

@jdev082 jdev082 left a comment

Choose a reason for hiding this comment

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

.

Comment on lines 16 to 17
background-color: invert(100%);
color: invert(100%);
Copy link
Member

Choose a reason for hiding this comment

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

You can't use a filter on a color. If you used something like document.body.style.filter = 'invert(1)', it would technically work, but images would have their color inverted, and the inverts of many Scratch colors just don't look that great.

@jdev082
Copy link
Author

jdev082 commented Nov 24, 2022

@rgantzos requested changes implemented

@jdev082
Copy link
Author

jdev082 commented Nov 24, 2022

@rgantzos Alright, images are inverted a second time so that they uninvert.

@rgantzos rgantzos linked an issue Nov 24, 2022 that may be closed by this pull request
@jdev082
Copy link
Author

jdev082 commented Nov 25, 2022

@rgantzos I think its best to call this feature high contrast.

@jdev082 jdev082 closed this Nov 25, 2022
@jdev082 jdev082 reopened this Nov 25, 2022
@jdev082 jdev082 changed the title feat: add forced dark mode feat: high contrast theme Nov 25, 2022
@rgantzos
Copy link
Member

I'll work on a few changes just to keep some of the basic elements normal.

@callumjt
Copy link
Member

when will this be finished?

@jdev082
Copy link
Author

jdev082 commented Aug 19, 2023

Forgot about this PR

@callumjt
Copy link
Member

Forgot about this PR

Lol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants