Closed (outdated)
Project:
Drupal core
Version:
8.0.x-dev
Component:
Seven theme
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
29 Jul 2013 at 19:22 UTC
Updated:
21 Jun 2017 at 02:17 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
echoz commentedtags with comma
Comment #2
mgiffordSo jumping from #0074bd to #00619d would fix this based on http://webaim.org/resources/contrastchecker/
Is that too much of a jump? Going to #0073bd doesn't seem to be enough against this background.
Comment #3
echoz commentedOne detail of #2022695: Content header style update is it omits the breadcrumbs, which would make this bug obsolete if no other links show up here.
In general, perhaps this should go back to checking anywhere links appear in Seven against non white backgrounds, because any link color change made should be checked throughout Seven.
We can also lighten backgrounds in combination with darkening link color.
Comment #4
Bojhan commentedI am not sure if we should be placing the breadcrumbs here too.
So I am really worried about the disconnect from the UX activities and the a11y team, in Drupal 7 we had much more collaboration. If an issue is noticed with contrast, that requires the drastic step of changing our base Drupal Blue color we should be contacted and we should look at whether the style guide effort relieve us of this problem.
Comment #5
dead_armWhen I created this follow up issue I added it to the outstanding issues list of #1986434: New visual style for Seven so that it would be on the Seven revamp team's radar.
I don't think people have posted patches to adjust the contrast here because they recognize that there are still changes being made to Seven's header and breadcrumbs, so this could become a WCAG AA non-failure once things firm up.
Comment #6
lewisnymanTagging with styleguide. Thanks for this. We're have a few options when handling breadcrumbs.
Comment #7
mgiffordPatch moves all links to #00619d in Seven.
Comment #8
mgifford7: drupal8.seven-theme.2053145-7.patch queued for re-testing.
Comment #9
Bojhan commentedWon't fixing this, the new tabs and breadcrumb placement already fixes this.
Comment #10
charles belov