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

Set z-index of hovers to 1000 #15172

Merged
merged 1 commit into from
Mar 18, 2025
Merged

Set z-index of hovers to 1000 #15172

merged 1 commit into from
Mar 18, 2025

Conversation

sgraband
Copy link
Contributor

@sgraband sgraband commented Mar 12, 2025

What it does

Value taken from #15154.

Part of #15118.

How to test

  1. Build this branch
  2. Open Theia
  3. Set the window.tabbar.enhancedPreview to visual or enhanced
  4. Check that this does not happen:
    image

Follow-ups

Breaking changes

  • This PR introduces breaking changes and requires careful review. If yes, the breaking changes section in the changelog has been updated.

Attribution

Review checklist

Reminder for reviewers

Value taken from #15154.

Part of #15118.

Signed-off-by: Simon Graband <[email protected]>
@JonasHelming
Copy link
Contributor

@cdamus Do you want to review this?

@cdamus
Copy link
Contributor

cdamus commented Mar 14, 2025

@cdamus Do you want to review this?

Happy to.

Copy link
Contributor

@cdamus cdamus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! The problems I had did not recur. I assume that this is unrelated as it appears to be a transparency glitch. I mean, who would put the icons of toolbar buttons at a higher z-index than 1000, amirite?

CleanShot 2025-03-14 at 08 40 21

@sgraband
Copy link
Contributor Author

Is the behavior from the screenshot on this branch? If yes, which view is this?

@cdamus
Copy link
Contributor

cdamus commented Mar 17, 2025

Is the behavior from the screenshot on this branch? If yes, which view is this?

Yes, I captured it in testing this PR. The toolbar icons are from the File Explorer.

@sgraband
Copy link
Contributor Author

Thanks. I can reproduce this. I will take a look and fix this with this PR as well, while i am at it.

@sgraband
Copy link
Contributor Author

It's a bit more complicated than originally thought and the question would be where to stop, as we have z-index issues all over the place. Therefore i will go ahead and merge this. I have created #15214 to hopefully fix the z-index issues and make the values a bit more maintainable.

@sgraband
Copy link
Contributor Author

@JonasHelming can you approve?

@sgraband sgraband merged commit c4b21c3 into master Mar 18, 2025
10 of 11 checks passed
@github-actions github-actions bot added this to the 1.60.0 milestone Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants