From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Thu, 1 Nov 2007 20:26:11 +0000 (+0000)
Subject: set the focus to the tree before generating the selection change events to have a... 
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/cb776b667157c8d1e5887407d53723e2e747f8ec

set the focus to the tree before generating the selection change events to have a more expected (and compatible with other platforms) order of tree and focus events

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@49588 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/msw/treectrl.cpp b/src/msw/treectrl.cpp
index fe5e91ee1d..2f3b650043 100644
--- a/src/msw/treectrl.cpp
+++ b/src/msw/treectrl.cpp
@@ -2593,6 +2593,19 @@ bool wxTreeCtrl::MSWOnNotify(int idCtrl, WXLPARAM lParam, WXLPARAM *result)
                     event.m_itemOld = tv->itemOld.hItem;
                 }
             }
+
+            // we receive this message from WM_LBUTTONDOWN handler inside
+            // comctl32.dll and so before the click is passed to
+            // DefWindowProc() which sets the focus to the window which was
+            // clicked and this can lead to unexpected event sequences: for
+            // example, we may get a "selection change" event from the tree
+            // before getting a "kill focus" event for the text control which
+            // had the focus previously, thus breaking user code doing input
+            // validation
+            //
+            // to avoid such surprises, we force the generation of focus events
+            // now, before we generate the selection change ones
+            SetFocus();
             break;
 
             // instead of explicitly checking for _WIN32_IE, check if the