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 KBDane Powell
screen1.PNG37.56 KBDane Powell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dane Powell’s picture

Dane Powell CreditAttribution: Dane Powell 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 CreditAttribution: hefox commented
Assigned: Unassigned » hefox
hefox’s picture

hefox CreditAttribution: 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 CreditAttribution: hefox commented
FileSize
features_981274_color.patch744 bytes

I guess I could upload the patch.

hefox’s picture

hefox CreditAttribution: hefox commented
Assigned: hefox » Unassigned
hefox’s picture

hefox CreditAttribution: hefox commented
Status: Needs review » Needs work

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

mpotter’s picture

mpotter CreditAttribution: 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...