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

Improve icon readability in light theme #2323

Merged
merged 2 commits into from
Dec 18, 2022

Conversation

NickColley
Copy link
Collaborator

@NickColley NickColley commented Dec 17, 2022

Increase the contrast of unpressed action buttons so they're easier to see.

This brings the icons to 3:1 contrast while keeping colour in themes.

Before

After

Boost contrast of the default colour theme, to be closer to
the other theme's saturation then boost the unpressed state for action
buttons.

This brings the icons to 3:1 contrast while keeping colour in themes.
@NickColley NickColley force-pushed the improve-default-theme-icon-unpressed-state branch from 7266184 to 86173ed Compare December 18, 2022 12:53
@NickColley NickColley changed the title fix: icon colour contrast unpressed Improve icon readability in light theme Dec 18, 2022
Copy link
Owner

@nolanlawson nolanlawson left a comment

Choose a reason for hiding this comment

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

Overall this is awesome and I'm really happy to keep the colors while still getting to that 3:1 ratio. I just have one small nitpick and then I'm 👍.

@NickColley
Copy link
Collaborator Author

Niceee 👍

@vercel
Copy link

vercel bot commented Dec 18, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
pinafore 🔄 Building (Inspect) Dec 18, 2022 at 8:05PM (UTC)

@nolanlawson nolanlawson merged commit f81778d into master Dec 18, 2022
alice-werefox pushed a commit to alice-werefox/sema-werefox-cafe that referenced this pull request Apr 3, 2023
Boost contrast of the default colour theme, to be closer to
the other theme's saturation then boost the unpressed state for action
buttons.

This brings the icons to 3:1 contrast while keeping colour in themes.

Co-authored-by: Nolan Lawson <nolan@nolanlawson.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Accessibility (a11y)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants