From: Julian Smart <julian@anthemion.co.uk>
Date: Wed, 8 Dec 2004 17:51:48 +0000 (+0000)
Subject: Fixed spurious selection of combobox text during resize
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/55e0d33c979274c996a6e9369e2a3d151a7b23b9

Fixed spurious selection of combobox text during resize


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

diff --git a/docs/changes.txt b/docs/changes.txt
index 261cd05d5a..a87632042a 100644
--- a/docs/changes.txt
+++ b/docs/changes.txt
@@ -57,6 +57,7 @@ wxMSW:
 
 - fixed enhanced metafiles loading from files (Andreas Goebel)
 - wxRadioButtons no longer have to be consecutive in a group
+- fixed spurious selection of combobox text during resize
 
 wxUniv:
 
diff --git a/src/msw/choice.cpp b/src/msw/choice.cpp
index 362f98ccb7..95632e7057 100644
--- a/src/msw/choice.cpp
+++ b/src/msw/choice.cpp
@@ -505,8 +505,24 @@ void wxChoice::DoSetSize(int x, int y,
         height += hItem*(nItems + 1);
     }
 
+    // To work around a Windows bug (see "Bug in Windows Combobox" thread in Google Groups)
+    // we have to reset the selection if it was accidentally selected in the size.
+    DWORD oldSelStart = 0;
+    DWORD oldSelEnd = 0;
+    DWORD newSelStart = 0;
+    DWORD newSelEnd = 0;
+
+    ::SendMessage(GetHwnd(), CB_GETEDITSEL, (WPARAM) & oldSelStart, (LPARAM) & oldSelEnd);
+
     wxControl::DoSetSize(x, y, width, height, sizeFlags);
 
+    ::SendMessage(GetHwnd(), CB_GETEDITSEL, (WPARAM) & newSelStart, (LPARAM) & newSelEnd);
+
+    if (oldSelStart != newSelStart || oldSelEnd != newSelEnd)
+    {
+        ::SendMessage(GetHwnd(), CB_SETEDITSEL, (WPARAM) 0, (LPARAM) MAKELPARAM(oldSelStart, oldSelEnd));
+    }
+
     // I'm commenting this out since the code appears to make choices
     // and comboxes too high when they have associated sizers. I'm sure this
     // is not the end of the story, which is why I'm leaving it #if'ed out for