Skip to content

fix(playground): fix diagnostics tooltip being behind buttons #1367

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

WarningImHack3r
Copy link

Spotted that on Paolo's TWIS today: diagnostics that are high enough and long enough are obstructed by the top buttons, which have a z-index.

This PR addresses that issue: it removes the z-index: 999 on the buttons and adds a z-index: 2 to both parts of the buttons’ hover tooltips — likely the reason the original z-index was used for.

Before After
Before After
Preserved tooltips The button tooltips are preserved

I'm not sure if there are other reasons for this high z-index value beyond the tooltips; if so, please let me know and I'll add them to the fix as well.


A note on documentation PRs

If this is a documentation PR (i.e. changing content within apps/svelte.dev/content/docs), then this is the wrong repository to make those changes. The content in this folder is synced from other repositories. Therefore, these changes should be made in their respective repositories (at https://github.com/sveltejs/svelte or https://github.com/sveltejs/kit, or example).

Before submitting the PR, please make sure you do the following

  • It's really useful if your PR references an issue where it is discussed ahead of time.
  • Prefix your PR title with feat:, fix:, chore:, or docs:.
  • This message body should clearly illustrate what problems it solves.

Copy link

vercel bot commented May 30, 2025

@WarningImHack3r is attempting to deploy a commit to the Svelte Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant