[Bf-committers] Delkey for button reset problem (issue 5693045)

[Damir Prebeg]

That zero key was replaced 8 months ago with del key and undo bug was
fixed. So yes, currently del key should work on any field.
But problem is that, if you use del key in the node editor, it'll
delete active node. Until that that key becomes customizable, with
this patch I suggest it should be replaced with hardcoded backspace.

On 23 February 2012 14:52, Alberto Torres <kungfoobar at gmail.com> wrote:
> Will this also work for any input field? I used to press zero for
> resetting fields (but it has some bugs regarding undo) but it seems it
> was removed.
>
>
>
> 2012/2/23  <blend.factory at gmail.com>:
>> Reviewers: bf-codereview_blender.org, bf-committers_blender.org,
>>
>> Description:
>> Sorry for creating a new issue but for some reason old one
>> (http://codereview.appspot.com/4535083/) fails and throws an error when
>> I try to upload new patch set.
>>
>> Basically, I've seen some complaints on bf-commiters mailing list that
>> use of Del key under Node editor instead of field reset actually deletes
>> active node. So, this patch changes Delkey to Backspace key. Again, I
>> suggest that sole Backspace key should reset only active field and
>> Ctrl+Backspace should reset a whole active array.
>>
>> Please review this at http://codereview.appspot.com/5693045/
>>
>> Affected files:
>>   source/blender/editors/interface/interface_handlers.c
>>
>>
>> Index: source/blender/editors/interface/interface_handlers.c
>> ===================================================================
>> --- source/blender/editors/interface/interface_handlers.c       (revision 44356)
>> +++ source/blender/editors/interface/interface_handlers.c       (working copy)
>> @@ -4767,10 +4767,10 @@
>>                }
>>                /* reset to default */
>>                /* XXX hardcoded keymap check.... */
>> -               else if(ELEM(event->type, DELKEY, PADPERIOD) && event->val == KM_PRESS) {
>> -                       /* ctrl+del - reset active button; del - reset a whole array*/
>> +               else if(event->type == BACKSPACEKEY && event->val == KM_PRESS) {
>> +                       /* backspace = reset active button; ctrl+backspace = reset a whole
>> array*/
>>                        if (!(ELEM3(but->type, HSVCIRCLE, HSVCUBE, HISTOGRAM)))
>> -                               ui_set_but_default(C, !event->ctrl);
>> +                               ui_set_but_default(C, event->ctrl);
>>                }
>>                /* handle menu */
>>                else if(event->type == RIGHTMOUSE && event->val == KM_PRESS) {
>>
>>
>> _______________________________________________
>> Bf-committers mailing list
>> Bf-committers at blender.org
>> http://lists.blender.org/mailman/listinfo/bf-committers