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

fix(MultipleTooltip): changed tooltips and it's backdrop styles #179

Merged
merged 2 commits into from
Feb 1, 2024

Conversation

shvorak
Copy link
Contributor

@shvorak shvorak commented Jan 31, 2024

Hi

In this PR, I propose to change the tooltip styles

  1. The background color of the tooltip itself has been changed with the help of the product design team.
  2. Backdrop style is suggested to correct the wrong appearance in different themes.

As a basic idea, I took the background color variable and made a gradient.

Before (AS IS) After (TO BE)
image image
image image

@shvorak shvorak self-assigned this Jan 31, 2024
@shvorak shvorak force-pushed the multiple-tooltip-color-change branch from 213e996 to b988141 Compare January 31, 2024 12:36
@gravity-ui-bot
Copy link
Contributor

Preview is ready.

@shvorak
Copy link
Contributor Author

shvorak commented Jan 31, 2024

UPD: I just received approval from the product design team for the 2nd point.

@shvorak shvorak requested a review from Lunory February 1, 2024 12:14
@shvorak shvorak merged commit 0a12da5 into main Feb 1, 2024
3 checks passed
@shvorak shvorak deleted the multiple-tooltip-color-change branch February 1, 2024 15:25
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.

4 participants