Skip to content

Conversation

@assembled-michael
Copy link

We noticed that the behavior on Draggable was different when right-clicking vs. ctrl-clicking, even though they should be treated the same. This modifies it so ctrl-clicks are not considered as left-clicks.

@misumi-khoi-nguyen-01
Copy link

Good

@STRML
Copy link
Collaborator

STRML commented Dec 31, 2025

Closing this PR as stale. This PR has been open since 2020 without being merged. If ctrl-click handling is still an issue, please open a fresh PR against the current codebase. Thank you for the contribution!

@STRML STRML closed this Dec 31, 2025
@STRML
Copy link
Collaborator

STRML commented Dec 31, 2025

Reopening - this is a correct fix for macOS ctrl+click behavior.

@STRML STRML reopened this Dec 31, 2025
STRML added a commit that referenced this pull request Dec 31, 2025
On macOS, ctrl+click opens the context menu (equivalent to right-click).
This change prevents drag from starting when ctrl is held during click,
matching the expected behavior.

Closes #498
@STRML
Copy link
Collaborator

STRML commented Dec 31, 2025

Closing in favor of #786 which implements this fix against the current codebase. Thank you for the contribution!

@STRML STRML closed this Dec 31, 2025
STRML added a commit that referenced this pull request Dec 31, 2025
On macOS, ctrl+click opens the context menu (equivalent to right-click).
This change prevents drag from starting when ctrl is held during click,
matching the expected behavior.

Closes #498
STRML added a commit that referenced this pull request Dec 31, 2025
On macOS, ctrl+click opens the context menu (equivalent to right-click).
This change prevents drag from starting when ctrl is held during click,
matching the expected behavior.

Closes #498
STRML added a commit that referenced this pull request Dec 31, 2025
On macOS, ctrl+click opens the context menu (equivalent to right-click).
This change prevents drag from starting when ctrl is held during click,
matching the expected behavior.

Closes #498
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

Successfully merging this pull request may close these issues.

3 participants