blender
/
blender
136
407
Fork 591
Code Issues 6.3k Pull Requests 775 Projects 19 Wiki Activity

Anim: Add 'Select Left' and 'Select Right Handles' entries in Graph Editor #120570

Open
Nika Kutsniashvili wants to merge 1 commits from nickberckley/blender:select-handles into main
pull from: nickberckley/blender:select-handles
merge into: blender:main

When changing the target branch, be careful to rebase the branch in your fork to match. See documentation.
Conversation 2 Commits 1 Files Changed 1 +9 -1
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
scripts/startup/bl_ui/space_graph.py
View File

@ -225,10 +225,18 @@ class GRAPH_MT_select(Menu):
props.mode = 'RIGHT'
layout.separator()
props = layout.operator("graph.select_key_handles", text="Select Handles")
props = layout.operator("graph.select_key_handles", text="Select Both Handles")
props.left_handle_action = 'SELECT'
props.right_handle_action = 'SELECT'
props.key_action = 'KEEP'
props = layout.operator("graph.select_key_handles", text="Select Left Handles")
props.left_handle_action = 'SELECT'
props.right_handle_action = 'KEEP'
Nathan Vegdahl commented 2024-04-19 12:58:58 +02:00
Review
Copy Link

Should this be DESELECT? (Also equivalently for left_handle_action in "Select Right Handles".) At least, my understanding was that the use case for this is to quickly select just the left or right handles, to grab and move them around.

Should this be `DESELECT`? (Also equivalently for `left_handle_action` in "Select Right Handles".) At least, my understanding was that the use case for this is to quickly select *just* the left or right handles, to grab and move them around.
Nika Kutsniashvili commented 2024-04-19 13:22:51 +02:00
Review
Copy Link

It can be. But before we have mechanism to choose between keeping selection or not, think this is more useful, because it gives you ability to quickly select both handles without key.

It can be. But before we have mechanism to choose between keeping selection or not, think this is more useful, because it gives you ability to quickly select both handles without key.
Nathan Vegdahl commented 2024-05-01 11:03:34 +02:00
Review
Copy Link

(Sorry for the delayed response!)

I guess it just seems kind of esoteric to me. From the name I would expect both the key and the opposite handle to be KEEP, or both to be DESELECT. Having one be DESELECT and the other KEEP feels odd and unexpected to me.

If selecting both handles + deselecting the key is a common enough use case to justify this (IMO) unexpected behavior, then I think it's common enough to instead just make its own menu item. "Select Only Handles" or something like that. And that way the user can immediately jump to the selection state they want to be in, without having to juggle or consider the current selection state to get there.

(Sorry for the delayed response!) I guess it just seems kind of esoteric to me. From the name I would expect both the key and the opposite handle to be `KEEP`, or both to be `DESELECT`. Having one be `DESELECT` and the other `KEEP` feels odd and unexpected to me. If selecting both handles + deselecting the key is a common enough use case to justify this (IMO) unexpected behavior, then I think it's common enough to instead just make its own menu item. "Select Only Handles" or something like that. And that way the user can immediately jump to the selection state they want to be in, without having to juggle or consider the current selection state to get there.
props.key_action = 'DESELECT'
props = layout.operator("graph.select_key_handles", text="Select Right Handles")
props.left_handle_action = 'KEEP'
props.right_handle_action = 'SELECT'
props.key_action = 'DESELECT'
props = layout.operator("graph.select_key_handles", text="Select Key")
props.left_handle_action = 'DESELECT'
props.right_handle_action = 'DESELECT'