Page MenuHome
Paste P327

Possible fix for T47532
ActivePublic
Actions

Authored by Julian Eisel (Severin) on Feb 23 2016, 4:27 PM.
diff --git a/source/blender/editors/interface/interface_handlers.c b/source/blender/editors/interface/interface_handlers.c
index 43f1801..37a00ea 100644
--- a/source/blender/editors/interface/interface_handlers.c
+++ b/source/blender/editors/interface/interface_handlers.c
@@ -9934,10 +9934,14 @@ static int ui_popup_handler(bContext *C, const wmEvent *event, void *userdata)
menu_region = CTX_wm_menu(C);
CTX_wm_menu_set(C, menu->region);
- if (event->type == EVT_DROP) {
- /* if we're handling drop event we'll want it to be handled by popup callee as well,
- * so it'll be possible to perform such operations as opening .blend files by dropping
- * them into blender even if there's opened popup like splash screen (sergey)
+ if (event->type == EVT_DROP || event->val == KM_DBL_CLICK) {
+ /* EVT_DROP:
+ * If we're handling drop event we'll want it to be handled by popup callee as well,
+ * so it'll be possible to perform such operations as opening .blend files by dropping
+ * them into blender, even if there's opened popup like splash screen (sergey).
+ * KM_DBL_CLICK:
+ * Continue in case of double click so wm_handlers_do calls handler again with KM_PRESS
+ * event. This is needed to ensure correct button handling for fast clicking (T47532).
*/
retval = WM_UI_HANDLER_CONTINUE;

Event Timeline

Julian Eisel (Severin) edited the content of this paste. (Show Details)Feb 23 2016, 4:27 PM
Julian Eisel (Severin) changed the title of this paste from untitled to Possible fix for T47532.
Julian Eisel (Severin) updated the paste's language from autodetect to diff.
Julian Eisel (Severin) mentioned this in T47532: F6-menu drops mouseclicks.