From 4d11369db6f07e78ccc6bc4264867fee2548f4ab Mon Sep 17 00:00:00 2001
From: Jaakko Salli <jaakko.salli@dnainternet.net>
Date: Sat, 18 Oct 2008 16:09:48 +0000
Subject: [PATCH] Do not skip key events so eagerly

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@56426 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---
 src/propgrid/propgrid.cpp | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/src/propgrid/propgrid.cpp b/src/propgrid/propgrid.cpp
index cf1c5b4e9e..37cb0846fb 100644
--- a/src/propgrid/propgrid.cpp
+++ b/src/propgrid/propgrid.cpp
@@ -5013,11 +5013,14 @@ void wxPropertyGrid::HandleKeyEvent( wxKeyEvent &event, bool fromChild )
         return;
     }
 
-    event.Skip();
-
     // Except for TAB and ESC, handle child control events in child control
     if ( fromChild )
+    {
+        event.Skip();
         return;
+    }
+
+    bool wasHandled = false;
 
     if ( m_selected )
     {
@@ -5037,16 +5040,16 @@ void wxPropertyGrid::HandleKeyEvent( wxKeyEvent &event, bool fromChild )
             if ( action == wxPG_ACTION_COLLAPSE_PROPERTY || secondAction == wxPG_ACTION_COLLAPSE_PROPERTY )
             {
                 if ( (m_windowStyle & wxPG_HIDE_MARGIN) || Collapse(p) )
-                    keycode = 0;
+                    wasHandled = true;
             }
             else if ( action == wxPG_ACTION_EXPAND_PROPERTY || secondAction == wxPG_ACTION_EXPAND_PROPERTY )
             {
                 if ( (m_windowStyle & wxPG_HIDE_MARGIN) || Expand(p) )
-                    keycode = 0;
+                    wasHandled = true;
             }
         }
 
-        if ( keycode )
+        if ( !wasHandled )
         {
             if ( action == wxPG_ACTION_PREV_PROPERTY || secondAction == wxPG_ACTION_PREV_PROPERTY )
             {
@@ -5063,6 +5066,7 @@ void wxPropertyGrid::HandleKeyEvent( wxKeyEvent &event, bool fromChild )
             p = wxPropertyGridIterator::OneStep( m_pState, wxPG_ITERATE_VISIBLE, p, selectDir );
             if ( p )
                 DoSelectProperty(p);
+            wasHandled = true;
         }
     }
     else
@@ -5073,8 +5077,12 @@ void wxPropertyGrid::HandleKeyEvent( wxKeyEvent &event, bool fromChild )
         {
             wxPGProperty* p = wxPropertyGridInterface::GetFirst();
             if ( p ) DoSelectProperty(p);
+            wasHandled = true;
         }
     }
+
+    if ( !wasHandled )
+        event.Skip();
 }
 
 // -----------------------------------------------------------------------
-- 
2.45.2