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

Tooltip's race condition #16949

Closed
Reinmar opened this issue Aug 20, 2024 · 1 comment · Fixed by #17177
Closed

Tooltip's race condition #16949

Reinmar opened this issue Aug 20, 2024 · 1 comment · Fixed by #17177
Labels
domain:ui/ux This issue reports a problem related to UI or UX. package:ui type:bug This issue reports a buggy (incorrect) behavior. type:regression This issue reports a bug that was not present in the previous releases.
Milestone

Comments

@Reinmar
Copy link
Member

Reinmar commented Aug 20, 2024

📝 Provide detailed reproduction steps (if any)

  1. Move your mouse over the heading dropdown
  2. Wait ~100-300ms
  3. Move your mouse away
Screen.Recording.2024-08-20.at.16.57.57.mov

I can't say how long your mouse need to be over the button. But it seems that it's just a tad less than it takes the tooltip to appear. However, I stumbled upon this at least twice doing other things and can reproduce quite reliably.

There seems to be some race condition.

✔️ Expected result

The tooltip doesn't show up / hides.

❌ Actual result

The tooltip stays visible until the next mousein&out.

📃 Other details

  • Browser: …
  • OS: …
  • First affected CKEditor version: First version I can reproduce this on is 41.3.0. Seems to work fine on 41.2.0.
  • Installed CKEditor plugins: …

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@Reinmar Reinmar added type:bug This issue reports a buggy (incorrect) behavior. type:regression This issue reports a bug that was not present in the previous releases. package:ui domain:ui/ux This issue reports a problem related to UI or UX. labels Aug 20, 2024
@oleq
Copy link
Member

oleq commented Aug 21, 2024

Confirmed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:ui/ux This issue reports a problem related to UI or UX. package:ui type:bug This issue reports a buggy (incorrect) behavior. type:regression This issue reports a bug that was not present in the previous releases.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants