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 navigation readability #2274

Merged
merged 1 commit into from
Dec 2, 2022
Merged

Conversation

NickColley
Copy link
Collaborator

@NickColley NickColley commented Nov 28, 2022

Make the background of selected navigation items darker so it is easier to read with white text (this meets WCAG contrast requirements).

Make the indicator bar more prominent to make it clearer which item is selected now that the background is done less to indicate this.

Screenshots

Dark mode

Page selected

Before

After

Page hovered

Before

After

Light

Page selected

Before

After

Page hovered

Before

After

Mobile

Before

After

Increase the background contrast for the selected page.
Increase the prominance of the indictor bar so we dont need to rely on
the background to have a strong contrast difference.
@NickColley NickColley added the accessibility Accessibility (a11y) label Nov 28, 2022
@NickColley NickColley marked this pull request as ready for review November 28, 2022 23:36
@nolanlawson
Copy link
Owner

Look great to me! Thanks a bunch. 🙂

@nolanlawson nolanlawson merged commit da28e98 into master Dec 2, 2022
@NickColley NickColley deleted the improve-navigation-contrast branch December 2, 2022 23:36
alice-werefox pushed a commit to alice-werefox/sema-werefox-cafe that referenced this pull request Apr 3, 2023
Increase the background contrast for the selected page.
Increase the prominance of the indictor bar so we dont need to rely on
the background to have a strong contrast difference.
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