From 50a39165aaf08b38ab5fe252b0a56e49e8dadd1c Mon Sep 17 00:00:00 2001 From: Diederick Niehorster Date: Sat, 21 Jul 2012 03:11:57 +0000 Subject: [PATCH] fixes two cases of menu's not closing when they should: 1. open a menu in one of the freeglut windows, then activate another app (or the console window) by clicking on that. The freeglut menu doesn't close, and as it is topmost, it shines through everything. 2. open a menu in one of the freeglut windows, then click on the non-client area of another freeglut window (e.g. the title bar): menu doesn't close git-svn-id: svn+ssh://svn.code.sf.net/p/freeglut/code/trunk/freeglut/freeglut@1352 7f0cb862-5218-0410-a997-914c9d46530a --- src/mswin/fg_main_mswin.c | 22 +++++++++++++++++++++- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/src/mswin/fg_main_mswin.c b/src/mswin/fg_main_mswin.c index fea1505..c72e467 100644 --- a/src/mswin/fg_main_mswin.c +++ b/src/mswin/fg_main_mswin.c @@ -497,9 +497,29 @@ LRESULT CALLBACK fgPlatformWindowProc( HWND hWnd, UINT uMsg, WPARAM wParam, lRet = DefWindowProc( hWnd, uMsg, wParam, lParam ); INVOKE_WCB( *window, Entry, ( GLUT_LEFT ) ); + /* If this is a menu that lost focus, see if user either switched + application or FreeGLUT window (if one is running multiple + windows). If so, close menu that lost focus. + */ if( window->IsMenu && window->ActiveMenu && window->ActiveMenu->IsActive ) - fgUpdateMenuHighlight( window->ActiveMenu ); + { + SFG_Window* wnd = NULL; + HWND hwnd = GetForegroundWindow(); /* Get window with current focus */ + if (hwnd) + /* See if its one of our windows */ + wnd = fgWindowByHandle(hwnd); + + if (!hwnd || !wnd) + /* User switched to another application*/ + fgDeactivateMenu(window->ActiveMenu->ParentWindow); + else if ( + ( wnd->IsMenu && wnd->ActiveMenu->ParentWindow!=window->ActiveMenu->ParentWindow) || /* Make sure we don't kill the menu when trying to enter a submenu */ + (!wnd->IsMenu && wnd!=window->ActiveMenu->ParentWindow) + ) + /* User switched to another FreeGLUT window */ + fgDeactivateMenu(window->ActiveMenu->ParentWindow); + } break; -- 1.7.10.4