[Mono-bugs] [Bug 82160][Nor] Changed - NotifyIcon: Context menu is not closed when focus is lost

[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 georgegiolfan at yahoo.com.

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

--- shadow/82160	2007-07-22 10:47:59.000000000 -0400
+++ shadow/82160.tmp.22353	2007-08-30 16:39:23.000000000 -0400
@@ -10,13 +10,12 @@
 Component: Windows.Forms
 AssignedTo: everaldo at simios.org                            
 ReportedBy: gert.driesen at pandora.be               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
 URL: 
-Cc: 
 Summary: NotifyIcon: Context menu is not closed when focus is lost
 
 On Mono/Windows, the context menu of the NotifyIcon is not closed when you 
 click on another part of the notification area.
 
 This works fine on Linux though.
@@ -29,6 +28,22 @@
 2. follow the on-screen instructions.
 
 ------- Additional Comments From gert.driesen at pandora.be  2007-07-22 05:10 -------
 Created an attachment (id=19597)
 Repro
 
+
+------- Additional Comments From georgegiolfan at yahoo.com  2007-08-30 16:39 -------
+About the Windows part of the bug:
+I think the reason the menu is not closed is that in MWF a context 
+menu is closed when its parent window receives a button down message 
+outside the area of the menu (in Form.WmButtonDown). The parent 
+window (usually) gets this message because it captures the mouse 
+before the menu is shown (in Form.ActiveTracker).
+In the case of the notification, something must go wrong. The parent 
+window does not seem to receive the button down message so the menu 
+is not closed. The documentation says that only the foreground window 
+can capture the mouse. Maybe clicking on a notification icon does not 
+make put its window into the foreground so the capture does not work.
+I have not extensively tested the following patch. It seems to fix 
+the bug but I am not sure that this approach (capturing the mouse) is 
+good.