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

Click on toolbar area between buttons hides the toolbar #12172

Closed
mmotyczynska opened this issue Jul 28, 2022 · 2 comments · Fixed by #14000
Closed

Click on toolbar area between buttons hides the toolbar #12172

mmotyczynska opened this issue Jul 28, 2022 · 2 comments · Fixed by #14000
Labels
domain:accessibility This issue reports an accessibility problem. domain:ui/ux This issue reports a problem related to UI or UX. package:ui type:bug This issue reports a buggy (incorrect) behavior.

Comments

@mmotyczynska
Copy link
Contributor

📝 Provide detailed reproduction steps (if any)

  1. In ClassicEditor scroll down so the toolbar is not visible (in balloon toolbar it's not necessary).
  2. Make a selection in the editor so the toolbar appears.
  3. Click the toolbar somewhere between buttons.

This should be useful in case someone misses the button and accidentally clicks the toolbar instead of a button.

✔️ Expected result

The toolbar is still visible.

❌ Actual result

The toolbar hides.

Screen.Recording.2022-07-28.at.10.53.40.mov

❓ Possible solution

When someone clicks the toolbar instead of a button we could move the focus to the first item in the toolbar.

📃 Other details

  • Browser: …
  • OS: …
  • First affected CKEditor version: …
  • Installed CKEditor plugins: …

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

@mmotyczynska mmotyczynska added the type:bug This issue reports a buggy (incorrect) behavior. label Jul 28, 2022
@mlewand mlewand added package:ui domain:ui/ux This issue reports a problem related to UI or UX. labels Jul 28, 2022
@Reinmar Reinmar added the domain:accessibility This issue reports an accessibility problem. label Jul 29, 2022
@tony1377
Copy link

Any idea of an ETA on this?

@mlewand
Copy link
Contributor

mlewand commented Aug 29, 2022

@tony1377 not yet. Please add 👍 reaction to the main post to show your support for it.

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

Successfully merging a pull request may close this issue.

6 participants