Add back sculpt mode brush shortcuts #112898

Julien Kaspar · 2023-09-26T12:39:00+02:00

Julien Kaspar commented

2023-09-26 12:39:00 +02:00

Based on feedback from users this PR brings back the most commonly used brush shortcuts in sculpt mode. Essentially reverting some of the changes from #108712

There are multiple reasons behind this decision:

Anyone coming from a previous version will instinctively reassign the brush shortcuts to the same keys as before, introducing shortcut conflicts as a result. Having the shortcuts there by default makes this case less likely as it makes available brush shortcuts more obvious.
The default paint.brush_select operator is hard to set up manually and is offering behavior that isn't available with the wm.tool_set_by_id which is far easier to create from the UI
There are too many brushes available to know which ones are the most common ones

There are some additional changes:

S is now used for the Smooth brush.
Draw brush is now assigned to V to avoid conflicts on X.
Shift T is using the Scrape instead of Flatten brush.
C is using the Clay Strip instead of Clay brush.

Removing the brush shortcuts can be revisited later once the brush management is overhauled.

Based on feedback from users this PR brings back the most commonly used brush shortcuts in sculpt mode. Essentially reverting some of the changes from #108712 There are multiple reasons behind this decision: - Anyone coming from a previous version will instinctively reassign the brush shortcuts to the same keys as before, introducing shortcut conflicts as a result. Having the shortcuts there by default makes this case less likely as it makes available brush shortcuts more obvious. - The default `paint.brush_select` operator is hard to set up manually and is offering behavior that isn't available with the `wm.tool_set_by_id` which is far easier to create from the UI - There are too many brushes available to know which ones are the most common ones There are some additional changes: - `S` is now used for the Smooth brush. - Draw brush is now assigned to `V` to avoid conflicts on `X`. - `Shift T` is using the Scrape instead of Flatten brush. - `C` is using the Clay Strip instead of Clay brush. Removing the brush shortcuts can be revisited later once the brush management is overhauled.

Julien Kaspar added this to the 4.0 milestone 2023-09-26 12:39:00 +02:00

Julien Kaspar added the

Module

Sculpt, Paint & Texture

label 2023-09-26 12:39:00 +02:00

Julien Kaspar added 1 commit 2023-09-26 12:39:14 +02:00

6e493ac480 Added back most used brush shortcuts

Julien Kaspar requested review from Campbell Barton 2023-09-26 12:40:18 +02:00

Campbell Barton referenced this issue from a commit

2023-09-28 04:53:50 +02:00

Keymap: restore most used brush shortcuts

Campbell Barton manually merged commit fa27e6561e into main