Tool settings changes in undo history #62360

Closed
opened 2019-03-08 14:59:08 +01:00 by Julien Kaspar · 2 comments
Member

Broken: cf09a2443cef

Currently when changing the brush size, strength and many more tool specific settings it is added to the undo history. I think there is a bigger design discussion behind this if this should even be allowed but the bug that is tied to this is that these steps, when undone are not actually undone.
They are just extra undo steps that do nothing. And since when sculpting, painting or drawing it is very common to very often change the size & strength of the current brush, the undo history gets flooded with undo steps that don't undo anything.

To add my take on how this should be fixed: Tool settings should never be included in the default undo history, just like interface changes have no undo steps.
My suggestion would be to perhaps add a second undo history for settings and interface changes, but only if these should still be undo-able.

Broken: cf09a2443cef Currently when changing the brush size, strength and many more tool specific settings it is added to the undo history. I think there is a bigger design discussion behind this if this should even be allowed but the bug that is tied to this is that these steps, when undone are not actually undone. They are just extra undo steps that do nothing. And since when sculpting, painting or drawing it is very common to very often change the size & strength of the current brush, the undo history gets flooded with undo steps that don't undo anything. To add my take on how this should be fixed: Tool settings should never be included in the default undo history, just like interface changes have no undo steps. My suggestion would be to perhaps add a second undo history for settings and interface changes, but only if these should still be undo-able.
Author
Member

Added subscriber: @JulienKaspar

Added subscriber: @JulienKaspar

Closed as duplicate of blender/blender#61948

Closed as duplicate of blender/blender#61948
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: studio/blender-studio#62360
No description provided.