Move Channels operator moves Grease Pencil Layers, causing keymap conflict #72897

New Issue

Bun ny · 2020-01-04T13:35:41+01:00

Bun ny commented

2020-01-04 13:35:41 +01:00

System Information
Operating system: Windows 7 x64
Graphics card: NVIDIA GeForce GTX 580

Blender Version
Broken: 2.82 a854840e76

Short description of error
The Move Channels operator moves Grease Pencil layers, in reverse order and without updating the Layers panel in the Properties Editor. The dedicated Move Grease Pencil Layer operator works as expected. Since there isn't a way to assign different shortcuts dependent on the editing context, there's currently no way to assign PgUp and PgDn to both operators. It seems that if Move Channels didn't affect Grease Pencil layers, there would be no conflict assigning the same keys to both operators.

Exact steps for others to reproduce the error
Create Grease Pencil object, add multiple layers, use the Move Channels operator (PgUp, PgDn)

**System Information** Operating system: Windows 7 x64 Graphics card: NVIDIA GeForce GTX 580 **Blender Version** Broken: 2.82 a854840e76ae **Short description of error** The Move Channels operator moves Grease Pencil layers, in reverse order and without updating the Layers panel in the Properties Editor. The dedicated Move Grease Pencil Layer operator works as expected. Since there isn't a way to assign different shortcuts dependent on the editing context, there's currently no way to assign PgUp and PgDn to both operators. It seems that if Move Channels didn't affect Grease Pencil layers, there would be no conflict assigning the same keys to both operators. **Exact steps for others to reproduce the error** Create Grease Pencil object, add multiple layers, use the Move Channels operator (PgUp, PgDn)

Bun ny commented

2020-01-04 13:35:41 +01:00

Added subscriber: @bunny

Antonio Vazquez commented

2020-01-04 13:40:10 +01:00

Added subscriber: @antoniov

Antonio Vazquez commented

2020-01-04 13:40:10 +01:00

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

Antonio Vazquez commented

2020-01-04 13:40:10 +01:00

I'm not sure what you mean, but in the last version there are buttons in the Dopesheet header to move UP/DW the layer.

Can you explain more about your problem,

Note: Grease Pencil Layers are not "real" channels, so maybe the problem is this operator must be hidden in GP mode.

I'm not sure what you mean, but in the last version there are buttons in the Dopesheet header to move UP/DW the layer. ![image.png](https://archive.blender.org/developer/F8259567/image.png) Can you explain more about your problem, Note: Grease Pencil Layers are not "real" channels, so maybe the problem is this operator must be hidden in GP mode.

Bun ny commented

2020-01-04 20:50:59 +01:00

There's two operators that both move gpencil layers: the newer bpy.ops.gpencil.layer_move, which works perfectly (only moves gpencil layers), and the older operator bpy.ops.anim.channels_move, which is meant for animation channels but still moves gpencil layers since the 2.7 series (moves both animation channels and gpencil layers).

User expects that anim.channels_move should move only animation channels, and not gpencil layers.

Or, that anim.channels_move would move gpencil layers and update the layers panel in the Properties Editor exactly the same way as gpencil.layer_move does. Currently anim.channels_move(direction='UP') moves gpencil layer down, since gpencil layer order is reversed, and user has to move mouse cursor over to the properties editor to force the layers panel to update.

There's two operators that both move gpencil layers: the newer bpy.ops.gpencil.layer_move, which works perfectly (only moves gpencil layers), and the older operator bpy.ops.anim.channels_move, which is meant for animation channels but still moves gpencil layers since the 2.7 series (moves both animation channels and gpencil layers). User expects that anim.channels_move should move only animation channels, and not gpencil layers. Or, that anim.channels_move would move gpencil layers and update the layers panel in the Properties Editor exactly the same way as gpencil.layer_move does. Currently anim.channels_move(direction='UP') moves gpencil layer down, since gpencil layer order is reversed, and user has to move mouse cursor over to the properties editor to force the layers panel to update.

blender-admin commented

2020-01-04 22:02:03 +01:00

This issue was referenced by 5c66739b78

This issue was referenced by 5c66739b782e991034c872c4f7175093e98c9982

Antonio Vazquez commented

2020-01-04 22:02:52 +01:00

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

Antonio Vazquez closed this issue

2020-01-04 22:02:52 +01:00

Antonio Vazquez self-assigned this 2020-01-04 22:02:52 +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

Move Channels operator moves Grease Pencil Layers, causing keymap conflict #72897