[Mono-bugs] [Bug 78682][Nor] New - [PATCH] Menu keyboard navigation issues

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by atsushi at ximian.com.

http://bugzilla.ximian.com/show_bug.cgi?id=78682

--- shadow/78682	2006-06-21 07:32:29.000000000 -0400
+++ shadow/78682.tmp.968	2006-06-21 07:32:29.000000000 -0400
@@ -0,0 +1,79 @@
+Bug#: 78682
+Product: Mono: Class Libraries
+Version: 1.1
+OS: 
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: 
+Priority: Normal
+Component: Windows.Forms
+AssignedTo: peter at novonyx.com                            
+ReportedBy: atsushi at ximian.com               
+QAContact: mono-bugs at ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: [PATCH] Menu keyboard navigation issues
+
+Our Menu classes (MainMenu / ContextMenu) do not handle keyboard inputs
+while they are written to work.
+
+Simple repro:
+
+using System;
+using System.Windows.Forms;
+
+public class Test : Form
+{
+        public static void Main ()
+        {
+                Application.Run (new Test ());
+        }
+
+        public Test ()
+        {
+                MainMenu menu = new MainMenu ();
+                menu.MenuItems.Add (new MenuItem ("&File"));
+                menu.MenuItems.Add (new MenuItem ("&Edit"));
+                menu.MenuItems [0].MenuItems.Add (new MenuItem ("E&xit",
+                        delegate (object o, EventArgs e) { Application.Exit
+(); },
+                        Shortcut.CtrlX));
+                Menu = menu;
+
+                //Button b = new Button ();
+                //Controls.Add (b);
+
+                ContextMenu ctx = new ContextMenu ();
+                ctx.MenuItems.Add (new MenuItem ("Popup&1"));
+                ctx.MenuItems.Add (new MenuItem ("Popup&2"));
+                ctx.MenuItems [0].MenuItems.Add (new MenuItem ("Exi&t",
+                        delegate (object o, EventArgs e) { Application.Exit
+(); },
+                        Shortcut.CtrlT));
+                ContextMenu = ctx;
+        }
+}
+
+
+Steps to reproduce the problem:
+1. compile and run above.
+2. Push alt key.
+3. Push down key.
+
+Actual Results:
+
+nothing happens.
+
+Expected Results:
+
+It should expand "File" menu and draw "Exit" item as an active menu.
+
+Additional Information:
+
+A fix for the first issue is being attached, but I think there would be a
+reason for what I removed in the patch.
+
+Since there is a focus handling problem on Windows, the patch won't work on
+Windows.