-
-
Notifications
You must be signed in to change notification settings - Fork 21.1k
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
Allow to remove signal connections using Delete
#82821
Allow to remove signal connections using Delete
#82821
Conversation
35c62d9
to
e19008b
Compare
e19008b
to
27081a9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment style nitpicks. I don't see an issue in the rest, other than I don't think connection
needs to be initialized as it's only used once.
Please amend the commit message so it's meaningful, "Initial Commit" isn't a very good description for this change ;) |
27081a9
to
027a853
Compare
Cheers, should've been more careful with the comments. I got an error when I removed the
Done! 😄 |
Makes sense then to keep it then |
Please squash commits into one. To do this you need a local git client, GitHub web interface doesn't support it. |
da75d05
to
3b9fc45
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Note that when the Script workspace is open, you can press Delete regardless of the mouse cursor position. But when a 2D or 3D workspace is open, the cursor must be inside the Connection Editor, otherwise deleting a node in the Scene Tree will be triggered. This is not directly related to this PR, just a note.
Delete
Thanks! And congrats on your first merged Godot pull request! |
This pull request improves usability with the Node connection panel, to allow the user to quickly delete connections by pressing the DEL key.
The
_rmb_pressed
method in theConnectionsDock
class has been renamed to_gui_input
to reflect the new handling of other inputs.This fixes the usability issue outlined in #82616.