Features is practically unusable with dark themes (which have light text), because it defines background-color: #FFFFFF<code> for many elements but does not define <code>color.

Ideally, the background-color should be based on the theme colors somehow. Otherwise, Features should at least define text color, so that it's at least usable (if ugly).

CommentFileSizeAuthor
#4 features_981274_color.patch744 byteshefox
screen2.PNG20.63 KBdanepowell
screen1.PNG37.56 KBdanepowell

Comments

danepowell’s picture

danepowell commented

Bah, sorry I failed to preview that post and catch the code typo, but I think you get the idea.

hefox’s picture

hefox commented
Assigned: Unassigned » hefox
hefox’s picture

hefox commented
Version: 6.x-1.0 » 6.x-1.x-dev
Status: Active » Needs review

Haven't tested, but think I got the areas.

The .css file has coding standard issues so patch may have to be updated if that is fixed.

hefox’s picture

hefox commented
StatusFileSize
new features_981274_color.patch744 bytes

I guess I could upload the patch.

hefox’s picture

hefox commented
Assigned: hefox » Unassigned
hefox’s picture

hefox commented
Status: Needs review » Needs work

Need to update it since css changed, doesn't apply.

mpotter’s picture

mpotter commented
Status: Needs work » Patch (to be ported)

Committed to cb888b3.

  • mpotter committed cb888b3 on 8.x-3.x 
    Issue #981274 by hefox | Dane Powell: Fixed (Accessibility) Background...