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

[8.x] [Vega] Improve contrast ratio for vega visualizations tooltip key text (#200208) #202192

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

elastic#200208)

## Summary

Closes elastic#151923

We need to improve the contrast ratio in vega kibana tooltip key text.
The current contrast ratio between the background (#404040) and the text
color (#98A2B3) is below 4.5:1 and fails WCAG SC 1.4.3 Contrast
(Minimum).

In order to solve this issue, we have updated the color used for the
**key value** the to use `colors.lightShade` (#D3DAE6). With this
change, we have increased the contrast ratio from 4.02 to 7.37.

You can see the available shades
[here](https://eui.elastic.co/#/theming/colors/values#shades).

### Screenshots

#### Before - contrast ratio 4.02

![Screenshot 2024-11-14 at 16 46
13](https://github.com/user-attachments/assets/19c22538-6540-4e6a-909d-8abd58facbbc)

#### After  - contrast ratio 7.37

![Screenshot 2024-11-14 at 16 44
43](https://github.com/user-attachments/assets/a3ca319f-0cfa-4e79-a4ae-fddd64fc2e42)

### Checklist

Check the PR satisfies following conditions.

- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_node:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 2b1f12b)
@kibanamachine kibanamachine merged commit 754d4ee into elastic:8.x Nov 28, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants