[Bf-blender-cvs] [f46b384] master: Change to key modifier should trigger re-evaluation of drag-and-drop
Campbell Barton
noreply at git.blender.org
Tue Jun 3 15:03:48 CEST 2014
Commit: f46b384fc5e66949c0c2ce974d2842dd76ee6be8
Author: Campbell Barton
Date: Tue Jun 3 23:00:39 2014 +1000
https://developer.blender.org/rBf46b384fc5e66949c0c2ce974d2842dd76ee6be8
Change to key modifier should trigger re-evaluation of drag-and-drop
This change is part of a fix for T40435, but will postpone for now.
Original patch by Sergey Sharybin
===================================================================
M source/blender/editors/space_view3d/space_view3d.c
M source/blender/windowmanager/intern/wm_event_system.c
===================================================================
diff --git a/source/blender/editors/space_view3d/space_view3d.c b/source/blender/editors/space_view3d/space_view3d.c
index 4d958a5..b9d8c2a 100644
--- a/source/blender/editors/space_view3d/space_view3d.c
+++ b/source/blender/editors/space_view3d/space_view3d.c
@@ -593,8 +593,11 @@ static int view3d_ima_empty_drop_poll(bContext *C, wmDrag *drag, const wmEvent *
Base *base = ED_view3d_give_base_under_cursor(C, event->mval);
/* either holding and ctrl and no object, or dropping to empty */
- if ((event->ctrl && !base) || (base && base->object->type == OB_EMPTY))
+ if (((base == NULL) && event->ctrl) ||
+ ((base != NULL) && base->object->type == OB_EMPTY))
+ {
return view3d_ima_drop_poll(C, drag, event);
+ }
return 0;
}
diff --git a/source/blender/windowmanager/intern/wm_event_system.c b/source/blender/windowmanager/intern/wm_event_system.c
index 64289e0..175a8d2 100644
--- a/source/blender/windowmanager/intern/wm_event_system.c
+++ b/source/blender/windowmanager/intern/wm_event_system.c
@@ -2140,7 +2140,7 @@ static void wm_event_drag_test(wmWindowManager *wm, wmWindow *win, wmEvent *even
return;
}
- if (event->type == MOUSEMOVE)
+ if (event->type == MOUSEMOVE || ISKEYMODIFIER(event->type))
win->screen->do_draw_drag = true;
else if (event->type == ESCKEY) {
BLI_freelistN(&wm->drags);
@@ -2289,10 +2289,12 @@ void wm_event_do_handlers(bContext *C)
/* call even on non mouse events, since the */
wm_region_mouse_co(C, event);
- /* does polls for drop regions and checks uibuts */
- /* need to be here to make sure region context is true */
- if (ELEM(event->type, MOUSEMOVE, EVT_DROP)) {
- wm_drags_check_ops(C, event);
+ if (!BLI_listbase_is_empty(&wm->drags)) {
+ /* does polls for drop regions and checks uibuts */
+ /* need to be here to make sure region context is true */
+ if (ELEM(event->type, MOUSEMOVE, EVT_DROP) || ISKEYMODIFIER(event->type)) {
+ wm_drags_check_ops(C, event);
+ }
}
action |= wm_handlers_do(C, event, &ar->handlers);
More information about the Bf-blender-cvs
mailing list