Color labels are not intuitive when searching for the keyword “background”, leading users to guess and check.

4 participants were confused about which of the background colors were changing. They were expecting the banner color to change, but the site background changed.

Suggested recommendation included:
Add the word “background” to background colors.

CommentFileSizeAuthor
#4 drupal-1164828-4.patch650 bytestim.plunkett

Comments

Jeff Burnz’s picture

Component: color.module » Bartik theme

Moving to right queue, right now this should only apply to Bartik since we're going to remove Garland in D8. Themes provide there own labels.

From the report I assume what is meant is to add the keyword "background" to the header top and bottom options, e.g.:

    'top' => t('Header background top'),
    'bottom' => t('Header background bottom'),
droplet’s picture

can it also reorder & grouping

- background color
- text & link color

tim.plunkett’s picture

tim.plunkett’s picture

Status: Active » Needs review
StatusFileSize
new650 bytes

Status: Needs review » Needs work
Issue tags: -Usability, -UMN 2011

The last submitted patch, drupal-1164828-4.patch, failed testing.

tim.plunkett’s picture

Status: Needs work » Needs review
Issue tags: +Usability, +UMN 2011

#4: drupal-1164828-4.patch queued for re-testing.

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

Back to RTBC

Jeff Burnz’s picture

Status: Reviewed & tested by the community » Needs review

Fine by me, good to go.

Jeff Burnz’s picture

Status: Needs review » Reviewed & tested by the community

Hmmm, back to RTBC.

dries’s picture

Version: 8.x-dev » 7.x-dev

Committed this to 8.x as it is a pretty small patch. Should probably go into 7.x too.

webchick’s picture

Version: 7.x-dev » 8.x-dev
Status: Reviewed & tested by the community » Fixed
Issue tags: +String freeze

Hm. This would break translations. I don't think we can commit it to D7. :(

However, I've opened up a discussion about this at http://groups.drupal.org/node/154304 and will see what comes out of it.

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