Now the tweak mode is on by default and regardless the setting (Drag
Immediately).

Could this fix check for the option at least?
I'm not a fan of the tweak mode but it can get really annoying after
moving stuff around by mistake > unstable undo > crash :(


On Sun, Mar 14, 2010 at 19:58, Martin Poirier <the...@yahoo.com> wrote:
> Revision: 27494
>          
> http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=27494
> Author:   theeth
> Date:     2010-03-14 19:58:14 +0100 (Sun, 14 Mar 2010)
>
> Log Message:
> -----------
> [#21523] Drag Immediately only working when "Select With" is set to LMB
>
> Partial fix. Check lauch event if left or right mouse.
>
> Also added a bug fix for manipulator (sometimes, type = 0)
>
> Modified Paths:
> --------------
>    trunk/blender/source/blender/editors/transform/transform.c
>
> Modified: trunk/blender/source/blender/editors/transform/transform.c
> ===================================================================
> --- trunk/blender/source/blender/editors/transform/transform.c  2010-03-14 
> 18:22:04 UTC (rev 27493)
> +++ trunk/blender/source/blender/editors/transform/transform.c  2010-03-14 
> 18:58:14 UTC (rev 27494)
> @@ -1046,7 +1046,7 @@
>
>                /* confirm transform if launch key is released after mouse 
> move */
>                /* XXX Keyrepeat bug in Xorg fucks this up, will test when 
> fixed */
> -               if (event->type == LEFTMOUSE /*t->launch_event*/ && t->state 
> != TRANS_STARTING)
> +               if (event->type == t->launch_event && (t->launch_event == 
> LEFTMOUSE || t->launch_event == RIGHTMOUSE) && t->state != TRANS_STARTING)
>                {
>                        t->state = TRANS_CONFIRM;
>                }
> @@ -1456,6 +1456,22 @@
>
>        t->launch_event = event ? event->type : -1;
>
> +       if (t->launch_event == EVT_TWEAK_R)
> +       {
> +               t->launch_event = RIGHTMOUSE;
> +       }
> +       else if (t->launch_event == EVT_TWEAK_L)
> +       {
> +               t->launch_event = LEFTMOUSE;
> +       }
> +       // XXX Remove this when wm_operator_call_internal doesn't use 
> window->eventstate (which can have type = 0)
> +       // For manipulator only, so assume LEFTMOUSE
> +       else if (t->launch_event == 0)
> +       {
> +               t->launch_event = LEFTMOUSE;
> +       }
> +
> +
>        if (!initTransInfo(C, t, op, event))                                   
>  // internal data, mouse, vectors
>        {
>                return 0;
>
>
> _______________________________________________
> Bf-blender-cvs mailing list
> bf-blender-...@blender.org
> http://lists.blender.org/mailman/listinfo/bf-blender-cvs
>
_______________________________________________
Bf-committers mailing list
Bf-committers@blender.org
http://lists.blender.org/mailman/listinfo/bf-committers

Reply via email to