Commit: e167f9c9aa6f07360d3e92e7ed0f3b24ad65e153
Author: Antony Riakiotakis
Date:   Tue Sep 2 19:45:23 2014 +0200
Branches: terrible_consequencer
https://developer.blender.org/rBe167f9c9aa6f07360d3e92e7ed0f3b24ad65e153

Revert "Gooseberry feature request, allow select left-right strips
operator to select all"

This change has been in master all the time after all, exposed the
operation better in master.

This reverts commit 799f974668004b3d2b7873f51951eb22b5fcf2f3.

===================================================================

M       release/scripts/startup/bl_ui/space_sequencer.py
M       source/blender/editors/space_sequencer/sequencer_select.c

===================================================================

diff --git a/release/scripts/startup/bl_ui/space_sequencer.py 
b/release/scripts/startup/bl_ui/space_sequencer.py
index eefc1af..532047d 100644
--- a/release/scripts/startup/bl_ui/space_sequencer.py
+++ b/release/scripts/startup/bl_ui/space_sequencer.py
@@ -209,13 +209,6 @@ class SEQUENCER_MT_select(Menu):
 
         layout.operator("sequencer.select_active_side", text="Strips to the 
Left").side = 'LEFT'
         layout.operator("sequencer.select_active_side", text="Strips to the 
Right").side = 'RIGHT'
-        op = layout.operator("sequencer.select_active_side", text="All strips 
to the Left")
-        op.side = 'LEFT'
-        op.all = True
-        op = layout.operator("sequencer.select_active_side", text="All strips 
to the Right")
-        op.side = 'RIGHT'
-        op.all = True
-        
         layout.separator()
         layout.operator("sequencer.select_handles", text="Surrounding 
Handles").side = 'BOTH'
         layout.operator("sequencer.select_handles", text="Left Handle").side = 
'LEFT'
diff --git a/source/blender/editors/space_sequencer/sequencer_select.c 
b/source/blender/editors/space_sequencer/sequencer_select.c
index efc097e..8e49d56 100644
--- a/source/blender/editors/space_sequencer/sequencer_select.c
+++ b/source/blender/editors/space_sequencer/sequencer_select.c
@@ -80,12 +80,12 @@ static void select_surrounding_handles(Scene *scene, 
Sequence *test) /* XXX BRIN
 }
 
 /* used for mouse selection and for SEQUENCER_OT_select_active_side() */
-static void select_active_side(ListBase *seqbase, int sel_side, int channel, 
int frame, bool use_same_channel)
+static void select_active_side(ListBase *seqbase, int sel_side, int channel, 
int frame)
 {
        Sequence *seq;
 
        for (seq = seqbase->first; seq; seq = seq->next) {
-               if (!use_same_channel || channel == seq->machine) {
+               if (channel == seq->machine) {
                        switch (sel_side) {
                                case SEQ_SIDE_LEFT:
                                        if (frame > (seq->startdisp)) {
@@ -452,7 +452,7 @@ static int sequencer_select_invoke(bContext *C, wmOperator 
*op, const wmEvent *e
                                                                if (extend == 
0) ED_sequencer_deselect_all(scene);
                                                                seq->flag |= 
SELECT;
 
-                                                               
select_active_side(ed->seqbasep, SEQ_SIDE_LEFT, seq->machine, seq->startdisp, 
true);
+                                                               
select_active_side(ed->seqbasep, SEQ_SIDE_LEFT, seq->machine, seq->startdisp);
                                                        }
                                                        else {
                                                                if (extend == 
0) ED_sequencer_deselect_all(scene);
@@ -469,7 +469,7 @@ static int sequencer_select_invoke(bContext *C, wmOperator 
*op, const wmEvent *e
                                                                if (extend == 
0) ED_sequencer_deselect_all(scene);
                                                                seq->flag |= 
SELECT;
 
-                                                               
select_active_side(ed->seqbasep, SEQ_SIDE_RIGHT, seq->machine, seq->startdisp, 
true);
+                                                               
select_active_side(ed->seqbasep, SEQ_SIDE_RIGHT, seq->machine, seq->startdisp);
                                                        }
                                                        else {
                                                                if (extend == 
0) ED_sequencer_deselect_all(scene);
@@ -485,7 +485,7 @@ static int sequencer_select_invoke(bContext *C, wmOperator 
*op, const wmEvent *e
                                }
                                else {
                                        if (extend == 0) 
ED_sequencer_deselect_all(scene);
-                                       select_active_side(ed->seqbasep, 
sel_side, seq->machine, seq->startdisp, true);
+                                       select_active_side(ed->seqbasep, 
sel_side, seq->machine, seq->startdisp);
                                }
                        }
                        recurs_sel_seq(seq);
@@ -806,7 +806,7 @@ static int sequencer_select_active_side_exec(bContext *C, 
wmOperator *op)
 
        seq_act->flag |= SELECT;
 
-       select_active_side(ed->seqbasep, RNA_enum_get(op->ptr, "side"), 
seq_act->machine, seq_act->startdisp, false);
+       select_active_side(ed->seqbasep, RNA_enum_get(op->ptr, "side"), 
seq_act->machine, seq_act->startdisp);
 
        WM_event_add_notifier(C, NC_SCENE | ND_SEQUENCER | NA_SELECTED, scene);
 
@@ -829,7 +829,6 @@ void SEQUENCER_OT_select_active_side(wmOperatorType *ot)
 
        /* properties */
        RNA_def_enum(ot->srna, "side", prop_side_types, SEQ_SIDE_BOTH, "Side", 
"The side of the handle that is selected");
-       RNA_def_boolean(ot->srna, "all", false, "All", "Select in other 
channels as well");
 }

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to