From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Sat, 29 Nov 2008 14:49:47 +0000 (+0000)
Subject: don't reuse the double click handled in activation event handler for toggling the... 
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/0e2ad3233b80eca2dc008296a801f8f8fa18f15c?ds=sidebyside

don't reuse the double click handled in activation event handler for toggling the item (#9761)

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

diff --git a/src/msw/treectrl.cpp b/src/msw/treectrl.cpp
index 46a215d3ba..7aa4079cf6 100644
--- a/src/msw/treectrl.cpp
+++ b/src/msw/treectrl.cpp
@@ -2858,11 +2858,12 @@ bool wxTreeCtrl::MSWOnNotify(int idCtrl, WXLPARAM lParam, WXLPARAM *result)
     switch ( hdr->code )
     {
         case NM_DBLCLK:
-            // we translate NM_DBLCLK into ACTIVATED event, so don't interpret
-            // the return code of this event handler as the return value for
-            // NM_DBLCLK - otherwise, double clicking the item to toggle its
-            // expanded status would never work
-            *result = false;
+            // we translate NM_DBLCLK into ACTIVATED event and if the user
+            // handled the activation of the item we shouldn't proceed with
+            // also using the same double click for toggling the item expanded
+            // state -- but OTOH do let the user to expand/collapse the item by
+            // double clicking on it if the activation is not handled specially
+            *result = processed;
             break;
 
         case NM_RCLICK: