diff options
author | Campbell Barton <ideasman42@gmail.com> | 2020-11-18 08:30:32 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2020-11-18 08:40:48 +0300 |
commit | dfa2b0e1c95bf72d58bbff9fb5c4045521af04d6 (patch) | |
tree | 06a49e23f39fb9adb8ac8cc21e1fb3dfda9597a2 /source/blender/editors/interface | |
parent | 0688309988e546382748b9e755d84ae8a5059192 (diff) |
Fix T82800: Changing settings from a pie menu doesn't add undo steps
Pie menu had inconsistent behavior for dragging & releasing the key
compared to clicking on the button.
This was caused by the `onfree` argument being set to true,
preventing the button from running callbacks such as setting
up undo data & auto-keyframe
This argument should only be used when freeing the button,
set this to false as is done for regular menus.
Diffstat (limited to 'source/blender/editors/interface')
-rw-r--r-- | source/blender/editors/interface/interface_handlers.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/source/blender/editors/interface/interface_handlers.c b/source/blender/editors/interface/interface_handlers.c index 7e9461e7751..5be335f0e6e 100644 --- a/source/blender/editors/interface/interface_handlers.c +++ b/source/blender/editors/interface/interface_handlers.c @@ -10211,8 +10211,7 @@ static int ui_but_pie_menu_apply(bContext *C, menu->menuretval = UI_RETURN_CANCEL; } else { - ui_apply_but(C, but->block, but, but->active, false); - button_activate_exit((bContext *)C, but, but->active, false, true); + button_activate_exit((bContext *)C, but, but->active, false, false); menu->menuretval = UI_RETURN_OK; } |