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

style(tooltip): replace selection unicode check with svg #2091

Merged
merged 2 commits into from
Jul 6, 2023

Conversation

nickofthyme
Copy link
Collaborator

@nickofthyme nickofthyme commented Jul 5, 2023

Summary

Replaces selection unicode check with svg from eui.

Screen Recording 2023-07-05 at 10 04 07 AM

Details

The previous implementation of this selection UI was using a ::before pseudo element with the (U+2713) unicode character. This lead to inconsistent check styles when used across different environments.

Issues

closes #2090

Checklist

  • The proper chart type label has been added (e.g. :xy, :partition)
  • The proper feature labels have been added (e.g. :interactions, :axis)
  • The code has been checked for cross-browser compatibility (Chrome, Firefox, Safari, Edge)
  • Visual changes have been tested with all available themes including dark, light, eui-dark & eui-light

@nickofthyme nickofthyme added :styling Styling related issue :tooltip Related to hover tooltip :all Applies to all chart types labels Jul 5, 2023
@nickofthyme
Copy link
Collaborator Author

buildkite update screenshots

@nickofthyme nickofthyme merged commit 3142521 into elastic:main Jul 6, 2023
11 checks passed
@nickofthyme nickofthyme deleted the svg-check branch July 6, 2023 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:all Applies to all chart types :styling Styling related issue :tooltip Related to hover tooltip
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[tooltip] replace unicode with svg icon
2 participants