From: Robert Roebling <robert@roebling.de>
Date: Sat, 12 Mar 2005 10:30:47 +0000 (+0000)
Subject:   There shouldn't be any need to draw the backgound
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/ab5ea030d80e237d54c2364e85ffd8a9d3644134

  There shouldn't be any need to draw the backgound
    under a sash.


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

diff --git a/src/gtk/renderer.cpp b/src/gtk/renderer.cpp
index e89a2beb74..4642766fb8 100644
--- a/src/gtk/renderer.cpp
+++ b/src/gtk/renderer.cpp
@@ -303,10 +303,13 @@ wxRendererGTK::DrawSplitterSash(wxWindow *win,
         erase_rect.width = w;
     }
 
-    // we must erase everything first, otherwise the garbage from the old sash
-    // is left when dragging it
-    //
-    // TODO: is this the right way to draw themed background?
+#if 0
+    // RR: After a correction to the orientation of the sash
+    //     this doesn't seem to be required anymore and it
+    //     seems to confuse some themes
+
+    // we must erase everything first, otherwise the garbage
+    // from the old sash is left when dragging it
     gtk_paint_flat_box
     (
         win->m_wxwindow->style,
@@ -315,12 +318,13 @@ wxRendererGTK::DrawSplitterSash(wxWindow *win,
         GTK_SHADOW_NONE,
         NULL,
         win->m_wxwindow,
-        (char *)"base", // const_cast
+        (char *)"viewportbin", // const_cast
         erase_rect.x,
         erase_rect.y,
         erase_rect.width,
         erase_rect.height
     );
+#endif
 
 #ifdef __WXGTK20__
     gtk_paint_handle
diff --git a/src/gtk1/renderer.cpp b/src/gtk1/renderer.cpp
index e89a2beb74..4642766fb8 100644
--- a/src/gtk1/renderer.cpp
+++ b/src/gtk1/renderer.cpp
@@ -303,10 +303,13 @@ wxRendererGTK::DrawSplitterSash(wxWindow *win,
         erase_rect.width = w;
     }
 
-    // we must erase everything first, otherwise the garbage from the old sash
-    // is left when dragging it
-    //
-    // TODO: is this the right way to draw themed background?
+#if 0
+    // RR: After a correction to the orientation of the sash
+    //     this doesn't seem to be required anymore and it
+    //     seems to confuse some themes
+
+    // we must erase everything first, otherwise the garbage
+    // from the old sash is left when dragging it
     gtk_paint_flat_box
     (
         win->m_wxwindow->style,
@@ -315,12 +318,13 @@ wxRendererGTK::DrawSplitterSash(wxWindow *win,
         GTK_SHADOW_NONE,
         NULL,
         win->m_wxwindow,
-        (char *)"base", // const_cast
+        (char *)"viewportbin", // const_cast
         erase_rect.x,
         erase_rect.y,
         erase_rect.width,
         erase_rect.height
     );
+#endif
 
 #ifdef __WXGTK20__
     gtk_paint_handle