Remove OPTYPE_REGISTER flag from click handler operator for anim editors
authorJoshua Leung <aligorith@gmail.com>
Thu, 14 Mar 2013 05:58:13 +0000 (05:58 +0000)
committerJoshua Leung <aligorith@gmail.com>
Thu, 14 Mar 2013 05:58:13 +0000 (05:58 +0000)
1) It made no sense to show this as the last operator which was used, since
these can only be used from the anim editors (and not the 3D View where this
panel appears most of the time)
2) Mouse select operators in other places didn't do this
3) There aren't really any editable parameters for this operator anyway
4) It's highly dependent on valid mouse coordinates as input.

Apart from that, undo still works fine, so no need to really keep this here.

source/blender/editors/animation/anim_channels_edit.c
source/blender/editors/space_nla/nla_channels.c

index 96dc1010e2cdd696364af046c3962383bcee944d..492762025171e3d6e8146d6642cb325b4d9c2898 100644 (file)
@@ -2652,7 +2652,7 @@ static void ANIM_OT_channels_click(wmOperatorType *ot)
        ot->poll = animedit_poll_channels_active;
        
        /* flags */
-       ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
+       ot->flag = OPTYPE_UNDO;
        
        /* properties */
        /* NOTE: don't save settings, otherwise, can end up with some weird behaviour (sticky extend) */
index ba1a3accf93b1e11dc423e8e64f05597fac1d8a0..1dccc874c179a580ecb560d24d499c74bfe7f016 100644 (file)
@@ -380,7 +380,7 @@ void NLA_OT_channels_click(wmOperatorType *ot)
        ot->poll = ED_operator_nla_active;
        
        /* flags */
-       ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
+       ot->flag = OPTYPE_UNDO;
        
        /* props */
        prop = RNA_def_boolean(ot->srna, "extend", 0, "Extend Select", ""); // SHIFTKEY