"Assign Shortcut" doesn't work for "View 2D Zoom" #79640

New Issue

Yevgeny Makarov · 2020-08-08T13:37:45+02:00

Yevgeny Makarov commented

2020-08-08 13:37:45 +02:00

System Information
Operating system: Darwin-19.5.0-x86_64-i386-64bit 64 Bits
Graphics card: AMD Radeon Pro 455 OpenGL Engine ATI Technologies Inc. 4.1 ATI-3.9.15

Blender Version
Broken: version: all

Short description of error
For "View2D" operators (view2d.zoom_in, view2d.zoom_border)
you can change existing shortcut with "Change Shortcut" but
there is no "Assign Shortcut" if a shortcut doesn't already exist.

For example Zoom In/Out in Node Editor or Zoom Border in Sequencer.

Exact steps for others to reproduce the error

Node Editor
View > Zoom In
Remove existing shortcuts
Try "Assign Shortcut" with right click

**System Information** Operating system: Darwin-19.5.0-x86_64-i386-64bit 64 Bits Graphics card: AMD Radeon Pro 455 OpenGL Engine ATI Technologies Inc. 4.1 ATI-3.9.15 **Blender Version** Broken: version: all **Short description of error** For "View2D" operators (view2d.zoom_in, view2d.zoom_border) you can change existing shortcut with "Change Shortcut" but there is no "Assign Shortcut" if a shortcut doesn't already exist. For example Zoom In/Out in Node Editor or Zoom Border in Sequencer. **Exact steps for others to reproduce the error** - Node Editor - View > Zoom In - Remove existing shortcuts - Try "Assign Shortcut" with right click

Yevgeny Makarov commented

2020-08-08 13:37:45 +02:00

Added subscriber: @jenkm

Philipp Oeser commented

2020-08-10 11:30:51 +02:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2020-08-10 11:30:51 +02:00

Changed status from 'Needs Triage' to: 'Needs User Info'

Philipp Oeser commented

2020-08-10 11:30:51 +02:00

Not sure I can reproduce.

This is set to a non-keyboard shortcut by default for me ({key Wheel In})
Change Shortcut is thus prevented (see tooltip), but you can remove the shortcut as you noted
This then gets {key Numpad +} assigned automatically here
I can then change the shortcut to something else without problems...

Just to make sure: is this different for you @jenkm ?

Not sure I can reproduce. - This is set to a non-keyboard shortcut by default for me ({key Wheel In}) - `Change Shortcut` is thus prevented (see tooltip), but you can remove the shortcut as you noted ![image.png](https://archive.blender.org/developer/F8765497/image.png) - This then gets {key Numpad +} assigned automatically here - I can then change the shortcut to something else without problems... ![image.png](https://archive.blender.org/developer/F8765500/image.png) Just to make sure: is this different for you @jenkm ?

Yevgeny Makarov commented

2020-08-10 11:36:22 +02:00

Remove {key Numpad +} as well.

You can change existing shortcut with "Change Shortcut" but
there is no "Assign Shortcut" if a shortcut doesn't already exist.

Remove {key Numpad +} as well. You can change existing shortcut with "Change Shortcut" but there is no "Assign Shortcut" if a shortcut doesn't already exist.

Philipp Oeser commented

2020-08-10 11:39:32 +02:00

Changed status from 'Needs User Info' to: 'Confirmed'

Philipp Oeser commented

2020-08-10 11:39:32 +02:00

Ah, get it now...

Philipp Oeser self-assigned this 2020-08-10 12:16:16 +02:00

Philipp Oeser commented

2020-08-10 12:20:08 +02:00

Note there is a comment at the start of WM_keymap_guess_opname

  /* Op types purposely skipped  for now:
   *     BRUSH_OT
   *     BOID_OT
   *     BUTTONS_OT
   *     CONSTRAINT_OT
   *     PAINT_OT
   *     ED_OT
   *     FLUID_OT
   *     TEXTURE_OT
   *     UI_OT
   *     VIEW2D_OT
   *     WORLD_OT
   */

So this is not really a bug (will change to TODO), the "View2D" is there, it is just not bound to a specific spacetype.
Guess this can be supported though, patch incoming...

Note there is a comment at the start of `WM_keymap_guess_opname` ``` /* Op types purposely skipped for now: * BRUSH_OT * BOID_OT * BUTTONS_OT * CONSTRAINT_OT * PAINT_OT * ED_OT * FLUID_OT * TEXTURE_OT * UI_OT * VIEW2D_OT * WORLD_OT */ ``` So this is not really a bug (will change to TODO), the "View2D" is there, it is just not bound to a specific spacetype. Guess this can be supported though, patch incoming...

blender-admin commented

2020-08-10 12:26:48 +02:00

This issue was referenced by dbf5bb7af2

This issue was referenced by dbf5bb7af2e45ea59c7a15346a5ee989828f2d07

Philipp Oeser commented

2020-08-11 22:18:07 +02:00

Changed status from 'Confirmed' to: 'Resolved'

Philipp Oeser closed this issue

2020-08-11 22:18:07 +02: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

"Assign Shortcut" doesn't work for "View 2D Zoom" #79640