UI: rename "Select Playhead" to "Select -> Side of Current Frame"

Make this consistent with meshes select side of active,
also rename "Under" to "Overlap" as this is confusing since
strips can be considered above/under each other
relative to their channels.
This commit is contained in:
Campbell Barton 2020-06-01 14:46:46 +10:00
parent 946ae26349
commit c71be9a552
4 changed files with 11 additions and 11 deletions

View File

@ -2522,13 +2522,13 @@ def km_sequencer(params):
("sequencer.select", {"type": 'RIGHT_BRACKET', "value": 'PRESS'},
{"properties": [("left_right", 'RIGHT'), ("linked_time", True)]}),
("sequencer.select", {"type": 'EQUAL', "value": 'PRESS'},
{"properties": [("left_right", 'UNDER'), ("linked_time", True)]}),
{"properties": [("left_right", 'OVERLAP'), ("linked_time", True)]}),
("sequencer.select", {"type": 'LEFT_BRACKET', "value": 'PRESS', "shift": True},
{"properties": [("left_right", 'LEFT'), ("linked_time", True), ("extend", True)]}),
("sequencer.select", {"type": 'RIGHT_BRACKET', "value": 'PRESS', "shift": True},
{"properties": [("left_right", 'RIGHT'), ("linked_time", True), ("extend", True)]}),
("sequencer.select", {"type": 'EQUAL', "value": 'PRESS', "shift": True},
{"properties": [("left_right", 'UNDER'), ("linked_time", True), ("extend", True)]}),
{"properties": [("left_right", 'OVERLAP'), ("linked_time", True), ("extend", True)]}),
*_template_items_context_menu("SEQUENCER_MT_context_menu", params.context_menu_event),
])

View File

@ -394,14 +394,14 @@ class SEQUENCER_MT_select_linked(Menu):
layout.operator("sequencer.select_more", text="More")
class SEQUENCER_MT_select_playhead(Menu):
bl_label = "Select Playhead"
class SEQUENCER_MT_select_side_of_frame(Menu):
bl_label = "Side of Current Frame"
def draw(self, _context):
layout = self.layout
props = layout.operator("sequencer.select", text="Under")
props.left_right = 'UNDER'
props = layout.operator("sequencer.select", text="Overlap")
props.left_right = 'OVERLAP'
props.linked_time = True
props = layout.operator("sequencer.select", text="Left")
props.left_right = 'LEFT'
@ -429,7 +429,7 @@ class SEQUENCER_MT_select(Menu):
layout.separator()
layout.menu("SEQUENCER_MT_select_playhead", text="Playhead")
layout.menu("SEQUENCER_MT_select_side_of_frame")
layout.menu("SEQUENCER_MT_select_handle", text="Handle")
layout.menu("SEQUENCER_MT_select_channel", text="Channel")
layout.menu("SEQUENCER_MT_select_linked", text="Linked")
@ -2221,7 +2221,7 @@ classes = (
SEQUENCER_MT_view_toggle,
SEQUENCER_MT_preview_zoom,
SEQUENCER_MT_proxy,
SEQUENCER_MT_select_playhead,
SEQUENCER_MT_select_side_of_frame,
SEQUENCER_MT_select_handle,
SEQUENCER_MT_select_channel,
SEQUENCER_MT_select_linked,

View File

@ -191,7 +191,7 @@ enum {
SEQ_SELECT_LR_MOUSE,
SEQ_SELECT_LR_LEFT,
SEQ_SELECT_LR_RIGHT,
SEQ_SELECT_LR_UNDER_PLAYHEAD,
SEQ_SELECT_LR_OVERLAP,
};
/* Defines used internally. */

View File

@ -436,7 +436,7 @@ static int sequencer_select_exec(bContext *C, wmOperator *op)
case SEQ_SELECT_LR_RIGHT:
x = CFRA + 1.0f;
break;
case SEQ_SELECT_LR_UNDER_PLAYHEAD:
case SEQ_SELECT_LR_OVERLAP:
default:
x = CFRA;
break;
@ -645,7 +645,7 @@ void SEQUENCER_OT_select(wmOperatorType *ot)
{SEQ_SELECT_LR_MOUSE, "MOUSE", 0, "Mouse", "Use mouse position for selection"},
{SEQ_SELECT_LR_LEFT, "LEFT", 0, "Left", "Select to the left of the current frame"},
{SEQ_SELECT_LR_RIGHT, "RIGHT", 0, "Right", "Select to the right of the current frame"},
{SEQ_SELECT_LR_UNDER_PLAYHEAD, "UNDER", 0, "Under", "Select under the current frame"},
{SEQ_SELECT_LR_OVERLAP, "OVERLAP", 0, "Overlap", "Select overlapping the current frame"},
{0, NULL, 0, NULL, NULL},
};
PropertyRNA *prop;