add IsOk() to all classes having Ok() method (patch 1570985)

[wxWidgets.git] / src / mac / carbon / notebmac.cpp

diff --git a/src/mac/carbon/notebmac.cpp b/src/mac/carbon/notebmac.cpp
index df02de6a4b2cc5f20549cfa9e574e0e5cd3f5eb1..3406606bb8be9972d1dbeae8e3652a18b3e2a2d7 100644 (file)
--- a/src/mac/carbon/notebmac.cpp
+++ b/src/mac/carbon/notebmac.cpp
@@ -13,12 +13,17 @@
 
 #if wxUSE_NOTEBOOK
 
 
 #if wxUSE_NOTEBOOK
 

-#include "wx/app.h"
+#include "wx/notebook.h"
+
+#ifndef WX_PRECOMP
+    #include "wx/string.h"
+    #include "wx/log.h"
+    #include "wx/app.h"
+    #include "wx/image.h"
+#endif
+

 #include "wx/string.h"
 #include "wx/string.h"

-#include "wx/log.h"

 #include "wx/imaglist.h"
 #include "wx/imaglist.h"

-#include "wx/image.h"
-#include "wx/notebook.h"

 #include "wx/mac/uma.h"
 
 
 #include "wx/mac/uma.h"
 
 


@@ -75,6 +80,9 @@ bool wxNotebook::Create( wxWindow *parent,
 {
     m_macIsUserPane = false ;
 
 {
     m_macIsUserPane = false ;
 

+    if (! (style & wxBK_ALIGN_MASK))
+        style |= wxBK_TOP;
+

     if ( !wxNotebookBase::Create( parent, id, pos, size, style, name ) )
         return false;
 
     if ( !wxNotebookBase::Create( parent, id, pos, size, style, name ) )
         return false;
 


@@ -96,18 +104,18 @@ bool wxNotebook::Create( wxWindow *parent,
     ControlTabSize tabsize;
     switch (GetWindowVariant())
     {
     ControlTabSize tabsize;
     switch (GetWindowVariant())
     {

-       case wxWINDOW_VARIANT_MINI:
+        case wxWINDOW_VARIANT_MINI:

             if ( UMAGetSystemVersion() >= 0x1030 )
                 tabsize = 3 ;
             else
                 tabsize = kControlSizeSmall;
             break;
 
             if ( UMAGetSystemVersion() >= 0x1030 )
                 tabsize = 3 ;
             else
                 tabsize = kControlSizeSmall;
             break;
 

-       case wxWINDOW_VARIANT_SMALL:
+        case wxWINDOW_VARIANT_SMALL:

             tabsize = kControlTabSizeSmall;
             break;
 
             tabsize = kControlTabSizeSmall;
             break;
 

-       default:
+        default:

             tabsize = kControlTabSizeLarge;
             break;
     }
             tabsize = kControlTabSizeLarge;
             break;
     }


@@ -152,24 +160,30 @@ wxSize wxNotebook::CalcSizeFromPage(const wxSize& sizePage) const
     return DoGetSizeFromClientSize( sizePage );
 }
 
     return DoGetSizeFromClientSize( sizePage );
 }
 

-int wxNotebook::SetSelection(size_t nPage)
+int wxNotebook::DoSetSelection(size_t nPage, int flags)

 {
 {

-    wxCHECK_MSG( IS_VALID_PAGE(nPage), wxNOT_FOUND, wxT("SetSelection: invalid notebook page") );
+    wxCHECK_MSG( IS_VALID_PAGE(nPage), wxNOT_FOUND, wxT("DoSetSelection: invalid notebook page") );

 
 

-    if ( int(nPage) != m_nSelection )
+    if ( m_nSelection != wxNOT_FOUND && nPage != (size_t)m_nSelection )

     {
     {

-        wxNotebookEvent event(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGING, m_windowId);
-        event.SetSelection(nPage);
-        event.SetOldSelection(m_nSelection);
-        event.SetEventObject(this);
-        if ( !GetEventHandler()->ProcessEvent(event) || event.IsAllowed() )
+        if ( flags & SetSelection_SendEvent )

         {
         {

-            // program allows the page change
+            wxNotebookEvent event(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGING, m_windowId);
+            event.SetSelection(nPage);
+            event.SetOldSelection(m_nSelection);
+            event.SetEventObject(this);
+            if ( GetEventHandler()->ProcessEvent(event) && !event.IsAllowed() )
+            {
+                // vetoed by program
+                return m_nSelection;
+            }
+            //else: program allows the page change
+

             event.SetEventType(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGED);
             (void)GetEventHandler()->ProcessEvent(event);
             event.SetEventType(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGED);
             (void)GetEventHandler()->ProcessEvent(event);

-
-            ChangePage(m_nSelection, nPage);

         }
         }

+
+        ChangePage(m_nSelection, nPage);

     }
 
     return m_nSelection;
     }
 
     return m_nSelection;


@@ -323,11 +337,11 @@ int wxNotebook::HitTest(const wxPoint& pt, long * flags) const
     const int countPages = GetPageCount();
 
     // we have to convert from Client to Window relative coordinates
     const int countPages = GetPageCount();
 
     // we have to convert from Client to Window relative coordinates

-    wxPoint adjustedPt = pt + GetClientAreaOrigin(); 
+    wxPoint adjustedPt = pt + GetClientAreaOrigin();

     // and now to HIView native ones
     adjustedPt.x -= MacGetLeftBorderSize() ;
     adjustedPt.y -= MacGetTopBorderSize() ;
     // and now to HIView native ones
     adjustedPt.x -= MacGetLeftBorderSize() ;
     adjustedPt.y -= MacGetTopBorderSize() ;

-    
+

     HIPoint hipoint= { adjustedPt.x , adjustedPt.y } ;
     HIViewPartCode outPart = 0 ;
     OSStatus err = HIViewGetPartHit( m_peer->GetControlRef(), &hipoint, &outPart );
     HIPoint hipoint= { adjustedPt.x , adjustedPt.y } ;
     HIViewPartCode outPart = 0 ;
     OSStatus err = HIViewGetPartHit( m_peer->GetControlRef(), &hipoint, &outPart );


@@ -368,9 +382,9 @@ int wxNotebook::HitTest(const wxPoint& pt, long * flags) const
 
         // we cannot differentiate better
         if (resultV >= 0)
 
         // we cannot differentiate better
         if (resultV >= 0)

-            *flags |= wxNB_HITTEST_ONLABEL;
+            *flags |= wxBK_HITTEST_ONLABEL;

         else
         else

-            *flags |= wxNB_HITTEST_NOWHERE;
+            *flags |= wxBK_HITTEST_NOWHERE;

     }
 
     return resultV;
     }
 
     return resultV;