[PATCH] [bug3009] Allow alt+space to highlight system menu

Jason Edmeades jason.edmeades at googlemail.com
Sun Aug 12 16:29:01 CDT 2007


The code already handle alt+space, but a check was added years ago to
prevent "alt up/down" bringing up the system menu on managed windows.
Testing, I think this was trying to avoid on a managed window with no
menu, the alt key alone popping up the system menu (eg the wine about
box on notepad).

This patch allows the alt+space combination to work as expected and
prevents the alt - alone combination as I believe the original patch
intended.
---
 dlls/user32/menu.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/dlls/user32/menu.c b/dlls/user32/menu.c
index 2f57779..2a2d3ab 100644
--- a/dlls/user32/menu.c
+++ b/dlls/user32/menu.c
@@ -3387,7 +3387,7 @@ void MENU_TrackKbdMenuBar( HWND hwnd, UINT wParam, WCHAR wChar)
 
     MENU_SelectItem( hwnd, hTrackMenu, uItem, TRUE, 0 );
 
-    if (wParam & HTSYSMENU)
+    if (wParam & HTSYSMENU && wChar != ' ')
     {
         /* prevent sysmenu activation for managed windows on Alt down/up */
         if (GetPropA( hwnd, "__wine_x11_managed" ))
-- 
1.5.0




More information about the wine-patches mailing list