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

Fixed sufficient color contrast #1035

Merged
merged 1 commit into from
Oct 31, 2024
Merged

Conversation

Priyankagarai
Copy link

Closes #1034

Element is having low color contrast issue, I have made necessary modifications with font style to meet sufficient color contrast.

@birm birm changed the base branch from master to develop October 11, 2024 17:26
Copy link
Member

@birm birm left a comment

Choose a reason for hiding this comment

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

Hi and thank you for opening this PR!

Can you open a report or something stating that this fixes color contrast? It doesn't look to me like you're changing color at all.

A list of tools can be found here: https://www.w3.org/WAI/test-evaluate/tools/list/

@Priyankagarai
Copy link
Author

Hi, thank you for your feedback!

I’ve tested the page using Lighthouse in Chrome DevTools. Initially, Lighthouse flagged poor color contrast (as shown before the fix), but after correcting the colors, the issue no longer appears, meaning the contrast now meets the accessibility standards.
Since Lighthouse doesn’t display warnings once the issue is fixed, there’s no message in the updated report.

Let me know if further clarification is needed!

@birm birm merged commit 142b818 into camicroscope:develop Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Accessibility] Export Page: Element has low color contrast
2 participants