[Mono-bugs] [Bug 77903][Maj] Changed - Focus handling is broken in some controls and Control.Select () doesn't work

[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=77903

--- shadow/77903	2006-05-14 14:33:01.000000000 -0400
+++ shadow/77903.tmp.27540	2006-05-23 08:29:55.000000000 -0400
@@ -54,6 +54,28 @@
 
 
 ------- Additional Comments From xenomorph2 at onlinehome.de  2006-05-14 14:33 -------
 Created an attachment (id=17015)
 Textbox focus test case
 
+
+------- Additional Comments From atsushi at ximian.com  2006-05-23 08:29 -------
+I have a pretty obvious fix for XplatUIX11.cs (no need to fix
+XplatUIOSX.cs as it already has this event order).
+
+However, see System.Windows.Forms/ChangeLog:
+
+2006-03-08  Peter Dennis Bartok  <pbartok at novell.com>
+
+	* ThemeWin32Classic.cs: TabPages cannot have focus. Determine
+	  when to draw focus rectangle by looking at parent focus and
+	  selected state instead. This fixes TabPages on Linux sometimes
+	  having none or multiple focus rectangles.
+	* XplatUIX11.cs (SetFocus): 
+	  - Don't set the focus if the same window already has focus
+	  - Use SendMessage instead of PostMessage (like it's Win32
+	    equivalent) and send the WM_SETFOCUS before the WM_KILLFOCUS
+	    to match MS behaviour
+	* TabControl.cs(SelectedIndex): Don't set Focus on TabPage, TabPages
+	  are not selectable.
+
+Peter, do you remember which control required this change?