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

No focusable inserter in zoom out #66346

Open
3 of 6 tasks
jeryj opened this issue Oct 22, 2024 · 1 comment
Open
3 of 6 tasks

No focusable inserter in zoom out #66346

jeryj opened this issue Oct 22, 2024 · 1 comment
Labels
[Type] Bug An existing feature does not function as intended

Comments

@jeryj
Copy link
Contributor

jeryj commented Oct 22, 2024

Description

In zoom out, an inserter appear on the selected block at the end of the block boundary. It only gets rendered if this block boundary is visible within the viewport. When using a keyboard, this means there is no focusable inserter if the block boundary is not within the viewport, which is fairly common in zoom out mode.

Step-by-step reproduction instructions

  • Insert a tall pattern that takes up all of the viewport in zoom out
  • Enter zoom out
  • Select the block
  • Make sure the bottom of the block boundary is not in the viewport
  • Press Tab
  • Focus is not moved to the inserter

Screenshots, screen recording, code snippet

No response

Environment info

No response

Please confirm that you have searched existing issues in the repo.

  • Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

  • Yes

Please confirm which theme type you used for testing.

  • Block
  • Classic
  • Hybrid (e.g. classic with theme.json)
  • Not sure
@jeryj jeryj added the [Type] Bug An existing feature does not function as intended label Oct 22, 2024
@jeryj
Copy link
Contributor Author

jeryj commented Oct 22, 2024

A way to mitigate but not fix the core issue would be to add an inserter back to the top of the block boundary as well. This makes it more likely that there will be an inserter, but doesn't fix the core issue. The inserter should be able to be rendered without being in the viewport.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

1 participant