blender/blender
172
532
Fork 807
Code Issues 6.5k Pull Requests 973 Projects 21 Wiki Activity

Fix #108096: UI: copy-paste buttons can change source properties #108270

Merged
Philipp Oeser merged 3 commits from lichtwerk/blender:108096 into blender-v3.6-release 2023-06-07 11:19:10 +02:00
Conversation 6 Commits 3 Files Changed 1 +7 -1
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
source/blender/editors/interface/interface_handlers.cc
View File

@ -276,7 +276,13 @@ static void ui_selectcontext_apply(bContext *C,
const double value,
const double value_orig);
# define IS_ALLSELECT_EVENT(event) (((event)->modifier & KM_ALT) != 0)
/**
Campbell Barton commented 2023-06-06 15:39:14 +02:00
Outdated
Review
Copy Link

There should be an explanation for why only certain events are filtered (with a reference to #108096). Otherwise this seems fine.

There should be an explanation for why only certain events are filtered (with a reference to `#108096`). Otherwise this seems fine.
* Only respond to events which are expected to be used for multi button editing,
* e.g. ALT is also used for button array pasting, see #108096.
*/
# define IS_ALLSELECT_EVENT(event) \
(((event)->modifier & KM_ALT) != 0 && \
(ISMOUSE((event)->type) || ELEM((event)->type, EVT_RETKEY, EVT_PADENTER)))
/** just show a tinted color so users know its activated */
# define UI_BUT_IS_SELECT_CONTEXT UI_BUT_NODE_ACTIVE