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

Color visibility of some note #2093

Closed
edcheyjr opened this issue Dec 3, 2023 · 5 comments · Fixed by #2094
Closed

Color visibility of some note #2093

edcheyjr opened this issue Dec 3, 2023 · 5 comments · Fixed by #2094

Comments

@edcheyjr
Copy link

edcheyjr commented Dec 3, 2023

What docs page needs to be fixed?

Not a big problem but the color of the links under "Defining the mapDispatchToProps Function with bindActionCreators" does not seem visible will reading

What is the problem?

The color of the links note under Defining the mapDispatchToProps Function with bindActionCreators

What should be changed to fix the problem?

the purple and grey color don't seem to make it visible while reading i would recommend changing the colors a bit

@EskiMojo14
Copy link
Collaborator

the contrast ratio between the background and foreground is 8.61, which passes both AA and AAA WCAG standards:
image
image

if we change it from primary-darkest back to the default of primary, it drops down to 5.43 which only passes AA:
image
image

i do agree that the links stand out less with the current primary-darkest colour though.

@edcheyjr
Copy link
Author

edcheyjr commented Dec 3, 2023

Okay, thanks for the quick response. my concern is in dark mode. I don't know if in dark mode this is how the note should look like?

Screenshot from 2023-12-03 20-14-20

@EskiMojo14
Copy link
Collaborator

ah, you're right - that definitely doesn't look great

@EskiMojo14
Copy link
Collaborator

raised #2094 which fixes the background colour for codeblocks inside blockquotes

image

it's a little odd that blockquotes are light even during dark theme but this is at least consistent with that

@edcheyjr
Copy link
Author

edcheyjr commented Dec 3, 2023

yeah.... i thought so.... but that's better

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 a pull request may close this issue.

2 participants