[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [38452] trunk/blender/source/blender/ editors/space_node/node_header.c: Removed the autoconnect call when adding new nodes, this hardly ever gives usable results and leads to annoyed artists.

[PabloVazquez.org]

Its been a topic for a while already, but in my perspective, -1 on this.

In 2.4x was more intuitive, adding a Mix node with two nodes selected
(say two rgbcurves) will automatically connect them, saving time, of
course would be better if they connect in the order we select them, so
is more predictable but still was better than now only connecting with
the active node.
Another example, adding a Separate RGB then Combine RGB node would
connect all of the inputs/outputs in 2.4x, not the case in 2.5x.

I know is easier to remove than to fix, but at least should be a preference.

*edits his node_header.c back...*


On Sun, Jul 17, 2011 at 18:14, Lukas Toenne <lukas.toenne at googlemail.com> wrote:
> Revision: 38452
>          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=38452
> Author:   lukastoenne
> Date:     2011-07-17 16:14:52 +0000 (Sun, 17 Jul 2011)
> Log Message:
> -----------
> Removed the autoconnect call when adding new nodes, this hardly ever gives usable results and leads to annoyed artists.
>
> Modified Paths:
> --------------
>    trunk/blender/source/blender/editors/space_node/node_header.c
>
> Modified: trunk/blender/source/blender/editors/space_node/node_header.c
> ===================================================================
> --- trunk/blender/source/blender/editors/space_node/node_header.c       2011-07-17 13:29:50 UTC (rev 38451)
> +++ trunk/blender/source/blender/editors/space_node/node_header.c       2011-07-17 16:14:52 UTC (rev 38452)
> @@ -94,8 +94,6 @@
>                if(node->flag & NODE_TEST) node->flag |= NODE_SELECT;
>        }
>
> -       snode_autoconnect(snode, 1, 0);
> -
>        /* deselect after autoconnection */
>        for(node= snode->edittree->nodes.first; node; node= node->next) {
>                if(node->flag & NODE_TEST) node->flag &= ~NODE_SELECT;
>
> _______________________________________________
> Bf-blender-cvs mailing list
> Bf-blender-cvs at blender.org
> http://lists.blender.org/mailman/listinfo/bf-blender-cvs
>