From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Tue, 27 Oct 2009 16:44:16 +0000 (+0000)
Subject: No changes, just add a comment to the button focus handling code.
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/eb74a51f7ed2a76dbb443e48727a410a0c81247a?ds=sidebyside

No changes, just add a comment to the button focus handling code.

The workarounds for the button focus issues should become unnecessary if we
switch to setting the focus correctly as explained in the linked post.

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

diff --git a/src/msw/button.cpp b/src/msw/button.cpp
index c81472bebc..6be8b813b3 100644
--- a/src/msw/button.cpp
+++ b/src/msw/button.cpp
@@ -677,6 +677,14 @@ wxSize wxButtonBase::GetDefaultSize()
 // ----------------------------------------------------------------------------
 
 /*
+   The comment below and all this code is probably due to not using WM_NEXTDLGCTL
+   message when changing focus (but just SetFocus() which is not enough), see
+   http://blogs.msdn.com/oldnewthing/archive/2004/08/02/205624.aspx for the
+   full explanation.
+
+   TODO: Do use WM_NEXTDLGCTL and get rid of all this code.
+
+
    "Everything you ever wanted to know about the default buttons" or "Why do we
    have to do all this?"