Made wxWindow::Enable recursive in order to solve colouring problems; forced the colour

of a wxTextCtrl back to the custom colour in wxTextCtrl::Enable. This appears
to fix inconsistent colour problems.


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

src/gtk/textctrl.cpp

@@ -756,6 +756,20 @@ bool wxTextCtrl::Enable( bool enable )
     if (m_windowStyle & wxTE_MULTILINE)
     {
         gtk_text_set_editable( GTK_TEXT(m_text), enable );
+
+	// If we have a custom background colour, we use this colour in both
+	// disabled and enabled mode, or we end up with a different colour under the
+	// text.
+        wxColour oldColour = GetBackgroundColour();
+        if (oldColour.Ok())
+        {
+            // Need to set twice or it'll optimize the useful stuff out
+            if (oldColour == * wxWHITE)
+                SetBackgroundColour(*wxBLACK);
+            else
+                SetBackgroundColour(*wxWHITE);
+            SetBackgroundColour(oldColour);
+        }
     }
     else
     {

src/gtk/window.cpp

@@ -2948,6 +2948,16 @@ bool wxWindow::Enable( bool enable )
     if ( m_wxwindow )
         gtk_widget_set_sensitive( m_wxwindow, enable );
 
+    // Recurse, so that children have the opportunity to Do The Right Thing.
+    for ( wxWindowList::Node *node = GetChildren().GetFirst();
+          node;
+          node = node->GetNext() )
+    {
+        wxWindow *child = node->GetData();
+	if (!child->IsKindOf(CLASSINFO(wxDialog)) && !child->IsKindOf(CLASSINFO(wxFrame)))
+            child->Enable(enable);
+    }
+
     return TRUE;
 }
 

src/gtk1/textctrl.cpp

@@ -756,6 +756,20 @@ bool wxTextCtrl::Enable( bool enable )
     if (m_windowStyle & wxTE_MULTILINE)
     {
         gtk_text_set_editable( GTK_TEXT(m_text), enable );
+
+	// If we have a custom background colour, we use this colour in both
+	// disabled and enabled mode, or we end up with a different colour under the
+	// text.
+        wxColour oldColour = GetBackgroundColour();
+        if (oldColour.Ok())
+        {
+            // Need to set twice or it'll optimize the useful stuff out
+            if (oldColour == * wxWHITE)
+                SetBackgroundColour(*wxBLACK);
+            else
+                SetBackgroundColour(*wxWHITE);
+            SetBackgroundColour(oldColour);
+        }
     }
     else
     {

src/gtk1/window.cpp

@@ -2948,6 +2948,16 @@ bool wxWindow::Enable( bool enable )
     if ( m_wxwindow )
         gtk_widget_set_sensitive( m_wxwindow, enable );
 
+    // Recurse, so that children have the opportunity to Do The Right Thing.
+    for ( wxWindowList::Node *node = GetChildren().GetFirst();
+          node;
+          node = node->GetNext() )
+    {
+        wxWindow *child = node->GetData();
+	if (!child->IsKindOf(CLASSINFO(wxDialog)) && !child->IsKindOf(CLASSINFO(wxFrame)))
+            child->Enable(enable);
+    }
+
     return TRUE;
 }