On the style-guide page you have a css rule (line 19 of styleguide.css) that set's a background color to the top navigation block but you don't set a text color as well.
In my case - light text on dark background theme - this looks like a major accessibility issue.
Two possible solutions will be not setting the background color or setting the text color as well.
This screenshot demonstrate my situation :
http://rookery9.aviary.com.s3.amazonaws.com/7842500/7842523_33ad.png

Thanks anyway for this very useful module.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dcmouyard’s picture

I would recommend removing color from the css altogether.

If you do decide to use color, then you should specify background colors, text colors, and link colors.

agentrickard’s picture

Patch?

dcmouyard’s picture

I'd be happy to provide a patch once I know what direction to take: remove color declarations or make them accessible?

agentrickard’s picture

Let's go for accessible. I think we need some base styles for the Style Guide labels and links.

dcmouyard’s picture

Status: Active » Needs review
FileSize
1.99 KB

Updated the CSS with more accessible colors and tweaked some theme functions.

agentrickard’s picture

Status: Needs review » Patch (to be ported)

Nice. Tested against Bartik, Austin, and Abstract themes.

Committed to 7.x. Needs backport to 6, but should apply cleanly.

agentrickard’s picture

Status: Patch (to be ported) » Fixed
FileSize
1.97 KB

And committed to 6.x. Nice work!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.