fix: use CSS variable for tooltip z-index instead of hardcoded value#117
fix: use CSS variable for tooltip z-index instead of hardcoded value#117rohitrath0d wants to merge 2 commits intoknadh:masterfrom
Conversation
|
|
…dcoded value The tooltip pseudo-elements used a hardcoded z-index: 1000, inconsistent with the theme's CSS variable pattern for layering. Instead of adding a new variable, this uses calc(var(--z-modal) + 10) so the tooltip z-index is always relative to the modal layer.
rohitrath0d
force-pushed
the
fix/tooltip-zindex-variable
branch
from
0c7c4d3 to
616103c
Compare
|
hi @knadh , i have made the changes for calc(--z-modal + 10) in |
rohitrath0d
force-pushed
the
fix/tooltip-zindex-variable
branch
from
616103c to
094ec48
Compare
|
hi @knadh , sorry the pr got little messy from my end, the last commit |
src/css/01-theme.css
Outdated
| --z-modal: 200; | ||
| } | ||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
Please retain the line break at the end of the file in both files.
There was a problem hiding this comment.
Hi, I’ve added the missing newline at the end of both 01-theme.css and tooltip.css and pushed the update.
Could you please review and confirm if everything looks good now?
2 participants
The tooltip pseudo-elements used a hardcoded z-index: 1000 while the theme defines CSS variables for layering (--z-dropdown: 50, --z-modal: 200). This adds --z-tooltip to the theme and uses var(--z-tooltip) in tooltip.css, making the tooltip layer configurable and consistent with the design system.
Fixes #116