Skip to content

ParameterInput: Detect click & drag gestures #233

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

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

dylanmccall
Copy link
Contributor

If the user clicks and drags inside ParameterInput, start dragging the parent block. This makes it easier to manipulate blocks which contain a large number of parameter input widgets.

@dylanmccall
Copy link
Contributor Author

dylanmccall commented Sep 25, 2024

Here's a recording showing this in action:

Screencast.From.2024-09-25.15-38-51.webm

Note that this changes our option widgets to open when the mouse is released, instead of when the mouse is pressed. The Godot editor is a little inconsistent on which behaviour it uses, but I think it works better this way (and it's necessary for this feature to work).

Copy link
Contributor

@manuq manuq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes the dragging work much better while editing. A nice touch.

@dylanmccall dylanmccall force-pushed the parameter-input-drag-event branch from 8cc05ba to 19111ca Compare September 26, 2024 01:43
If the user clicks and drags inside ParameterInput, start dragging the
parent block. This makes it easier to manipulate blocks which contain a
large number of parameter input widgets.
@dylanmccall dylanmccall force-pushed the parameter-input-drag-event branch from 19111ca to 6daa3ca Compare September 26, 2024 02:02
@dylanmccall dylanmccall merged commit c7b6517 into main Sep 26, 2024
3 checks passed
@dylanmccall dylanmccall deleted the parameter-input-drag-event branch September 26, 2024 02:02
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.

2 participants