[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [50807] trunk/blender/source/blender/ editors/interface: revert own commit r50501 ( have a better way to implement this same feature)

Campbell Barton ideasman42 at gmail.com
Sat Sep 22 09:22:43 CEST 2012


Revision: 50807
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=50807
Author:   campbellbarton
Date:     2012-09-22 07:22:42 +0000 (Sat, 22 Sep 2012)
Log Message:
-----------
revert own commit r50501 (have a better way to implement this same feature)

Revision Links:
--------------
    http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=50501

Modified Paths:
--------------
    trunk/blender/source/blender/editors/interface/interface_handlers.c
    trunk/blender/source/blender/editors/interface/interface_intern.h
    trunk/blender/source/blender/editors/interface/interface_regions.c

Modified: trunk/blender/source/blender/editors/interface/interface_handlers.c
===================================================================
--- trunk/blender/source/blender/editors/interface/interface_handlers.c	2012-09-22 04:57:55 UTC (rev 50806)
+++ trunk/blender/source/blender/editors/interface/interface_handlers.c	2012-09-22 07:22:42 UTC (rev 50807)
@@ -5279,12 +5279,7 @@
 
 		/* automatic open pulldown block timer */
 		if (ELEM3(but->type, BLOCK, PULLDOWN, ICONTEXTROW)) {
-			if ((data->used_mouse == TRUE) &&
-			    (data->autoopentimer == FALSE) &&
-			    /* don't popup the first time,
-			     * see description on this member for info */
-			    (but->block->auto_is_first_event == FALSE))
-			{
+			if (data->used_mouse && !data->autoopentimer) {
 				int time;
 
 				if (but->block->auto_open == TRUE) {  /* test for toolbox */
@@ -5304,8 +5299,6 @@
 					data->autoopentimer = WM_event_add_timer(data->wm, data->window, TIMER, 0.02 * (double)time);
 				}
 			}
-
-			but->block->auto_is_first_event = FALSE;
 		}
 	}
 	else {

Modified: trunk/blender/source/blender/editors/interface/interface_intern.h
===================================================================
--- trunk/blender/source/blender/editors/interface/interface_intern.h	2012-09-22 04:57:55 UTC (rev 50806)
+++ trunk/blender/source/blender/editors/interface/interface_intern.h	2012-09-22 07:22:42 UTC (rev 50807)
@@ -305,17 +305,7 @@
 	char direction;
 	char dt; /* drawtype: UI_EMBOSS, UI_EMBOSSN ... etc, copied to buttons */
 	char auto_open;
-
-	/* this setting is used so newly opened menu's dont popout the first item under the mouse,
-	 * the reasoning behind this is because of muscle memory for opening menus.
-	 *
-	 * Without this, the first time opening a Submenu and activating an item in it will be 2 steps,
-	 * but the second time the same item is accessed the menu memory would auto activate the
-	 * last used menu and the key intended to select that submenu ends up being passed into the submenu.
-	 * - Campbell
-	 */
-	char auto_is_first_event;
-	char _pad[6];
+	char _pad[7];
 	double auto_open_last;
 
 	const char *lockstr;

Modified: trunk/blender/source/blender/editors/interface/interface_regions.c
===================================================================
--- trunk/blender/source/blender/editors/interface/interface_regions.c	2012-09-22 04:57:55 UTC (rev 50806)
+++ trunk/blender/source/blender/editors/interface/interface_regions.c	2012-09-22 07:22:42 UTC (rev 50807)
@@ -2491,7 +2491,6 @@
 	pup->block = uiBeginBlock(C, NULL, __func__, UI_EMBOSSP);
 	pup->block->flag |= UI_BLOCK_POPUP_MEMORY;
 	pup->block->puphash = ui_popup_menu_hash(title);
-	pup->block->auto_is_first_event = TRUE;
 	pup->layout = uiBlockLayout(pup->block, UI_LAYOUT_VERTICAL, UI_LAYOUT_MENU, 0, 0, 200, 0, style);
 	uiLayoutSetOperatorContext(pup->layout, WM_OP_EXEC_REGION_WIN);
 




More information about the Bf-blender-cvs mailing list