Key modifier(custom hotkey) does not trigger on a proper priority level. #44349

New Issue

Blendy · 2015-04-11T07:40:07+02:00

Blendy commented

2015-04-11 07:40:07 +02:00

System Information
Windows 7 64bit OS & AMD HD6770

Blender Version
Broken: (2.74 and previous versions)
Worked: (not sure, I just started using Blender since 2.63.)

Short description of error
When I bound "S key"+"right mouse" as view3d.zoom operator hotkey (It's Softimage style viewport navigation), the S key(as a keymodifier) does not take same priority as Ctrl, Shift and Alt key have. The problem is, for example, I wrote a custom python operator as a new selection method and bound "right mouse" as its hotkey, then, I attempted to zoom the 3d view by press "S + right mouse", Blender firstly only triggered the custom selection operator I wrote but couldn't zoom the 3d view as I supposed to. Whereas, when I bound "Alt + right mouse" as zoom 3d view hotkey, it worked properly. Maybe few people use Softimage style viewport navigation layout in Blender, so this problem was not so noticeable. I really hope you can fix it, and thanks for keeping Blender awesome :)
.
Exact steps for others to reproduce the error
Based on a (as simple as possible) attached .blend file with minimum amount of steps

**System Information** Windows 7 64bit OS & AMD HD6770 **Blender Version** Broken: (2.74 and previous versions) Worked: (not sure, I just started using Blender since 2.63.) **Short description of error** When I bound "S key"+"right mouse" as view3d.zoom operator hotkey (It's Softimage style viewport navigation), the S key(as a keymodifier) does not take same priority as Ctrl, Shift and Alt key have. The problem is, for example, I wrote a custom python operator as a new selection method and bound "right mouse" as its hotkey, then, I attempted to zoom the 3d view by press "S + right mouse", Blender firstly only triggered the custom selection operator I wrote but couldn't zoom the 3d view as I supposed to. Whereas, when I bound "Alt + right mouse" as zoom 3d view hotkey, it worked properly. Maybe few people use Softimage style viewport navigation layout in Blender, so this problem was not so noticeable. I really hope you can fix it, and thanks for keeping Blender awesome :) . **Exact steps for others to reproduce the error** Based on a (as simple as possible) attached .blend file with minimum amount of steps

Blendy commented

2015-04-11 07:40:07 +02:00

Changed status to: 'Open'

Brecht Van Lommel was assigned by Blendy

2015-04-11 07:40:07 +02:00

Blendy commented

2015-04-11 07:40:07 +02:00

Added subscriber: @Rays-Lv

Julian Eisel commented

2015-04-13 22:08:43 +02:00

Added subscriber: @JulianEisel

Julian Eisel commented

2015-04-13 22:08:43 +02:00

Did you assign the S-key using this field? :

