Skip to content

fix: use z-9999 on tooltip#6617

Closed
Udit-takkar wants to merge 2 commits intomainfrom
fix/tooltip-z-index
Closed

fix: use z-9999 on tooltip#6617
Udit-takkar wants to merge 2 commits intomainfrom
fix/tooltip-z-index

Conversation

@Udit-takkar
Copy link
Copy Markdown
Contributor

Fixes #6467

Alternate approach:- #6491

Environment: Staging(main branch) / Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in>
@vercel
Copy link
Copy Markdown

vercel bot commented Jan 22, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
cal ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 22, 2023 at 1:42PM (UTC)
1 Ignored Deployment
Name Status Preview Comments Updated
cal-com-storybook ⬜️ Ignored (Inspect) Jan 22, 2023 at 1:42PM (UTC)

@PeerRich
Copy link
Copy Markdown
Member

you tested this, right? if yes we can merge this

@Udit-takkar
Copy link
Copy Markdown
Contributor Author

Screenshot 2023-01-22 at 6 26 24 PM

yes

@mocherfaoui
Copy link
Copy Markdown
Contributor

this is better than my complicated approach 😅
note that react-hot-toast also uses z-index: 9999. But I don't think it's going to create a problem with tooltips
@Udit-takkar can you include this small fix #6491 (comment) in your PR too if it's possible?

fix
Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in>
@Udit-takkar
Copy link
Copy Markdown
Contributor Author

closing in favour of #6491

@keithwillcode keithwillcode deleted the fix/tooltip-z-index branch September 17, 2025 11: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.

[CAL-780] Tooltip is behind Dialog

3 participants