-
Notifications
You must be signed in to change notification settings - Fork 742
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
fix(wasm): unwanted preview shown when dragging over existing selection #18921
fix(wasm): unwanted preview shown when dragging over existing selection #18921
Conversation
to test this change:
@dr1rrb while this doesnt break the uno drag-n-drop. we have issue with native drag-n-drop, as that is effectively disabled... is this acceptable? |
This comment was marked as outdated.
This comment was marked as outdated.
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18921/index.html |
d2b3967
to
9e62116
Compare
9e62116
to
210bae8
Compare
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18921/index.html |
🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18921/index.html |
|
Yes it acceptable ... and it's actually the preferred behavior. Just like the right click and text selection has been disabled in the browser, we need to disable that. Uno's app are application embedded in a browser. It has to be opt-in if you want to restore that behavior, and the way to opt-in is to use the WinUI APIs of drag and drop, not injecting JS. |
210bae8
to
fb0185a
Compare
fb0185a
to
b6dfeab
Compare
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18921/index.html |
🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18921/index.html |
b6dfeab
to
2436ac6
Compare
🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18921/index.html |
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18921/index.html |
The build 148557 found UI Test snapshots differences: Details
|
|
GitHub Issue (If applicable): closes #18854, closes unoplatform/kahua-private#228
PR Type
What kind of change does this PR introduce?
What is the current behavior?
On WebAssembly, when there is selection underneath or around a control that typically uses drag gesture (eg: ScrollBar or Slider), the drag gesture will only work for the 1st frame/tick/delta, subsequent pointer capture events are lost, and there is an undesired browser dragging preview that follows the cursor.
What is the new behavior?
The browser dragstart event are globally blocked. Note that doesn't affect the DragEventArgs.DragUIOverride preview as that is rendered on the uno side, nor the control's ability to receive drop (handled via dragenter & dragleave).
PR Checklist
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Run
results.Other information