There are instances where the theme text (.form-item label and ) does not meet the minimum contrast ratio of 4.5 as given by WCAG 2.0.

Some examples

Ratio: 2.46

.collapsed .fieldset-title a {
  background:#eee;
  color:#999;
  }

Ratio: 2.85

.form-item label {
  margin:0px;
  padding:0px;
  color:#999;
  font-weight:bold;
  font-size:10px;
  text-transform:uppercase;
  }

Comments

fearlsgroove’s picture

Title: Font contrast fails WCAG AA criterion » Increase font and form element color contrast
Version: 6.x-3.0-beta2 » 7.x-4.x-dev
Priority: Minor » Normal
Status: Active » Needs review
StatusFileSize
new1.37 KB
new141.6 KB
new139.99 KB
new66.59 KB
new66.03 KB

Love me some rubik, but I've also found the font contrast to be insufficient. The attached patch darkens links, labels, unselected vertical tab items and form element borders somewhat. I've found it much more readible over time. Obviously this is at least somewhat subjective and easy to do with a subtheme, but I think the theme would benefit from the change overall.

To the OP's point: I don't know if these new colors pass the guideline ratio, but if they don't I think it makes sense that they should before being committed.

c4rl’s picture

@fearlsgroove: I usually use http://snook.ca/technical/colour_contrast/colour.html to check contrast.

adnanc’s picture

Status: Needs review » Reviewed & tested by the community

Looks good. While the changes are minor, they go a long way in making things easier on the eyes.

The only thing that I would suggest is to re-name the patch to follow the "unofficial patch naming convention" as seen here

Also agree on the point where the theme itself could use definite improvements in terms of contrast. I (and I can speak for the rest of my team) love Rubik, and I'm hoping that patches like these make their way into Rubik.

coffeymachine’s picture

Status: Reviewed & tested by the community » Fixed

I like this patch. Reviewed and committed to 7.x-dev. Thanks!

ericduran’s picture

Status: Fixed » Needs review
StatusFileSize
new292 bytes

I've actually been doing #ddd instead of #ccc for the border of the input elements.

I think #ddd is to better especially when compare with the outer border.

I'm actually re-opening this issue with a patch for #ddd hope this is ok. If not feel free to just close it :)

Patch attached.

haydeniv’s picture

Issue summary: View changes
Status: Needs review » Fixed

Committed b72b719

Thanks!

Status: Fixed » Closed (fixed)

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