Drag Treashold parameter - no reaction #61260

New Issue

Nanomanpro · 2019-02-06T22:45:01+01:00

Nanomanpro commented

2019-02-06 22:45:01 +01:00

System Information
Operating system: Win 7 64
Graphics card: NV GTX 770

Blender Version
Broken: 2.80, 8c87af7440, 06.02.2019

In my case to the "Front" view I assigned "2" key and to the "Back" view key "2+Drag".
as you can see in the video below, changing the "Drag Threshold" parameter does not bring any effect and blender still uses default value.
I also propose to increase the maximum value from 40 to 1024 px similar to "Tweak Threshold".
Thank you.

Video:
https://www.youtube.com/watch?v=hTS72Ckw5KM&feature=youtu.be

**System Information** Operating system: Win 7 64 Graphics card: NV GTX 770 **Blender Version** Broken: 2.80, **8c87af74409a**, 06.02.2019 In my case to the "Front" view I assigned "2" key and to the "Back" view key "2+Drag". as you can see in the video below, changing the "**Drag Threshold**" parameter does not bring any effect and blender still uses default value. I also propose to increase the maximum value from **40 to 1024** px similar to "Tweak Threshold". Thank you. Video: https://www.youtube.com/watch?v=hTS72Ckw5KM&feature=youtu.be

Nanomanpro commented

2019-02-06 22:45:01 +01:00

Added subscriber: @Nanomanpro

Campbell Barton commented

2019-02-07 00:11:30 +01:00

Added subscriber: @ideasman42

Campbell Barton commented

2019-02-07 00:11:30 +01:00

Note, it's using tweak threshold, would be careful changing this since we might not want to have different values for tweak & drag in the event handling (could cause bugs).

Brecht Van Lommel was assigned by Sebastian Parborg

2019-02-07 12:22:28 +01:00

Brecht Van Lommel commented

2019-02-07 12:25:59 +01:00

Changed status from 'Open' to: 'Archived'

Brecht Van Lommel closed this issue

2019-02-07 12:25:59 +01:00

Brecht Van Lommel commented

2019-02-07 12:26:00 +01:00

The naming might be poor, but the Drag Threshold tooltip says it applies to UI items.

I'm not sure what "2+Drag" is, but also in the keymap editor it is called Tweak, not Drag.

The naming might be poor, but the Drag Threshold tooltip says it applies to UI items. I'm not sure what "2+Drag" is, but also in the keymap editor it is called Tweak, not Drag.

Nanomanpro commented

2019-02-07 17:31:52 +01:00

Hello Brecht, and thank you very much for your explanation.
Names in this form may in fact be misleading, because, as you can see in the picture below "Click Drag" may suggest a relationship with a "Drag Treashold" parameter.
Maybe "Click Tweak" in keymap would be a better idea?

Hello Brecht, and thank you very much for your explanation. Names in this form may in fact be misleading, because, as you can see in the picture below "**Click Drag**" may suggest a relationship with a "**Drag Treashold**" parameter. Maybe "**Click Tweak**" in keymap would be a better idea? ![Click Drag.png](https://archive.blender.org/developer/F6540240/Click_Drag.png)

Brecht Van Lommel commented

2019-02-07 17:56:51 +01:00

Changed status from 'Archived' to: 'Open'

Brecht Van Lommel reopened this issue

2019-02-07 17:56:51 +01:00

Brecht Van Lommel commented

2019-02-07 17:56:51 +01:00

Ah wait, this is a new feature. Normally Tweak events are intended for this purpose.

@ideasman42, can we remove this CLICK_DRAG event? It's not used anymore in Blender code, tweak event handling is already complicated enough so would rather not have another similar way of doing it.

Ah wait, this is a new feature. Normally Tweak events are intended for this purpose. @ideasman42, can we remove this `CLICK_DRAG` event? It's not used anymore in Blender code, tweak event handling is already complicated enough so would rather not have another similar way of doing it.

Campbell Barton commented

2019-02-08 02:08:50 +01:00

Added subscriber: @pablovazquez

Campbell Barton commented

2019-02-08 02:08:50 +01:00

This was added to support keyboard shortcuts to be used for both pie menus and single key access.

For a while we had Tab-key access both pie menu and edit mode.

After testing we agreed this wasn't a good default because key-up events give the impression of lag, if you're not expecting it to behave this way.

But IIRC, from discussion at the time we thought this useful to be able to do, and people making their own keymaps customization could use it.

Without this there is no way to tell the difference between a click and a click-drag keyboard event:

This allows for some more flexible keymaps, OTOH, having click and click-drag on keyboard event's is not typically done.

@pablovazquez - what do you think? Is supporting different actions for tap/drag (typically pie menu) worth supporting?

EDIT posted on dev-talk: https://devtalk.blender.org/t/keyboard-tap-drag-dection-useful/5450

Even calling this tweak-threshold is misleading since it's used for click & double-click.

Having both is confusing, propose to use only one, see: D4322

Closing since this is more a UI/communication issue,
not a mistake in the code.

This was added to support keyboard shortcuts to be used for both pie menus and single key access. For a while we had Tab-key access both pie menu and edit mode. After testing we agreed this wasn't a good default because key-up events give the impression of lag, if you're not expecting it to behave this way. But IIRC, from discussion at the time we thought this useful to be able to do, and people making their own keymaps customization could use it. ---- Without this there is no way to tell the difference between a click and a click-drag keyboard event: This allows for some more flexible keymaps, OTOH, having click and click-drag on keyboard event's is not typically done. @pablovazquez - what do you think? Is supporting different actions for tap/drag (typically pie menu) worth supporting? ---- *EDIT* posted on dev-talk: https://devtalk.blender.org/t/keyboard-tap-drag-dection-useful/5450 Even calling this tweak-threshold is misleading since it's used for click & double-click. Having both is confusing, propose to use only one, see: [D4322](https://archive.blender.org/developer/D4322) Closing since this is more a UI/communication issue, not a mistake in the code.

Campbell Barton commented

2019-02-08 03:43:45 +01:00

Changed status from 'Open' to: 'Archived'

Campbell Barton closed this issue

2019-02-08 03:43:45 +01:00

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

Drag Treashold parameter - no reaction #61260