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

Fix: Insufficient color contrast of some elements #370

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

Conversation

varun-s22
Copy link
Contributor

  • Switched to a heavier color code (given in the config), which increases the color contrast between foreground color and background color.

  • Removed unnecessary imports from index.jsx page.

Fixes: #362

* Switched to a heavier color code (given in the config), which
  increases the color contrast between foreground color and background
color.

* Removed unnecessary imports from `index.jsx` page.
@vercel
Copy link

vercel bot commented Jul 7, 2023

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

Name Status Preview Comments Updated (UTC)
joinmastodon ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 19, 2023 10:38am

@andypiper andypiper added the a11y-and-ux Accessibility or UX issues and PRs label Jul 13, 2023
@andypiper
Copy link
Member

I'm holding of on merging this for now, as we have official brand colours and need to think about how they apply in this case. Leaving on hold for the moment, but thank you for this contribution, we may come back to it.

@varun-s22
Copy link
Contributor Author

whats the status now @andypiper?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-and-ux Accessibility or UX issues and PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Accessibility: Insufficient color contrast of some elements
2 participants