[Bf-taskforce25] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [19079] branches/blender2.5/blender/source /blender/editors/animation/anim_channels.c: Animation Editors: Added Tab-Key to toggle editability of selected channels.

[Joshua Leung]

Hi,

Unless I'm mistaken, we currently have separate operators for tools which
perform completely different operations/work in completely different ways
(i.e. subdivide tools are separate operators, with a dummy 'menu' operator
which groups all of them into a single one)

However, for operators which work in the same way, but perhaps set a
different flag, it is possible to have a single operator. For example, take
some of the various tools such as 'Toggle Channel Setting' in
dopesheet/graph editors. These tools 'could' be split up to have several
operators, one for each different flag that gets set, etc. which use the
same exec/etc. calls if it is really necessary. However, IMO, that's a bit
overkill.

These tools present a popup menu which lists the different ways they work
(i.e. with the 'Toggle Channel Setting' tool, this is menu is to let the
user choose what setting gets toggled for all the selected channels). Now,
if a user was to want to set a keymap which used one of the options (as in
the case of the operator which I've committed here), such an option is not
possible.


Regards,
Aligorith

On Sun, Feb 22, 2009 at 6:55 PM, Joshua Leung <aligorith at gmail.com> wrote:

> Revision: 19079
>
> http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=19079
> Author:   aligorith
> Date:     2009-02-22 06:55:37 +0100 (Sun, 22 Feb 2009)
>
> Log Message:
> -----------
> Animation Editors: Added Tab-Key to toggle editability of selected
> channels.
>
> Ton - Currently, I've had to add a new operator to set this, since it is
> not possible to specify via keymaps whether the invoke or exec should be
> called by default for an operator.
> (See corresponding mail on 2.5 mailing list for details about this)
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.blender.org/pipermail/bf-taskforce25/attachments/20090222/38a62b42/attachment.htm