Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use $callout-font-color-alt in _callout.scss #9415

Merged
merged 1 commit into from
Dec 1, 2016
Merged

Use $callout-font-color-alt in _callout.scss #9415

merged 1 commit into from
Dec 1, 2016

Conversation

IllusionPerdu
Copy link
Contributor

Resolve problem with font color in dark background and use $callout-font-color-alt ;)

Resolve problem with font color in dark background and use  $callout-font-color-alt ;)
@kball
Copy link
Contributor

kball commented Nov 30, 2016

Seems reasonable to me - @andycochran what do you think?

@andycochran
Copy link
Contributor

Nice, @IllusionPerdu! This is a good use of the new color-pick-contrast() function.

@andycochran
Copy link
Contributor

I think this PR is okay to merge. But it makes me think of a couple things.

  • Default-colored links do not meet AA contrast requirements in Callouts with non-white backgrounds. The primary blue is just barely dark enough to meet AA requirements on white.

  • Links in Callouts should get an accessible color. @wolfgang-demeter pointed this out in Fixed .callout text- and link-colors #9098, for which @ncoden had some great insight.

@kball
Copy link
Contributor

kball commented Dec 1, 2016

Ok gonna go ahead and merge this. Do we think #9098 captures enough info to address these 2 issues?

@kball kball merged commit 1bb32f4 into foundation:develop Dec 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants