Add a11y friendly default text-colors when selecting a background-color.

[allancole]

Before:

After:

Related Tickets

Fixes: #964

[joyously]

Don't the color settings change in the palette when you change the colors in the Customizer?
So setting the text color is only good for the defaults?
Wouldn't it be better to make the text color work regardless of background color?

[allancole]

Group Blocks and any block that uses any of the .has-background-xxxxxxx-color utility classes are also fixed with this PR. Here’s a gif of it working with this fix:

[mapk]

I just tested this in FF 70.0.1 and am seeing the a11y warning still. Is this right?

[mapk]

So double checking the contrast on these color combinations:

• RED (#cd2653) & WHITE (#FFF) – 5.25 contrast ratio
• BLACK (#000) & TAN (#f5efe0) – 18.30 contrast ratio
• DARK GRAY (#6d6d6d) & WHITE (#FFF) – 5.17 contrast ratio

Looks like something may be off with the contrast warning in Gutenberg?

[joyously]

I tried it with changed colors.

[audrasjb]

Hi, I just spotted a very minor coding standard issue.
I commented in the PR.
Cheers,
Jb

[audrasjb]

Coding standards: should be #fff for consistency.

[audrasjb]

Hi,
For what it worth, I tested the PR on my side, and it looks good to me.
Changing background color is automatically changing the font color when needed.

[audrasjb]