Did you assign the S-key using this field? : ![modifier_key_bug.png](https://archive.blender.org/developer/F161656/modifier_key_bug.png)

Blendy commented

2015-04-14 04:19:23 +02:00

In #44349#302484, @JulianEisel wrote:
Did you assign the S-key using this field? :

Yes，I exactly assign the S-key using this field. And it works perfectly in most circumstances, even Blender's native select operator written by C-language works properly with this key. Only python Operators will take higher priority over this key. So it doesn't make good sense, it's supposed to be a bug.

> In #44349#302484, @JulianEisel wrote: > Did you assign the S-key using this field? : ![modifier_key_bug.png](https://archive.blender.org/developer/F161656/modifier_key_bug.png) Yes，I exactly assign the S-key using this field. And it works perfectly in most circumstances, even Blender's native select operator written by C-language works properly with this key. Only python Operators will take higher priority over this key. So it doesn't make good sense, it's supposed to be a bug.

Julian Eisel commented

2015-04-21 19:55:24 +02:00

So you have an py-written operator and you want it to be called on RMB and you want the zoom operator to be triggered on S+RMB, right? Would be useful to see your script then, at least a simplified version of it, so I can test on my own

Blendy commented

2015-04-23 11:55:02 +02:00

In #44349#304318, @JulianEisel wrote:
So you have an py-written operator and you want it to be called on RMB and you want the zoom operator to be triggered on S+RMB, right? Would be useful to see your script then, at least a simplified version of it, so I can test on my own

Actually, it doesn't depend on what script written in Operator. It's like that ANY python operator causes the problem, even a blender native python Operator causes this problem, for example, like "Select All Toggle" operator(A key by default), you can still press A to toggle select all objects while S (S had already been set as a key modifier as I said previously) is being held. Whereas, while you're holding Alt or Shift or Ctrl, you're not able to toggle select any more, it's working properly. The key modifier should've been working that way as Alt, Shift or Ctrl do.

It seems that Blender haven't offered an event which can test if a key modifier is held, there are only event.alt, event.shift, event.ctrl in python APIs. Recently, I dug into python coding in blender a little bit, and found a way to test if S is held during the whole runtime of blender. This can be used for avoiding that operators conflict issue. But it's still a workaround, as dev team, you should implement it in some more proper way. I wrote a "testing if SKey is held" python script sample, it's based on a python template in blender.
.blend file here: test_if_SKey_held.blend
You can play with it, simply run the script in the text editor, then press S key a few times in 3d view, you'll see the difference. Hope it's helpful for you team to a proper solution soon.

> In #44349#304318, @JulianEisel wrote: > So you have an py-written operator and you want it to be called on RMB and you want the zoom operator to be triggered on S+RMB, right? Would be useful to see your script then, at least a simplified version of it, so I can test on my own Actually, it doesn't depend on what script written in Operator. It's like that ANY python operator causes the problem, even a blender native python Operator causes this problem, for example, like "Select All Toggle" operator(A key by default), you can still press A to toggle select all objects while S (S had already been set as a key modifier as I said previously) is being held. Whereas, while you're holding Alt or Shift or Ctrl, you're not able to toggle select any more, it's working properly. The key modifier should've been working that way as Alt, Shift or Ctrl do. It seems that Blender haven't offered an event which can test if a key modifier is held, there are only event.alt, event.shift, event.ctrl in python APIs. Recently, I dug into python coding in blender a little bit, and found a way to test if S is held during the whole runtime of blender. This can be used for avoiding that operators conflict issue. But it's still a workaround, as dev team, you should implement it in some more proper way. I wrote a "testing if SKey is held" python script sample, it's based on a python template in blender. .blend file here: [test_if_SKey_held.blend](https://archive.blender.org/developer/F166221/test_if_SKey_held.blend) You can play with it, simply run the script in the text editor, then press S key a few times in 3d view, you'll see the difference. Hope it's helpful for you team to a proper solution soon.

Bastien Montagne commented

2015-05-01 09:55:15 +02:00

Added subscriber: @mont29

Brecht Van Lommel removed their assignment 2015-05-07 23:04:00 +02:00

Sergey Sharybin commented

2015-05-25 21:08:21 +02:00

Added subscriber: @Sergey

Sergey Sharybin commented

2015-05-25 21:08:21 +02:00

Changed status from 'Open' to: 'Archived'

Sergey Sharybin closed this issue

2015-05-25 21:08:21 +02:00

Sergey Sharybin self-assigned this 2015-05-25 21:08:21 +02:00

Sergey Sharybin commented

2015-05-25 21:08:21 +02:00

To me it seems to be just a famous keymapping conflict, where pressing S (press == hold the key down) triggers transform operator and surely prevents S-RMB from invoking.

You can work around this locally by re-assigning scale to click event. It's not ideal but should work for you.

Re-considering the shortcuts is in our list for improvements and will happen outside of the bug tracker. So thanks for the report, but archiving it now.

To me it seems to be just a famous keymapping conflict, where pressing S (press == hold the key down) triggers transform operator and surely prevents S-RMB from invoking. You can work around this locally by re-assigning scale to click event. It's not ideal but should work for you. Re-considering the shortcuts is in our list for improvements and will happen outside of the bug tracker. So thanks for the report, but archiving it now.

Blendy commented

2015-07-02 07:41:39 +02:00

In #44349#311375, @Sergey wrote:
To me it seems to be just a famous keymapping conflict, where pressing S (press == hold the key down) triggers transform operator and surely prevents S-RMB from invoking.

You can work around this locally by re-assigning scale to click event. It's not ideal but should work for you.

Re-considering the shortcuts is in our list for improvements and will happen outside of the bug tracker. So thanks for the report, but archiving it now.

Thank you, but I'd already been not using S-key for scaling since the day I started to use blender. Instead, I always use X-key for all scaling operation types.

> In #44349#311375, @Sergey wrote: > To me it seems to be just a famous keymapping conflict, where pressing S (press == hold the key down) triggers transform operator and surely prevents S-RMB from invoking. > > You can work around this locally by re-assigning scale to click event. It's not ideal but should work for you. > > Re-considering the shortcuts is in our list for improvements and will happen outside of the bug tracker. So thanks for the report, but archiving it now. Thank you, but I'd already been not using S-key for scaling since the day I started to use blender. Instead, I always use X-key for all scaling operation types.

CJ Ferguson commented

2019-11-08 16:41:02 +01:00

Added subscriber: @CJ-Ferguson

CJ Ferguson commented

2019-11-08 16:41:02 +01:00

This comment was removed by @CJ-Ferguson

*This comment was removed by @CJ-Ferguson*

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Key modifier(custom hotkey) does not trigger on a proper priority level. #44349