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

Some circumstances lead to tooltip invasion with the web viewer #6512

Closed
abey79 opened this issue Jun 6, 2024 · 4 comments
Closed

Some circumstances lead to tooltip invasion with the web viewer #6512

abey79 opened this issue Jun 6, 2024 · 4 comments
Assignees
Labels
🪳 bug Something isn't working egui Requires egui/eframe work ui concerns graphical user interface 🕸️ web regarding running the viewer in a browser
Milestone

Comments

@abey79
Copy link
Member

abey79 commented Jun 6, 2024

Repro:

tooltip_overload.mp4

This doesn't seem to happen with the native built.

@abey79 abey79 added 🪳 bug Something isn't working ui concerns graphical user interface labels Jun 6, 2024
@abey79 abey79 added this to the 0.17 milestone Jun 6, 2024
@emilk
Copy link
Member

emilk commented Jun 10, 2024

If a tooltip contains something that has a tooltip, then we can get into a situation where we get a tooltip for a tooltip for a tooltip… This is not a problem in of itself, but we have some helpful "Open next tooltip quickly" logic that is meant for the situation where you have multiple widgets next to each other and want to quickly inspect their tooltips. That logic means the tooltip of the widget in a tooltip opens right away, causing the problem in the video.

One solution is to never apply that "if a tooltip is already open" logic for widgets in a tooltip layer (Order::Tooltip). Another is to only apply it if the open tooltip is in the same egui::LayerId as the previous tooltip.

I don't understand why it only happens on web though 🤔

@emilk emilk self-assigned this Jun 10, 2024
@emilk emilk added 🕸️ web regarding running the viewer in a browser egui Requires egui/eframe work labels Jun 10, 2024
@emilk
Copy link
Member

emilk commented Jun 10, 2024

I fail to reproduce

@emilk
Copy link
Member

emilk commented Jun 10, 2024

I wonder if this is a result of

If so, would be solved by:

@emilk
Copy link
Member

emilk commented Jun 10, 2024

@emilk emilk closed this as not planned Won't fix, can't repro, duplicate, stale Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working egui Requires egui/eframe work ui concerns graphical user interface 🕸️ web regarding running the viewer in a browser
Projects
None yet
Development

No branches or pull requests

2 participants