Fix wxGenericListCtrl best size calculation in report view.

[wxWidgets.git] / src / common / containr.cpp

diff --git a/src/common/containr.cpp b/src/common/containr.cpp
index 2fda844c9620a456882a0212a4857535c6624edb..8490cc5b9f07a6bd3e5f89af2dc900c6cf692e8b 100644 (file)
--- a/src/common/containr.cpp
+++ b/src/common/containr.cpp
@@ -653,17 +653,27 @@ bool wxSetFocusToChild(wxWindow *win, wxWindow **childLastFocused)
         // It might happen that the window got reparented
         if ( (*childLastFocused)->GetParent() == win )
         {
         // It might happen that the window got reparented
         if ( (*childLastFocused)->GetParent() == win )
         {

-            // And it also could have become hidden in the meanwhile, in this
-            // case focus its parent instead.
-            while ( !(*childLastFocused)->IsShown() )
+            // And it also could have become hidden in the meanwhile
+            // We want to focus on the deepest widget visible
+            wxWindow *deepestVisibleWindow = NULL;
+
+            while ( *childLastFocused )

             {
             {

+                if ( (*childLastFocused)->IsShown() )
+                {
+                    if ( !deepestVisibleWindow )
+                        deepestVisibleWindow = *childLastFocused;
+                }
+                else
+                    deepestVisibleWindow = NULL;
+

                 *childLastFocused = (*childLastFocused)->GetParent();
                 *childLastFocused = (*childLastFocused)->GetParent();

-                if ( !*childLastFocused )
-                    break;

             }
 
             }
 

-            if ( *childLastFocused )
+            if ( deepestVisibleWindow )

             {
             {

+                *childLastFocused = deepestVisibleWindow;
+

                 wxLogTrace(TRACE_FOCUS,
                            wxT("SetFocusToChild() => last child (0x%p)."),
                            (*childLastFocused)->GetHandle());
                 wxLogTrace(TRACE_FOCUS,
                            wxT("SetFocusToChild() => last child (0x%p)."),
                            (*childLastFocused)->GetHandle());