-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Select Mode: Prevent the inbetween inserter from triggering within sections #65529
Select Mode: Prevent the inbetween inserter from triggering within sections #65529
Conversation
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Size Change: +12 B (0%) Total Size: 1.77 MB
ℹ️ View Unchanged
|
This fixes the issues I've been seeing :) |
packages/block-editor/src/components/block-list/use-in-between-inserter.js
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah sorry, I think I was using the wrong mode 🤦
This is what I see in select/edit mode:
Trunk:
Screen.Recording.2024-09-20.at.16.59.44.mov
This PR:
Screen.Recording.2024-09-20.at.17.01.44.mov
In that case, I think this is working as described!
Related #60021
Raised in this PR #65485
What?
This PR prevents the in-between block inserter from showing up when you hover between blocks within sections. The sections only allow content changes, so inserter shouldn't show up in these sections.
Testing Instructions
1- Open 2024 front page template
2- Switch to select mode
3- Hover the area right under the "about" button.
4- No in-between inserter should show up (in trunk it does show up there)