X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/23645bfa01b6450257f140a30c87565225b1ef4a..e044f6001e70d815f72d3911ce6bd01255847ecd:/src/univ/notebook.cpp?ds=sidebyside

diff --git a/src/univ/notebook.cpp b/src/univ/notebook.cpp
index e1e2ff47ef..17d7aefce0 100644
--- a/src/univ/notebook.cpp
+++ b/src/univ/notebook.cpp
@@ -212,15 +212,21 @@ int wxNotebook::SetSelection(int nPage)
         return m_sel;
     }
 
-    if ( m_sel != INVALID_PAGE )
+    // we need to change m_sel first, before calling RefreshTab() below as
+    // otherwise the previously selected tab wouldn't be redrawn properly under
+    // wxGTK which calls Refresh() immediately and not during the next event
+    // loop iteration as wxMSW does and as it should
+    size_t selOld = m_sel;
+
+    m_sel = nPage;
+
+    if ( selOld != INVALID_PAGE )
     {
-        RefreshTab(m_sel);
+        RefreshTab(selOld, TRUE /* this tab was selected */);
 
-        m_pages[m_sel]->Hide();
+        m_pages[selOld]->Hide();
     }
 
-    m_sel = nPage;
-
     if ( m_sel != INVALID_PAGE ) // this is impossible - but test nevertheless
     {
         if ( HasSpinBtn() )
@@ -421,12 +427,12 @@ void wxNotebook::RefreshCurrent()
     }
 }
 
-void wxNotebook::RefreshTab(int page)
+void wxNotebook::RefreshTab(int page, bool forceSelected)
 {
     wxCHECK_RET( IS_VALID_PAGE(page), _T("invalid notebook page") );
 
     wxRect rect = GetTabRect(page);
-    if ( (size_t)page == m_sel )
+    if ( forceSelected || ((size_t)page == m_sel) )
     {
         const wxSize indent = GetRenderer()->GetTabIndent();
         rect.Inflate(indent.x, indent.y);