Cleanup: VSE: Tidy RCS select box tool keymap

Make `.seq_slide` happen on drag instead of immediate press.

Also remove the currently unused left-click box select items on the
tool (to be added back if/once we create a separate "scrub" tool).

Pull Request: https://projects.blender.org/blender/blender/pulls/153097
This commit is contained in:
John Kiril Swenson 2026-01-20 14:26:59 -06:00 • committed by Gitea
parent 4ffed0523f
commit ee44e3d752

View file

@ -8586,7 +8586,7 @@ def km_sequencer_tool_generic_select_rcs(params):
# Change frame takes precedence over the sequence slide operator. If a
# mouse press happens on a strip handle, it is canceled, and the sequence
# slide below activates instead.
("transform.seq_slide", {"type": 'LEFTMOUSE', "value": 'PRESS'},
("transform.seq_slide", {"type": 'LEFTMOUSE', "value": 'CLICK_DRAG'},
{"properties": [("view2d_edge_pan", True), ("use_restore_handle_selection", True)]}),
]
@ -8613,7 +8613,10 @@ def km_sequencer_tool_generic_select_box(params, *, fallback):
if (params.select_mouse == 'RIGHTMOUSE') else
km_sequencer_tool_generic_select_lcs(params)),
# Don't use `tool_maybe_tweak_event`, see comment for this slot.
*([] if (fallback and not params.use_fallback_tool) else _template_items_tool_select_actions_simple(
# We do not add box select with left-click keymap items for RCS since click-drag already scrubs.
# These items should be re-added to this tool if/once we create a separate scrub tool.
*([] if (params.select_mouse == 'RIGHTMOUSE' or (fallback and not params.use_fallback_tool))
else _template_items_tool_select_actions_simple(
"sequencer.select_box",
**(params.select_tweak_event if (fallback and params.use_fallback_tool_select_mouse) else
params.tool_tweak_event),