griefith/test2
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix T82800: Changing settings from a pie menu doesn't add undo steps

Browse Source 
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.
This commit is contained in:
Campbell Barton
2020-11-18 16:30:32 +11:00
parent 0688309988
commit dfa2b0e1c9
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
source/blender/editors/interface/interface_handlers.c
View File

@@ -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;
}