Resolve "Shift R" shortcut conflict #99607

New Issue

Julien Kaspar · 2022-07-11T12:54:01+02:00

Julien Kaspar commented

2022-07-11 12:54:01 +02:00

(NOTE) This task is part of the community & sub tasks of the workboard. Anyone is free to pick up this task and contribute. For any questions or needed reviewers, please tag @JosephEagar and @JulienKaspar or use blender.chat.

Issue

For a long time there has been a minor shortcut conflict in Sculpt Mode with Shift R.
The shortcut is assigned to both "Repeat Last" & "Edit Voxel Size".

In the near future we plan to add more operators and redo support to sculpt mode.
The new curve hair sytsem and their sculpting toolset will also rely more heavily on operators.
To prevent the shortcut conflict from spreading to new areas of Blender and blocking common operations, we need to solve it now.

Solution

We agreed to universally use the shortcut R for defining resolutions and density for voxels, dyntopo and hair curves.
The existing shortcut for the "Angle Control" menu will be replaced by this so there is no conflict.

(NOTE) This task is part of the community & sub tasks of the workboard. Anyone is free to pick up this task and contribute. For any questions or needed reviewers, please tag @JosephEagar and @JulienKaspar or use [blender.chat](https://blender.chat/channel/sculpt-paint-texture-module). # Issue For a long time there has been a minor shortcut conflict in Sculpt Mode with `Shift R`. The shortcut is assigned to both "Repeat Last" & "Edit Voxel Size". In the near future we plan to add more operators and redo support to sculpt mode. The new curve hair sytsem and their sculpting toolset will also rely more heavily on operators. To prevent the shortcut conflict from spreading to new areas of Blender and blocking common operations, we need to solve it now. # Solution We agreed to universally use the shortcut `R` for defining resolutions and density for voxels, dyntopo and hair curves. The existing shortcut for the "Angle Control" menu will be replaced by this so there is no conflict.

Julien Kaspar commented

2022-07-11 12:54:01 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'

Julien Kaspar commented

2022-07-11 12:54:01 +02:00

Added subscribers: @JulienKaspar, @HooglyBoogly, @JosephEagar

Ramil Roosileht commented

2022-07-11 19:39:17 +02:00

Added subscriber: @Limarest

Ramil Roosileht commented

2022-07-11 19:39:17 +02:00

@JulienKaspar

Honestly I dislike the Alt R solution because it makes it impossible to use "Edit Voxel Size" and "Voxel Remesh" [CTRL R] with the same fingers, making you move the hand away.
Also consider that Dyntopo has a Shift D "Edit Dyntopo Size" widget, which ideally should have the same logic as the voxel size edit one.

@JulienKaspar Honestly I dislike the Alt R solution because it makes it impossible to use "Edit Voxel Size" and "Voxel Remesh" [CTRL R] with the same fingers, making you move the hand away. Also consider that Dyntopo has a Shift D "Edit Dyntopo Size" widget, which ideally should have the same logic as the voxel size edit one.

Julien Kaspar commented

2022-07-11 19:44:15 +02:00

Changed status from 'Confirmed' to: 'Needs Developer To Reproduce'

Julien Kaspar commented

2022-07-14 12:09:19 +02:00

From recent module meeting:

This should be prioritised since the hair curves project is mimicking shortcut conflicts in sculpt mode. This is causing the shortcut conflict to spread.
Alt R was the first proposal but is not convenient to press when pressing Ctrl R just before.
Ctrl Shift R is a better solution but is making common shortcut too complex.
Another solution could be a new shortcut for Repeat Last?
But only Ctrl Alt R would be available and Repeat Last is used a lot to be made more inconvenient.
We need to investigate this further.

Proposal A

If we adjust the keybindings for the voxel remesher, this should also affect the dyntopo shortcuts (for consistency). So we could go for:
Edit Voxel SIze = Ctrl R
Voxel Remesh = Ctrl Shift R
Edit Dyntopo Size = Ctrl D
Enable Dyntopo = Ctrl Shift D

The same fingers would then be used for all of them.
Other modes can then reuse the shortcut Ctrl R for changing a relevant global size.
But this makes the shortcuts more complex for operations that are done very regularly.

Proposal B

There are also other options we didn't discuss yet, like using the same shortcut for resizing the voxel size and dyntopo size.
Because as of right now it's impossible to access voxel remesher shortcuts while dyntopo is used.
So in that case we could go for:
Edit Voxel Size = R
Voxel Remesh = Ctrl R

And when dyntopo is enabled:
Edit Dyntopo Size = R
Dyntopo Flood Fill = Ctrl R

The shortcut R can then be reused in curve sculpt mode.

From recent module meeting: > This should be prioritised since the hair curves project is mimicking shortcut conflicts in sculpt mode. This is causing the shortcut conflict to spread. > Alt R was the first proposal but is not convenient to press when pressing Ctrl R just before. > Ctrl Shift R is a better solution but is making common shortcut too complex. > Another solution could be a new shortcut for Repeat Last? > But only Ctrl Alt R would be available and Repeat Last is used a lot to be made more inconvenient. > We need to investigate this further. ### Proposal A If we adjust the keybindings for the voxel remesher, this should also affect the dyntopo shortcuts (for consistency). So we could go for: Edit Voxel SIze = `Ctrl R` Voxel Remesh = `Ctrl Shift R` Edit Dyntopo Size = `Ctrl D` Enable Dyntopo = `Ctrl Shift D` The same fingers would then be used for all of them. Other modes can then reuse the shortcut `Ctrl R` for changing a relevant global size. But this makes the shortcuts more complex for operations that are done very regularly. ### Proposal B There are also other options we didn't discuss yet, like using the same shortcut for resizing the voxel size **and** dyntopo size. Because as of right now it's impossible to access voxel remesher shortcuts while dyntopo is used. So in that case we could go for: Edit Voxel Size = `R` Voxel Remesh = `Ctrl R` And when dyntopo is enabled: Edit Dyntopo Size = `R` Dyntopo Flood Fill = `Ctrl R` The shortcut `R` can then be reused in curve sculpt mode.

Julien Kaspar commented

2022-11-11 11:53:23 +01:00

Changed status from 'Needs Developer To Reproduce' to: 'Confirmed'