-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Navigation Block: dragging a block in List View modal causes the modal to close unexpectedly #34254
Comments
CC: @gwwar in case you have any ideas. |
I'm not actually experiencing the issue now. I wonder if my build had some issues. Let me know if you can reproduce, but if not I'll close this tomorrow. |
This will likely happen if a drag-&-drop triggers the primary onClick handler |
FWIW I'm not able to trigger this on trunk, so it's probably okay to close out. |
🤔 So I dug around a bit, and I think the modal closing on a navigation block item select is a side-effect of the selected block focus (which is technically outside of the modal). I'll spin up a smaller PR that alters this behavior so the modal remains open on select, it doesn't appear like this was intentional. |
Hmm, yeah. I haven't been able make it happen again! I wonder what it was. I had a hunch that maybe LinkControl was stealing focus when a navigation link doesn't have a URL, but that doesn't seem to be the case. |
With changes in #34337 this should no longer be possible 👍 |
Description
As soon as a user drags a block in the List View modal it closes unexpectedly.
This feature worked when the drag and drop feature was introduced, so this appears to be a regression.
Step-by-step reproduction instructions
Expected: It's possible to drag and drop a block
Actual: The modal closes unexpectedly and the drag operation is cancelled
Pre-checks
The text was updated successfully, but these errors were encountered: