Animation: Add Slider operators to hotkey menu #104530
Currently the slider operators for the Graph Editor are not easily accessible since they don't have a hotkey.
While we could go down the route of giving each operator it's own hotkey, I can see a lot more operators being added in the future, so we need an option that scales better.
I think the most "Blender" way of doing that would be to use a popup menu such as when you press T in the graph editor to change the interpolation type.
That has the advantage, that your last used operator will always be under the cursor, so it's quick to access.
(It would be far better if pressing the hotkey twice would trigger it as well, but that's beside the point)
Additionally we can group operators to different popups
Currently they trigger with D (just because that wasn't used) but we could add another menu that triggers e.g. with SHIFT + D when we need to regroup
That is also the reason why I made a completely new operator instead of refactoring the existing operators into 1 with an enum.
Instead of making a new C operator with an enum and such, it is much simpler to go through python instead. In this case I am just reusing the existing menu, but it's even easier to change in the future.
original phabricator patch
Apart from the submodule change (that shouldn't be in this PR) the code LGTM. This is something to be discussed with animators, though, so I won't press the accept button just yet.
I'll add this to the agenda of the next module meeting. If there wasn't a decision before that time already, we can decide during the meeting.
I think this is the way to go.
But i dont know how to approve things in this new platform :(
I think 'D' is fine when it comes to the hotkey. It's the first one that's still free in the word "Slider", which IMO is good enough.
No due date set.
No dependencies set.
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?