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

Numerous drag and drop issues #67303

Open
2 of 6 tasks
ellatrix opened this issue Nov 26, 2024 · 4 comments
Open
2 of 6 tasks

Numerous drag and drop issues #67303

ellatrix opened this issue Nov 26, 2024 · 4 comments
Labels
[Feature] Drag and Drop Drag and drop functionality when working with blocks [Type] Bug An existing feature does not function as intended

Comments

@ellatrix
Copy link
Member

Description

Drag + scroll broken

Image

Drop trigger scroll to different location

Image

Can't drag within gallery

Image

Wrongful forbidden sign

Image

Undo after drag scroll to wrong location

Image

Step-by-step reproduction instructions

See GIFs

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
@ellatrix ellatrix added [Feature] Drag and Drop Drag and drop functionality when working with blocks [Type] Bug An existing feature does not function as intended labels Nov 26, 2024
@talldan
Copy link
Contributor

talldan commented Nov 27, 2024

Lots of issues.

Drag + scroll broken

It looks like this line returns the wrong element and then the effect tries setting scroll position on an element that doesn't have a scroll bar. It's possible the element that has the editor scrollbar changed at some point:

scrollParentYRef.current = getScrollContainer( event.target );

I think the element with the scrollbar is inside the iframe, and this hook listens for events on a button outside the iframe, so that might be a little interesting.

@ellatrix
Copy link
Member Author

@talldan Does the our custom scroll-when-dragging even work still? As far as I'm testing, it seems to try to scroll the main document outside the iframe, not within the iframe, so it doesn't have any effect. And what we see is actually the browser's default scroll behaviour.

@talldan
Copy link
Contributor

talldan commented Dec 3, 2024

Does the our custom scroll-when-dragging even work still?

It doesn't work!

As far as I'm testing, it seems to try to scroll the main document outside the iframe, not within the iframe, so it doesn't have any effect.

Yep, same thing I was trying to explain above. I probably should've led with something a sentence like "it isn't working at all" 😄

@ellatrix
Copy link
Member Author

ellatrix commented Dec 3, 2024

Ah, sorry, I misunderstood :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Drag and Drop Drag and drop functionality when working with blocks [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

2 participants