Skip to content

css: <li> color contrast doesn't meet WCAG 2.0 standards (#808080 is too light) #569

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

Closed
weedySeaDragon opened this issue Sep 11, 2016 · 4 comments
Labels

Comments

@weedySeaDragon
Copy link

color #808080 is too light for

  • elements in the body text. It doesn't meet the WCAG 2.0 AA standards for enough contrast. (I think using it in other places -- like the numbers for code is OK since those are ancillary to the information being presented.) (I find the WebAIM tools for quickly checking color contrast and for scanning a page for issues to be really easy to use to check basic accessibility items.

    I didn't want to do a pull request because I didn't want to do something that might unbalance the design as a whole. But it seems on the surface as though just darkening it a bit would be fine.
    Line 92 in bootstrap.css could be changed to use #606060, which would pass WCAG 2.0 AA at that size.

  • @SethTisue
    Copy link
    Member

    corresponding issue on scala-lang.org repo: scala/scala-lang#509

    @wenyangfu
    Copy link

    Since scala/scala-lang#509 has been closed, can this also be closed?

    @weedySeaDragon
    Copy link
    Author

    The <li> elements in the body text do have enough contrast now. (Yay!)

    But there are still issues with the contrast. Specifically, there's still not enough contrast with the <a> elements. It's easy to check with the WAVE tool -- just put in the URL of any page and it checks and shows you issues. http://wave.webaim.org/

    The red flags show where there are contrast issues on the Getting Started page:

    screen shot 2017-11-30 at 10 56 22 am


    @SethTisue Should we close this issue, and then I'll create a new issue?
    The new issue(s) can be either:
    (1) One issue titled "Meet minimum WCAG 2.0 usability standards," which will list all of the issues found? (not only the contrast issue, but also the missing text for images, etc.)
    or
    (2) separate issues for each main area that needs to be addressed? (one for the contrast, another issue for missing text for images, etc.) ?

    I'm open to suggestions as to whatever is the best path.

    @SethTisue
    Copy link
    Member

    @weedySeaDragon yes, please make new issue(s), and include the information about WAVE. I don't think we want, say, 10 new tickets, other than that I don't really have an opinion about one-vs-multiple

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    Projects
    None yet
    Development

    No branches or pull requests

    3 participants