[Bf-blender-cvs] [1ac7a840f61] blender2.8: Fix T58594: Modal circle select can't select & de-select
Campbell Barton
noreply at git.blender.org
Mon Dec 3 09:32:46 CET 2018
Commit: 1ac7a840f61fb7c76d605ce1b9051330098bc113
Author: Campbell Barton
Date: Mon Dec 3 19:29:06 2018 +1100
Branches: blender2.8
https://developer.blender.org/rB1ac7a840f61fb7c76d605ce1b9051330098bc113
Fix T58594: Modal circle select can't select & de-select
===================================================================
M source/blender/windowmanager/intern/wm_gesture_ops.c
===================================================================
diff --git a/source/blender/windowmanager/intern/wm_gesture_ops.c b/source/blender/windowmanager/intern/wm_gesture_ops.c
index b29da15a334..faaf5f61421 100644
--- a/source/blender/windowmanager/intern/wm_gesture_ops.c
+++ b/source/blender/windowmanager/intern/wm_gesture_ops.c
@@ -79,7 +79,7 @@ static void gesture_modal_end(bContext *C, wmOperator *op)
}
}
-static void gesture_modal_state_to_operator(wmOperator *op, int modal_state)
+static void gesture_modal_state_to_operator(wmOperator *op, int modal_state, bool check_is_set)
{
PropertyRNA *prop;
@@ -87,12 +87,12 @@ static void gesture_modal_state_to_operator(wmOperator *op, int modal_state)
case GESTURE_MODAL_SELECT:
case GESTURE_MODAL_DESELECT:
if ((prop = RNA_struct_find_property(op->ptr, "deselect"))) {
- if (!RNA_property_is_set(op->ptr, prop)) {
+ if (!check_is_set || !RNA_property_is_set(op->ptr, prop)) {
RNA_property_boolean_set(op->ptr, prop, (modal_state == GESTURE_MODAL_DESELECT));
}
}
if ((prop = RNA_struct_find_property(op->ptr, "mode"))) {
- if (!RNA_property_is_set(op->ptr, prop)) {
+ if (!check_is_set || !RNA_property_is_set(op->ptr, prop)) {
RNA_property_enum_set(op->ptr, prop, (modal_state == GESTURE_MODAL_DESELECT) ? SEL_OP_SUB : SEL_OP_ADD);
}
}
@@ -100,7 +100,7 @@ static void gesture_modal_state_to_operator(wmOperator *op, int modal_state)
case GESTURE_MODAL_IN:
case GESTURE_MODAL_OUT:
if ((prop = RNA_struct_find_property(op->ptr, "zoom_out"))) {
- if (!RNA_property_is_set(op->ptr, prop)) {
+ if (!check_is_set || !RNA_property_is_set(op->ptr, prop)) {
RNA_property_boolean_set(op->ptr, prop, (modal_state == GESTURE_MODAL_OUT));
}
}
@@ -171,7 +171,7 @@ static bool gesture_box_apply(bContext *C, wmOperator *op)
return 0;
}
- gesture_modal_state_to_operator(op, gesture->modal_state);
+ gesture_modal_state_to_operator(op, gesture->modal_state, true);
retval = op->type->exec(C, op);
OPERATOR_RETVAL_CHECK(retval);
@@ -331,7 +331,9 @@ static void gesture_circle_apply(bContext *C, wmOperator *op)
RNA_int_set(op->ptr, "y", rect->ymin);
RNA_int_set(op->ptr, "radius", rect->xmax);
- gesture_modal_state_to_operator(op, gesture->modal_state);
+ /* When 'wait_for_input' is false, use properties to get the selection state.
+ * typically tool settings. This is done so executing as a mode can select & de-select, see: T58594. */
+ gesture_modal_state_to_operator(op, gesture->modal_state, !gesture->wait_for_input);
if (op->type->exec) {
int retval;
More information about the Bf-blender-cvs
mailing list