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

Fix drag and drop regression on touchscreens in Tree #104657

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Alex2782
Copy link
Member

@Alex2782 Alex2782 commented Mar 26, 2025

Fixes #102256
Fixes #100393

This concerns the user actions drag scrolling and drag & drop. There were conflicts, and drag & drop was disabled 8 months ago. There are several options for scrolling but none for drag and drop on touchscreens.

More details: #94422 (comment)

This PR includes:

  • new property drag_scrolling_enabled (default = false)
  • InputEventPanGesture adaptation for Android touchscreens (TODO)

Test project: bug_102256_treedragtest.zip

14s video, Test on MacOS

with Emulate Touch From Mouse

Screen-2025-03-26-173540.mp4

TODOs:

  • more tests on Android
  • scroll_scale = 32 for InputEventPanGesture ok with Apple Magic Trackpad / Mouse / Android Touchscreens ?
  • Android Editor: drag_scrolling_enabled = true, where drag and drop is not possible ?

@AThousandShips
Copy link
Member

I don't think the code style sweep should be done as part of this, it pollutes the history

@Alex2782 Alex2782 force-pushed the fix_tree_drag_and_drop branch from f7f3e08 to 17fedac Compare March 27, 2025 23:29
@Alex2782
Copy link
Member Author

I don't think the code style sweep should be done as part of this, it pollutes the history

New PR for cleanup part #104708

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants