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 search input a11y for generated docs #11604

Merged
merged 2 commits into from
Jan 1, 2022

Conversation

chances
Copy link
Contributor

@chances chances commented Dec 17, 2021

Improve color contrast of search input placeholder in docs according to the W3C's WCAG specification color contrast recommendations.

Extant color contrast is a ratio of 1.7:1, below the minimum "WCAG A" color contrast recommendation. Notice the "Search..." placeholder is barely visible in this screenshot:
Screenshot from 2021-12-15 18-21-12

Improve color contrast of search input placeholder in docs according to the W3C's [WCAG specification](https://www.w3.org/TR/WCAG21/) color contrast recommendations.
@straight-shoota
Copy link
Member

I'm not sure about making it a flat black. Maybe some shade in between might be nicer to indicate it's a placeholder, not a value.

`#757575` is about the lightest color that passes the WCAG AA color contrast recommendation on a white background with text of a normal font weight.
@chances
Copy link
Contributor Author

chances commented Dec 17, 2021

#757575 is about the lightest color that passes the WCAG AA color contrast recommendation on a white background with text of a normal font weight. See WebAIM's color contrast checker.

@straight-shoota straight-shoota added kind:bug A bug in the code. Does not apply to documentation, specs, etc. and removed kind:feature labels Dec 20, 2021
@oprypin oprypin added this to the 1.3.0 milestone Dec 30, 2021
@straight-shoota straight-shoota merged commit 103b41c into crystal-lang:master Jan 1, 2022
@chances chances deleted the docs-a11y-search-input branch January 3, 2022 00:46
rdp pushed a commit to rdp/crystal that referenced this pull request Jan 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:tools:docs-generator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants