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

MudDropContainer : EventCallback ItemDropped triggered unexpectedly on mobile devices #7323

Open
1 of 2 tasks
chatcoda opened this issue Aug 9, 2023 · 4 comments
Open
1 of 2 tasks

Comments

@chatcoda
Copy link

chatcoda commented Aug 9, 2023

Bug type

Component

Component name

MudDropContainer

What happened?

Under certain circumstances, MudDropContainer's ItemDropped EventCallback is triggered unexpectedly on mobile devices only.
It seems to occur if another EventCallback exists on an item rendered within the MudDropContainer.
Note: could be the same cause than #7182

Expected behavior

MudDropContainer's ItemDropped EventCallback should not be triggered unexpectedly on mobile devices.

Reproduction link

https://try.mudblazor.com/snippet/mumnOCaXzXjgFUcX

Reproduction steps

  1. Enable device emulation in browser (iOS or Android)
  2. Click on DropItemX
  3. See that EventCallback ItemDropped is triggered (event is written below)

Relevant log output

No response

Version (bug)

6.8.0

Version (working)

No response

What browsers are you seeing the problem on?

Microsoft Edge, Safari

On what operating system are you experiencing the issue?

Windows, iOS

Pull Request

  • I would like to do a Pull Request

Code of Conduct

  • I agree to follow this project's Code of Conduct
@chatcoda chatcoda added the triage label Aug 9, 2023
@danielchalmers
Copy link
Contributor

Is this still a problem in v6.19.1? May have been fixed by #7424 - I think it's working on my end

@chatcoda
Copy link
Author

@chatcoda chatcoda reopened this Mar 31, 2024
@danielchalmers
Copy link
Contributor

Oh I misunderstood the premise completely, thanks for the update.

If you're able to take a look at submitting a PR yourself I'll make sure it gets reviewed quickly. Otherwise I don't see this getting fixed in the near future if I'm being honest. Too time constrained.

@ScarletKuro
Copy link
Member

Honestly, to fix most problems with the touch devices, it would probably require to use this polyfill DragDropTouch (it's not too big) with this modification Postlagerkarte/blazor-dragdrop#147 (comment)

@henon henon removed the triage label May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants