]> git.saurik.com Git - wxWidgets.git/blobdiff - src/generic/tabg.cpp
fixed unwanted owner data accesses in virtual list ctrl
[wxWidgets.git] / src / generic / tabg.cpp
index 5adc584c4c8ba48386dcf83d19bc405d82636138..bd0ba3d2ebdfa57d51f78fa6dce76c0abdc9d5d1 100644 (file)
@@ -6,7 +6,7 @@
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Copyright:   (c)
-// Licence:    wxWindows licence
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
@@ -20,6 +20,8 @@
 #pragma hdrstop
 #endif
 
+#if wxUSE_TAB_DIALOG
+
 #ifndef WX_PRECOMP
 #include "wx/wx.h"
 #endif
@@ -43,7 +45,6 @@ wxTabControl::wxTabControl(wxTabView *v)
 {
   m_view = v;
   m_isSelected = FALSE;
-  m_labelFont = (wxFont *) NULL;
   m_offsetX = 0;
   m_offsetY = 0;
   m_width = 0;
@@ -56,18 +57,18 @@ wxTabControl::wxTabControl(wxTabView *v)
 wxTabControl::~wxTabControl(void)
 {
 }
-    
+
 void wxTabControl::OnDraw(wxDC& dc, bool lastInRow)
 {
     // Old, but in some ways better (drawing opaque tabs)
 #ifndef wxUSE_NEW_METHOD
   if (!m_view)
     return;
-    
+
   // Top-left of tab view area
   int viewX = m_view->GetViewRect().x;
   int viewY = m_view->GetViewRect().y;
-  
+
   // Top-left of tab control
   int tabX = GetX() + viewX;
   int tabY = GetY() + viewY;
@@ -77,7 +78,7 @@ void wxTabControl::OnDraw(wxDC& dc, bool lastInRow)
     tabHeightInc = (m_view->GetTabSelectionHeight() - m_view->GetTabHeight());
     tabY -= tabHeightInc;
   }
-  
+
   dc.SetPen(*wxTRANSPARENT_PEN);
 
   // Draw grey background
@@ -86,9 +87,13 @@ void wxTabControl::OnDraw(wxDC& dc, bool lastInRow)
     dc.SetBrush(*m_view->GetBackgroundBrush());
 
     // Add 1 because the pen is transparent. Under Motif, may be different.
+#ifdef __WXMOTIF__
+    dc.DrawRectangle(tabX, tabY, (GetWidth()+1), (GetHeight() + tabHeightInc));
+#else
     dc.DrawRectangle(tabX, tabY, (GetWidth()+1), (GetHeight() + 1 + tabHeightInc));
+#endif
   }
-  
+
   // Draw highlight and shadow
   dc.SetPen(*m_view->GetHighlightPen());
 
@@ -143,7 +148,7 @@ void wxTabControl::OnDraw(wxDC& dc, bool lastInRow)
     if (lastInRow)
     {
       // 25/5/97 UNLESS it's less than the max number of positions in this row
-      
+
       int topY = m_view->GetViewRect().y - m_view->GetTopMargin();
 
       int maxPositions = ((wxTabLayer *)m_view->GetLayers().Nth(0)->Data())->Number();
@@ -152,6 +157,10 @@ void wxTabControl::OnDraw(wxDC& dc, bool lastInRow)
       if ( GetRowPosition() < (maxPositions - 1) )
         topY = tabY + GetHeight() + tabHeightInc;
 
+#ifdef __WXMOTIF__
+      topY -= 1;
+#endif
+
       // Shadow
       dc.DrawLine((tabX + GetWidth()), tabY, (tabX + GetWidth()), topY);
       // Draw black line to emphasize shadow
@@ -170,6 +179,10 @@ void wxTabControl::OnDraw(wxDC& dc, bool lastInRow)
       if (tabBeneath && tabBeneath->IsSelected())
         subtractThis = (m_view->GetTabSelectionHeight() - m_view->GetTabHeight());
 
+#ifdef __WXMOTIF__
+      subtractThis += 1;
+#endif
+
       // Draw only to next tab down.
       dc.DrawLine((tabX + GetWidth()), tabY,
          (tabX + GetWidth()), (tabY + GetHeight() + tabHeightInc - subtractThis));
@@ -180,14 +193,14 @@ void wxTabControl::OnDraw(wxDC& dc, bool lastInRow)
          (tabY + GetHeight() + tabHeightInc - subtractThis));
     }
   }
-  
+
   // Draw centered text
   int textY = tabY + m_view->GetVerticalTabTextSpacing() + tabHeightInc;
 
   if (m_isSelected)
-    dc.SetFont(*m_view->GetSelectedTabFont());
+    dc.SetFont(* m_view->GetSelectedTabFont());
   else
-    dc.SetFont(*GetFont());
+    dc.SetFont(* GetFont());
 
   wxColour col(m_view->GetTextColour());
   dc.SetTextForeground(col);
@@ -196,7 +209,12 @@ void wxTabControl::OnDraw(wxDC& dc, bool lastInRow)
   dc.GetTextExtent(GetLabel(), &textWidth, &textHeight);
 
   int textX = (int)(tabX + (GetWidth() - textWidth)/2.0);
+  if (textX < (tabX + 2))
+    textX = (tabX + 2);
+
+  dc.SetClippingRegion(tabX, tabY, GetWidth(), GetHeight());
   dc.DrawText(GetLabel(), textX, textY);
+  dc.DestroyClippingRegion();
 
   if (m_isSelected)
   {
@@ -213,258 +231,258 @@ void wxTabControl::OnDraw(wxDC& dc, bool lastInRow)
 #else
     // New HEL version with rounder tabs
 
-       if (!m_view) return;
-
-       int tabInc   = 0;
-       if (m_isSelected)
-       {
-               tabInc = m_view->GetTabSelectionHeight() - m_view->GetTabHeight();
-       }
-       int tabLeft  = GetX() + m_view->GetViewRect().x;
-       int tabTop   = GetY() + m_view->GetViewRect().y - tabInc;
-       int tabRight = tabLeft + m_view->GetTabWidth();
-       int left     = m_view->GetViewRect().x;
-       int top      = tabTop + m_view->GetTabHeight() + tabInc;
-       int right    = left + m_view->GetViewRect().width;
-       int bottom   = top + m_view->GetViewRect().height;
-
-       if (m_isSelected)
-       {
-               // TAB is selected - draw TAB and the View's full outline
-
-               dc.SetPen(*(m_view->GetHighlightPen()));
-               wxPoint pnts[10];
-               int n = 0;
-               pnts[n].x = left;                       pnts[n++].y = bottom;
-               pnts[n].x = left;                       pnts[n++].y = top;
-               pnts[n].x = tabLeft;            pnts[n++].y = top;
-               pnts[n].x = tabLeft;                    pnts[n++].y = tabTop + 2;
-               pnts[n].x = tabLeft + 2;                pnts[n++].y = tabTop;
-               pnts[n].x = tabRight - 1;       pnts[n++].y = tabTop;
-               dc.DrawLines(n, pnts);
-               if (!lastInRow)
-               {
-                       dc.DrawLine(
-                                       (tabRight + 2),
-                                       top,
-                                       right,
-                                       top
-                                       );
-               }
-
-               dc.SetPen(*(m_view->GetShadowPen()));
-               dc.DrawLine(
-                               tabRight,
-                               tabTop + 2,
-                               tabRight,
-                               top
-                               );
-               dc.DrawLine(
-                               right,
-                               top,
-                               right,
-                               bottom
-                               );
-               dc.DrawLine(
-                               right,
-                               bottom,
-                               left,
-                               bottom
-                               );
-
-               dc.SetPen(*wxBLACK_PEN);
-               dc.DrawPoint(
-                               tabRight,
-                               tabTop + 1
-                               );
-               dc.DrawPoint(
-                               tabRight + 1,
-                               tabTop + 2
-                               );
-               if (lastInRow)
-               {
-                       dc.DrawLine(
-                               tabRight + 1,
-                               bottom,
-                               tabRight + 1,
-                               tabTop + 1
-                               );
-               }
-               else
-               {
-                       dc.DrawLine(
-                               tabRight + 1,
-                               tabTop + 2,
-                               tabRight + 1,
-                               top
-                               );
-                       dc.DrawLine(
-                               right + 1,
-                               top,
-                               right + 1,
-                               bottom + 1
-                               );
-               }
-               dc.DrawLine(
-                               right + 1,
-                               bottom + 1,
-                               left + 1,
-                               bottom + 1
-                               );
-       }
-       else
-       {
-               // TAB is not selected - just draw TAB outline and RH edge
-               // if the TAB is the last in the row
-
-               int maxPositions = ((wxTabLayer*)m_view->GetLayers().Nth(0)->Data())->Number();
-               wxTabControl* tabBelow = 0;
-               wxTabControl* tabBelowRight = 0;
-               if (GetColPosition() > 0)
-               {
-                       tabBelow = m_view->FindTabControlForPosition(
-                                               GetColPosition() - 1,
-                                               GetRowPosition()
-                                               );
-               }
-               if (!lastInRow && GetColPosition() > 0)
-               {
-                       tabBelowRight = m_view->FindTabControlForPosition(
-                                               GetColPosition() - 1,
-                                               GetRowPosition() + 1
-                                               );
-               }
-
-               float raisedTop = top - m_view->GetTabSelectionHeight() +
-                                                       m_view->GetTabHeight();
-
-               dc.SetPen(*(m_view->GetHighlightPen()));
-               wxPoint pnts[10];
-               int n = 0;
-
-               pnts[n].x = tabLeft;
-
-               if (tabBelow && tabBelow->IsSelected())
-               {
-                       pnts[n++].y = (long)raisedTop;
-               }
-               else
-               {
-                       pnts[n++].y = top;
-               }
-               pnts[n].x = tabLeft;                    pnts[n++].y = tabTop + 2;
-               pnts[n].x = tabLeft + 2;                pnts[n++].y = tabTop;
-               pnts[n].x = tabRight - 1;       pnts[n++].y = tabTop;
-               dc.DrawLines(n, pnts);
-
-               dc.SetPen(*(m_view->GetShadowPen()));
-               if (GetRowPosition() >= maxPositions - 1)
-               {
-                       dc.DrawLine(
-                                       tabRight,
-                                       (tabTop + 2),
-                                       tabRight,
-                                       bottom
-                                       );
-                       dc.DrawLine(
-                                       tabRight,
-                                       bottom,
-                                       (tabRight - m_view->GetHorizontalTabOffset()),
-                                       bottom
-                                       );
-               }
-               else
-               {
-                       if (tabBelowRight && tabBelowRight->IsSelected())
-                       {
-                               dc.DrawLine(
-                                               tabRight,
-                                               (long)raisedTop,
-                                               tabRight,
-                                               tabTop + 1
-                                               );
-                       }
-                       else
-                       {
-                               dc.DrawLine(
-                                               tabRight,
-                                               top - 1,
-                                               tabRight,
-                                               tabTop + 1
-                                               );
-                       }
-               }
-
-               dc.SetPen(*wxBLACK_PEN);
-               dc.DrawPoint(
-                               tabRight,
-                               tabTop + 1
-                               );
-               dc.DrawPoint(
-                               tabRight + 1,
-                               tabTop + 2
-                               );
-               if (GetRowPosition() >= maxPositions - 1)
-               {
-                       // draw right hand edge to bottom of view
-                       dc.DrawLine(
-                                       tabRight + 1,
-                                       bottom + 1,
-                                       tabRight + 1,
-                                       tabTop + 2
-                                       );
-                       dc.DrawLine(
-                                       tabRight + 1,
-                                       bottom + 1,
-                                       (tabRight - m_view->GetHorizontalTabOffset()),
-                                       bottom + 1
-                                       );
-               }
-               else
-               {
-                       // draw right hand edge of TAB
-                       if (tabBelowRight && tabBelowRight->IsSelected())
-                       {
-                               dc.DrawLine(
-                                               tabRight + 1,
-                                               (long)(raisedTop - 1),
-                                               tabRight + 1,
-                                               tabTop + 2
-                                               );
-                       }
-                       else
-                       {
-                               dc.DrawLine(
-                                               tabRight + 1,
-                                               top - 1,
-                                               tabRight + 1,
-                                               tabTop + 2
-                                               );
-                       }
-               }
-       }
-
-       // Draw centered text
-       dc.SetPen(*wxBLACK_PEN);
-       if (m_isSelected)
-       {
-               dc.SetFont(*(m_view->GetSelectedTabFont()));
-       }
-       else
-       {
-               dc.SetFont(*(GetFont()));
-       }
-
-       wxColour col(m_view->GetTextColour());
-       dc.SetTextForeground(col);
-       dc.SetBackgroundMode(wxTRANSPARENT);
-       long textWidth, textHeight;
-       dc.GetTextExtent(GetLabel(), &textWidth, &textHeight);
-
-       float textX = (tabLeft + tabRight - textWidth) / 2;
-       float textY = (tabInc + tabTop + m_view->GetVerticalTabTextSpacing());
-       
-       dc.DrawText(GetLabel(), (long)textX, (long)textY);
+    if (!m_view) return;
+
+    int tabInc   = 0;
+    if (m_isSelected)
+    {
+        tabInc = m_view->GetTabSelectionHeight() - m_view->GetTabHeight();
+    }
+    int tabLeft  = GetX() + m_view->GetViewRect().x;
+    int tabTop   = GetY() + m_view->GetViewRect().y - tabInc;
+    int tabRight = tabLeft + m_view->GetTabWidth();
+    int left     = m_view->GetViewRect().x;
+    int top      = tabTop + m_view->GetTabHeight() + tabInc;
+    int right    = left + m_view->GetViewRect().width;
+    int bottom   = top + m_view->GetViewRect().height;
+
+    if (m_isSelected)
+    {
+        // TAB is selected - draw TAB and the View's full outline
+
+        dc.SetPen(*(m_view->GetHighlightPen()));
+        wxPoint pnts[10];
+        int n = 0;
+        pnts[n].x = left;            pnts[n++].y = bottom;
+        pnts[n].x = left;             pnts[n++].y = top;
+        pnts[n].x = tabLeft;         pnts[n++].y = top;
+        pnts[n].x = tabLeft;            pnts[n++].y = tabTop + 2;
+        pnts[n].x = tabLeft + 2;        pnts[n++].y = tabTop;
+        pnts[n].x = tabRight - 1;    pnts[n++].y = tabTop;
+        dc.DrawLines(n, pnts);
+        if (!lastInRow)
+        {
+            dc.DrawLine(
+                    (tabRight + 2),
+                    top,
+                    right,
+                    top
+                    );
+        }
+
+        dc.SetPen(*(m_view->GetShadowPen()));
+        dc.DrawLine(
+                tabRight,
+                tabTop + 2,
+                tabRight,
+                top
+                );
+        dc.DrawLine(
+                right,
+                top,
+                right,
+                bottom
+                );
+        dc.DrawLine(
+                right,
+                bottom,
+                left,
+                bottom
+                );
+
+        dc.SetPen(*wxBLACK_PEN);
+        dc.DrawPoint(
+                tabRight,
+                tabTop + 1
+                );
+        dc.DrawPoint(
+                tabRight + 1,
+                tabTop + 2
+                );
+        if (lastInRow)
+        {
+            dc.DrawLine(
+                tabRight + 1,
+                bottom,
+                tabRight + 1,
+                tabTop + 1
+                );
+        }
+        else
+        {
+            dc.DrawLine(
+                tabRight + 1,
+                tabTop + 2,
+                tabRight + 1,
+                top
+                );
+            dc.DrawLine(
+                right + 1,
+                top,
+                right + 1,
+                bottom + 1
+                );
+        }
+        dc.DrawLine(
+                right + 1,
+                bottom + 1,
+                left + 1,
+                bottom + 1
+                );
+    }
+    else
+    {
+        // TAB is not selected - just draw TAB outline and RH edge
+        // if the TAB is the last in the row
+
+        int maxPositions = ((wxTabLayer*)m_view->GetLayers().Nth(0)->Data())->Number();
+        wxTabControl* tabBelow = 0;
+        wxTabControl* tabBelowRight = 0;
+        if (GetColPosition() > 0)
+        {
+            tabBelow = m_view->FindTabControlForPosition(
+                        GetColPosition() - 1,
+                        GetRowPosition()
+                        );
+        }
+        if (!lastInRow && GetColPosition() > 0)
+        {
+            tabBelowRight = m_view->FindTabControlForPosition(
+                        GetColPosition() - 1,
+                        GetRowPosition() + 1
+                        );
+        }
+
+        float raisedTop = top - m_view->GetTabSelectionHeight() +
+                            m_view->GetTabHeight();
+
+        dc.SetPen(*(m_view->GetHighlightPen()));
+        wxPoint pnts[10];
+        int n = 0;
+
+        pnts[n].x = tabLeft;
+
+        if (tabBelow && tabBelow->IsSelected())
+        {
+            pnts[n++].y = (long)raisedTop;
+        }
+        else
+        {
+            pnts[n++].y = top;
+        }
+        pnts[n].x = tabLeft;            pnts[n++].y = tabTop + 2;
+        pnts[n].x = tabLeft + 2;        pnts[n++].y = tabTop;
+        pnts[n].x = tabRight - 1;    pnts[n++].y = tabTop;
+        dc.DrawLines(n, pnts);
+
+        dc.SetPen(*(m_view->GetShadowPen()));
+        if (GetRowPosition() >= maxPositions - 1)
+        {
+            dc.DrawLine(
+                    tabRight,
+                    (tabTop + 2),
+                    tabRight,
+                    bottom
+                    );
+            dc.DrawLine(
+                    tabRight,
+                    bottom,
+                    (tabRight - m_view->GetHorizontalTabOffset()),
+                    bottom
+                    );
+        }
+        else
+        {
+            if (tabBelowRight && tabBelowRight->IsSelected())
+            {
+                dc.DrawLine(
+                        tabRight,
+                        (long)raisedTop,
+                        tabRight,
+                        tabTop + 1
+                        );
+            }
+            else
+            {
+                dc.DrawLine(
+                        tabRight,
+                        top - 1,
+                        tabRight,
+                        tabTop + 1
+                        );
+            }
+        }
+
+        dc.SetPen(*wxBLACK_PEN);
+        dc.DrawPoint(
+                tabRight,
+                tabTop + 1
+                );
+        dc.DrawPoint(
+                tabRight + 1,
+                tabTop + 2
+                );
+        if (GetRowPosition() >= maxPositions - 1)
+        {
+            // draw right hand edge to bottom of view
+            dc.DrawLine(
+                    tabRight + 1,
+                    bottom + 1,
+                    tabRight + 1,
+                    tabTop + 2
+                    );
+            dc.DrawLine(
+                    tabRight + 1,
+                    bottom + 1,
+                    (tabRight - m_view->GetHorizontalTabOffset()),
+                    bottom + 1
+                    );
+        }
+        else
+        {
+            // draw right hand edge of TAB
+            if (tabBelowRight && tabBelowRight->IsSelected())
+            {
+                dc.DrawLine(
+                        tabRight + 1,
+                        (long)(raisedTop - 1),
+                        tabRight + 1,
+                        tabTop + 2
+                        );
+            }
+            else
+            {
+                dc.DrawLine(
+                        tabRight + 1,
+                        top - 1,
+                        tabRight + 1,
+                        tabTop + 2
+                        );
+            }
+        }
+    }
+
+    // Draw centered text
+    dc.SetPen(*wxBLACK_PEN);
+    if (m_isSelected)
+    {
+        dc.SetFont(*(m_view->GetSelectedTabFont()));
+    }
+    else
+    {
+        dc.SetFont(*(GetFont()));
+    }
+
+    wxColour col(m_view->GetTextColour());
+    dc.SetTextForeground(col);
+    dc.SetBackgroundMode(wxTRANSPARENT);
+    long textWidth, textHeight;
+    dc.GetTextExtent(GetLabel(), &textWidth, &textHeight);
+
+    float textX = (tabLeft + tabRight - textWidth) / 2;
+    float textY = (tabInc + tabTop + m_view->GetVerticalTabTextSpacing());
+
+    dc.DrawText(GetLabel(), (long)textX, (long)textY);
 #endif
 }
 
@@ -477,7 +495,7 @@ bool wxTabControl::HitTest(int x, int y) const
   // Bottom-right
   int tabX2 = tabX1 + GetWidth();
   int tabY2 = tabY1 + GetHeight();
-  
+
   if (x >= tabX1 && y >= tabY1 && x <= tabX2 && y <= tabY2)
     return TRUE;
   else
@@ -510,15 +528,16 @@ wxTabView::wxTabView(long style)
   m_shadowPen = wxGREY_PEN;
   m_backgroundPen = wxLIGHT_GREY_PEN;
   m_backgroundBrush = wxLIGHT_GREY_BRUSH;
-  m_tabFont = wxTheFontList->FindOrCreateFont(9, wxSWISS, wxNORMAL, wxNORMAL);
-  m_tabSelectedFont = wxTheFontList->FindOrCreateFont(9, wxSWISS, wxNORMAL, wxBOLD);
+  m_tabFont = wxSystemSettings::GetSystemFont(wxSYS_DEFAULT_GUI_FONT);
+  m_tabSelectedFont = wxSystemSettings::GetSystemFont(wxSYS_DEFAULT_GUI_FONT);
   m_window = (wxWindow *) NULL;
 }
 
 wxTabView::~wxTabView()
 {
+  ClearTabs(TRUE);
 }
-  
+
 // Automatically positions tabs
 // TODO: this should just add the tab to a list, and then
 // a layout function (e.g. Realize) should be called when all tabs have been added.
@@ -554,17 +573,17 @@ wxTabControl *wxTabView::AddTab(int id, const wxString& label, wxTabControl *exi
     }
   }
   int layer = m_layers.Number() - 1;
-  
+
   wxTabControl *tabControl = existingTab;
   if (!existingTab)
     tabControl = OnCreateTabControl();
   tabControl->SetRowPosition(tabLayer->Number());
   tabControl->SetColPosition(layer);
-  
+
   wxTabControl *lastTab = (wxTabControl *) NULL;
   if (lastTabNode)
     lastTab = (wxTabControl *)lastTabNode->Data();
-  
+
   // Top of new tab
   int verticalOffset = (- GetTopMargin()) - ((layer+1)*GetTabHeight());
   // Offset from view top-left
@@ -573,19 +592,67 @@ wxTabControl *wxTabView::AddTab(int id, const wxString& label, wxTabControl *exi
     horizontalOffset = layer*GetHorizontalTabOffset();
   else
     horizontalOffset = lastTab->GetX() + GetTabWidth() + GetHorizontalTabSpacing();
-  
+
   tabControl->SetPosition(horizontalOffset, verticalOffset);
   tabControl->SetSize(GetTabWidth(), GetTabHeight());
   tabControl->SetId(id);
   tabControl->SetLabel(label);
-  tabControl->SetFont(GetTabFont());
-  
+  tabControl->SetFont(GetTabFont());
+
   tabLayer->Append(tabControl);
   m_noTabs ++;
-  
+
   return tabControl;
 }
-  
+
+// Remove the tab without deleting the window
+bool wxTabView::RemoveTab(int id)
+{
+  wxNode *layerNode = m_layers.First();
+  while (layerNode)
+  {
+    wxTabLayer *layer = (wxTabLayer *)layerNode->Data();
+    wxNode *tabNode = layer->First();
+    while (tabNode)
+    {
+      wxTabControl *tab = (wxTabControl *)tabNode->Data();
+      if (tab->GetId() == id)
+      {
+        if (id == m_tabSelection)
+          m_tabSelection = -1;
+        delete tab;
+        delete tabNode;
+        m_noTabs --;
+
+        // The layout has changed
+        LayoutTabs();
+        return TRUE;
+      }
+      tabNode = tabNode->Next();
+    }
+    layerNode = layerNode->Next();
+  }
+  return FALSE;
+}
+
+bool wxTabView::SetTabText(int id, const wxString& label)
+{
+    wxTabControl* control = FindTabControlForId(id);
+    if (!control)
+      return FALSE;
+    control->SetLabel(label);
+    return TRUE;
+}
+
+wxString wxTabView::GetTabText(int id) const
+{
+    wxTabControl* control = FindTabControlForId(id);
+    if (!control)
+      return wxEmptyString;
+    else
+      return control->GetLabel();
+}
+
 // Returns the total height of the tabs component -- this may be several
 // times the height of a tab, if there are several tab layers (rows).
 int wxTabView::GetTotalTabHeight()
@@ -633,11 +700,13 @@ void wxTabView::ClearTabs(bool deleteTabs)
     delete layerNode;
     layerNode = nextLayerNode;
   }
+  m_noTabs = 0;
+  m_tabSelection = -1;
 }
 
 
 // Layout tabs (optional, e.g. if resizing window)
-void wxTabView::Layout(void)
+void wxTabView::LayoutTabs(void)
 {
   // Make a list of the tab controls, deleting the wxTabLayers.
   wxList controls;
@@ -660,12 +729,12 @@ void wxTabView::Layout(void)
     delete layerNode;
     layerNode = nextLayerNode;
   }
-  
+
   wxTabControl *lastTab = (wxTabControl *) NULL;
-  
+
   wxTabLayer *currentLayer = new wxTabLayer;
   m_layers.Append(currentLayer);
-  
+
   wxNode *node = controls.First();
   while (node)
   {
@@ -686,12 +755,12 @@ void wxTabView::Layout(void)
        lastTab = (wxTabControl *) NULL;
      }
     }
-    
+
     int layer = m_layers.Number() - 1;
 
     tabControl->SetRowPosition(currentLayer->Number());
     tabControl->SetColPosition(layer);
-  
+
     // Top of new tab
     int verticalOffset = (- GetTopMargin()) - ((layer+1)*GetTabHeight());
     // Offset from view top-left
@@ -700,7 +769,7 @@ void wxTabView::Layout(void)
       horizontalOffset = layer*GetHorizontalTabOffset();
     else
       horizontalOffset = lastTab->GetX() + GetTabWidth() + GetHorizontalTabSpacing();
-  
+
     tabControl->SetPosition(horizontalOffset, verticalOffset);
     tabControl->SetSize(GetTabWidth(), GetTabHeight());
 
@@ -720,91 +789,100 @@ void wxTabView::Layout(void)
 // Draw all tabs
 void wxTabView::Draw(wxDC& dc)
 {
-       // Draw top margin area (beneath tabs and above view area)
-       if (GetTabStyle() & wxTAB_STYLE_COLOUR_INTERIOR)
-       {
-               dc.SetPen(*wxTRANSPARENT_PEN);
-               dc.SetBrush(*GetBackgroundBrush());
-
-               // Add 1 because the pen is transparent. Under Motif, may be different.
-               dc.DrawRectangle(
-                               m_tabViewRect.x,
-                               (m_tabViewRect.y - m_topMargin),
-                               (m_tabViewRect.width + 1),
-                               (m_topMargin + 1)
-                               );
-       }
-
-       // Draw layers in reverse order
-       wxNode *node = m_layers.Last();
-       while (node)
-       {
-               wxTabLayer *layer = (wxTabLayer *)node->Data();
-               wxNode *node2 = layer->First();
-               while (node2)
-               {
-                       wxTabControl *control = (wxTabControl *)node2->Data();
-                       control->OnDraw(dc, (node2->Next() == NULL));
-                       node2 = node2->Next();
-               }
-
-               node = node->Previous();
-       }
+        // Don't draw anything if there are no tabs.
+        if (GetNumberOfTabs() == 0)
+          return;
+
+    // Draw top margin area (beneath tabs and above view area)
+    if (GetTabStyle() & wxTAB_STYLE_COLOUR_INTERIOR)
+    {
+        dc.SetPen(*wxTRANSPARENT_PEN);
+        dc.SetBrush(*GetBackgroundBrush());
+
+        // Add 1 because the pen is transparent. Under Motif, may be different.
+        dc.DrawRectangle(
+                m_tabViewRect.x,
+                (m_tabViewRect.y - m_topMargin),
+                (m_tabViewRect.width + 1),
+                (m_topMargin + 1)
+                );
+    }
+
+    // Draw layers in reverse order
+    wxNode *node = m_layers.Last();
+    while (node)
+    {
+        wxTabLayer *layer = (wxTabLayer *)node->Data();
+        wxNode *node2 = layer->First();
+        while (node2)
+        {
+            wxTabControl *control = (wxTabControl *)node2->Data();
+            control->OnDraw(dc, (node2->Next() == NULL));
+            node2 = node2->Next();
+        }
+
+        node = node->Previous();
+    }
 
 
 #ifndef wxUSE_NEW_METHOD
-       if (GetTabStyle() & wxTAB_STYLE_DRAW_BOX)
-       {
-               dc.SetPen(GetShadowPen());
-
-               // Draw bottom line
-               dc.DrawLine(
-                               (GetViewRect().x + 1),
-                               (GetViewRect().y + GetViewRect().height),
-                               (GetViewRect().x + GetViewRect().width + 1),
-                               (GetViewRect().y + GetViewRect().height)
-                               );
-
-               // Draw right line
-               dc.DrawLine(
-                               (GetViewRect().x + GetViewRect().width),
-                               (GetViewRect().y - GetTopMargin() + 1),
-                               (GetViewRect().x + GetViewRect().width),
-                               (GetViewRect().y + GetViewRect().height)
-                               );
-
-               dc.SetPen(wxBLACK_PEN);
-
-               // Draw bottom line
-               dc.DrawLine(
-                               (GetViewRect().x),
-                               (GetViewRect().y + GetViewRect().height + 1),
-                               (GetViewRect().x + GetViewRect().width + 2),
-                               (GetViewRect().y + GetViewRect().height + 1)
-                               );
-
-               // Draw right line
-               dc.DrawLine(
-                               (GetViewRect().x + GetViewRect().width + 1),
-                               (GetViewRect().y - GetTopMargin()),
-                               (GetViewRect().x + GetViewRect().width + 1),
-                               (GetViewRect().y + GetViewRect().height + 1)
-                               );
-       }
+    if (GetTabStyle() & wxTAB_STYLE_DRAW_BOX)
+    {
+        dc.SetPen(* GetShadowPen());
+
+        // Draw bottom line
+        dc.DrawLine(
+                (GetViewRect().x + 1),
+                (GetViewRect().y + GetViewRect().height),
+                (GetViewRect().x + GetViewRect().width + 1),
+                (GetViewRect().y + GetViewRect().height)
+                );
+
+        // Draw right line
+        dc.DrawLine(
+                (GetViewRect().x + GetViewRect().width),
+                (GetViewRect().y - GetTopMargin() + 1),
+                (GetViewRect().x + GetViewRect().width),
+                (GetViewRect().y + GetViewRect().height)
+                );
+
+        dc.SetPen(* wxBLACK_PEN);
+
+        // Draw bottom line
+        dc.DrawLine(
+                (GetViewRect().x),
+                (GetViewRect().y + GetViewRect().height + 1),
+#if defined(__WXMOTIF__)
+                (GetViewRect().x + GetViewRect().width + 1),
+#else
+                (GetViewRect().x + GetViewRect().width + 2),
+#endif
+
+                (GetViewRect().y + GetViewRect().height + 1)
+                );
+
+        // Draw right line
+        dc.DrawLine(
+                (GetViewRect().x + GetViewRect().width + 1),
+                (GetViewRect().y - GetTopMargin()),
+                (GetViewRect().x + GetViewRect().width + 1),
+                (GetViewRect().y + GetViewRect().height + 1)
+                );
+    }
 #endif
 }
-  
+
 // Process mouse event, return FALSE if we didn't process it
 bool wxTabView::OnEvent(wxMouseEvent& event)
 {
   if (!event.LeftDown())
     return FALSE;
-    
-  float x, y;
-  event.Position(&x, &y);
-  
+
+  wxCoord x, y;
+  event.GetPosition(&x, &y);
+
   wxTabControl *hitControl = (wxTabControl *) NULL;
-  
+
   wxNode *node = m_layers.First();
   while (node)
   {
@@ -822,21 +900,21 @@ bool wxTabView::OnEvent(wxMouseEvent& event)
       else
         node2 = node2->Next();
     }
-  
+
     if (node)
       node = node->Next();
   }
-  
+
   if (!hitControl)
     return FALSE;
-    
+
   wxTabControl *currentTab = FindTabControlForId(m_tabSelection);
-    
+
   if (hitControl == currentTab)
     return FALSE;
-    
+
   ChangeTab(hitControl);
-  
+
   return TRUE;
 }
 
@@ -846,13 +924,13 @@ bool wxTabView::ChangeTab(wxTabControl *control)
   int oldTab = -1;
   if (currentTab)
     oldTab = currentTab->GetId();
-  
+
   if (control == currentTab)
     return TRUE;
-    
+
   if (m_layers.Number() == 0)
     return FALSE;
-    
+
   if (!OnTabPreActivate(control->GetId(), oldTab))
     return FALSE;
 
@@ -861,12 +939,12 @@ bool wxTabView::ChangeTab(wxTabControl *control)
 
   if (currentTab)
     currentTab->SetSelected(FALSE);
-    
+
   control->SetSelected(TRUE);
   m_tabSelection = control->GetId();
 
   OnTabActivate(control->GetId(), oldTab);
-  
+
   // Leave window refresh for the implementing window
 
   return TRUE;
@@ -878,9 +956,9 @@ bool wxTabView::MoveSelectionTab(wxTabControl *control)
 {
   if (m_layers.Number() == 0)
     return FALSE;
-    
+
   wxTabLayer *firstLayer = (wxTabLayer *)m_layers.First()->Data();
-    
+
   // Find what column this tab is at, so we can swap with the one at the bottom.
   // If we're on the bottom layer, then no need to swap.
   if (!firstLayer->Member(control))
@@ -893,29 +971,29 @@ bool wxTabView::MoveSelectionTab(wxTabControl *control)
     wxNode *otherNode = firstLayer->Nth(col);
     if (!otherNode)
       return FALSE;
-      
+
     // If this is already in the bottom layer, return now
     if (otherNode == thisNode)
       return TRUE;
-      
+
     wxTabControl *otherTab = (wxTabControl *)otherNode->Data();
-     
+
     // We now have pointers to the tab to be changed to,
     // and the tab on the first layer. Swap tab structures and
     // position details.
-     
+
     int thisX = control->GetX();
     int thisY = control->GetY();
     int thisColPos = control->GetColPosition();
     int otherX = otherTab->GetX();
     int otherY = otherTab->GetY();
     int otherColPos = otherTab->GetColPosition();
-     
+
     control->SetPosition(otherX, otherY);
     control->SetColPosition(otherColPos);
     otherTab->SetPosition(thisX, thisY);
     otherTab->SetColPosition(thisColPos);
-     
+
     // Swap the data for the nodes
     thisNode->SetData(otherTab);
     otherNode->SetData(control);
@@ -927,7 +1005,7 @@ bool wxTabView::MoveSelectionTab(wxTabControl *control)
 void wxTabView::OnTabActivate(int /*activateId*/, int /*deactivateId*/)
 {
 }
-  
+
 void wxTabView::SetHighlightColour(const wxColour& col)
 {
   m_highlightColour = col;
@@ -949,23 +1027,32 @@ void wxTabView::SetBackgroundColour(const wxColour& col)
 
 void wxTabView::SetTabSelection(int sel, bool activateTool)
 {
+  if ( sel==m_tabSelection )
+    return;
+
   int oldSel = m_tabSelection;
   wxTabControl *control = FindTabControlForId(sel);
+  wxTabControl *oldControl = FindTabControlForId(m_tabSelection);
 
   if (!OnTabPreActivate(sel, oldSel))
     return;
-    
+
   if (control)
-    control->SetSelected((sel != 0)); // TODO ??
-  else
+    control->SetSelected((sel != -1)); // TODO ??
+  else if (sel != -1)
   {
-    wxMessageBox(_("Could not find tab for id"), _("Error"), wxOK);
+    wxFAIL_MSG(_("Could not find tab for id"));
     return;
   }
-    
+
+  if (oldControl)
+    oldControl->SetSelected(FALSE);
+
   m_tabSelection = sel;
-  MoveSelectionTab(control);
-  
+
+  if (control)
+    MoveSelectionTab(control);
+
   if (activateTool)
     OnTabActivate(sel, oldSel);
 }
@@ -1041,7 +1128,7 @@ int wxTabView::CalculateTabWidth(int noTabs, bool adjustView)
 /*
  * wxTabbedDialog
  */
+
 IMPLEMENT_CLASS(wxTabbedDialog, wxDialog)
 
 BEGIN_EVENT_TABLE(wxTabbedDialog, wxDialog)
@@ -1064,7 +1151,7 @@ wxTabbedDialog::~wxTabbedDialog(void)
   if (m_tabView)
     delete m_tabView;
 }
+
 void wxTabbedDialog::OnCloseWindow(wxCloseEvent& WXUNUSED(event) )
 {
   Destroy();
@@ -1086,7 +1173,7 @@ void wxTabbedDialog::OnPaint(wxPaintEvent& WXUNUSED(event) )
 /*
  * wxTabbedPanel
  */
+
 IMPLEMENT_CLASS(wxTabbedPanel, wxPanel)
 
 BEGIN_EVENT_TABLE(wxTabbedPanel, wxPanel)
@@ -1105,7 +1192,7 @@ wxTabbedPanel::~wxTabbedPanel(void)
 {
   delete m_tabView;
 }
+
 void wxTabbedPanel::OnMouseEvent(wxMouseEvent& event)
 {
   if (m_tabView)
@@ -1122,9 +1209,9 @@ void wxTabbedPanel::OnPaint(wxPaintEvent& WXUNUSED(event) )
 /*
  * wxPanelTabView
  */
+
 IMPLEMENT_CLASS(wxPanelTabView, wxTabView)
+
 wxPanelTabView::wxPanelTabView(wxPanel *pan, long style): wxTabView(style), m_tabWindows(wxKEY_INTEGER)
 {
   m_panel = pan;
@@ -1148,7 +1235,7 @@ void wxPanelTabView::OnTabActivate(int activateId, int deactivateId)
 {
   if (!m_panel)
     return;
-    
+
   wxWindow *oldWindow = ((deactivateId == -1) ? 0 : GetTabWindow(deactivateId));
   wxWindow *newWindow = GetTabWindow(activateId);
 
@@ -1156,11 +1243,11 @@ void wxPanelTabView::OnTabActivate(int activateId, int deactivateId)
     oldWindow->Show(FALSE);
   if (newWindow)
     newWindow->Show(TRUE);
-    
+
   m_panel->Refresh();
 }
 
-   
+
 void wxPanelTabView::AddTabWindow(int id, wxWindow *window)
 {
   m_tabWindows.Append((long)id, window);
@@ -1172,7 +1259,7 @@ wxWindow *wxPanelTabView::GetTabWindow(int id) const
   wxNode *node = m_tabWindows.Find((long)id);
   if (!node)
     return (wxWindow *) NULL;
-  return (wxWindow *)node->Data();    
+  return (wxWindow *)node->Data();
 }
 
 void wxPanelTabView::ClearWindows(bool deleteWindows)
@@ -1194,3 +1281,4 @@ void wxPanelTabView::ShowWindowForTab(int id)
   newWindow->Refresh();
 }
 
+#endif // wxUSE_TAB_DIALOG