Differential D7707
VSE: Add Select Neighboring handles Authored by Peter Fog (tintwotin) on May 12 2020, 5:14 PM.
Details This patch add the missing select neighboring handles functions(gif):  These functions are essential for adding the missing Strip Slide and Split Roll trim functions, which now can be implemented easily after this Select Neighboring Handles patch is committed. Not part of this patch(as single functions)  Split Roll(gif): 
Diff Detail Event TimelineComment Actions There is a problem, that you are looping and operating over strips that have seq->flag & SELECT set. If you set this flag to strip that has not yet been looped over, it will be considered as if it has been selected before you started operator. I guess easiest fix is to ony set SEQ_LEFTSEL and SEQ_RIGHTSEL flags and loop over strips again where you would finally select them. Comment Actions Cleaned up the code a bit and added shortcuts: Select Left Neighbor Handle = [ + shift [ and ] are already used for selecting on either side of the playhead, so when select Under is committed the shortcut should be Equal. Equal used to be assigned to Insert Gap, this has now been assigned Backspace + Ctrl. Comment Actions Remove the shortcuts again, as the brackets also are used in extended state(+shift). I can't find a triangle of keys with +shift free to use, so I'll leave the shortcut keys out of this patch. Comment Actions This seems to work ok. You have removed changes from space_sequencer.py so there isn't menu items for this feature.
Comment Actions Reinserted the menu options. Unhide and unlock changed to use ctrl instead of alt(they should be converted to toggles at some point)). Need instructions on what to do about the enum? Comment Actions I think I would accept this, but I am quite confused about shortcuts:
I think best approach would be to commit without keymap changes Comment Actions It's fine with me, if you want to commit this without the shortcut keys, but at some point before the next release, we must deal with all of the missing shortcuts in the Sequencer. Comment Actions Ah sorry I missed the alt modifier. Will check again. I will have to commit with changes anyway because of enum I mentioned in inlines. Comment Actions Updated to Master in the hope of getting it committed in 2.92. Shortcut keys removed to avoid debate. | ||||||