-
Notifications
You must be signed in to change notification settings - Fork 85
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: display tooltips over notifications #6074
Conversation
Kudos, SonarCloud Quality Gate passed!
|
Hi @sissbruecker , this commit cannot be picked to 24.1 by this bot, can you take a look and pick it manually? |
* fix: display tooltips over notifications * set manual mode via attribute (cherry picked from commit 171b663)
This ticket/PR has been released with Vaadin 24.2.0.alpha2 and is also targeting the upcoming stable 24.2.0 version. |
Description
Currently tooltips are always shown below notifications:
200
via the defaultvaadin-overlay
styles:web-components/packages/overlay/src/vaadin-overlay.js
Line 85 in 90992ed
1000
:web-components/packages/notification/src/vaadin-notification.js
Line 30 in 90992ed
This increases the z-index for tooltips to be larger than the one for notifications.
Fixes #5748
Type of change