uses wxString again (std::string optional)

[wxWidgets.git] / src / gtk / notebook.cpp

diff --git a/src/gtk/notebook.cpp b/src/gtk/notebook.cpp
index 7ec169b52664a25fbbe67ad5f1fd1ed4cb2696a9..4021fcaf49d543d9076fc936f5c3eb0a9cf1b943 100644 (file)
--- a/src/gtk/notebook.cpp
+++ b/src/gtk/notebook.cpp
@@ -5,7 +5,7 @@
 // Created:     01/02/97
 // Id:
 // Copyright:   (c) 1998 Robert Roebling, Julian Smart and Markus Holzem
 // Created:     01/02/97
 // Id:
 // Copyright:   (c) 1998 Robert Roebling, Julian Smart and Markus Holzem

-// Licence:    wxWindows licence
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__


@@ -16,6 +16,8 @@
 #include "wx/panel.h"
 #include "wx/utils.h"
 #include "wx/imaglist.h"
 #include "wx/panel.h"
 #include "wx/utils.h"
 #include "wx/imaglist.h"

+#include "wx/intl.h"
+#include "wx/log.h"

 
 //-----------------------------------------------------------------------------
 // wxNotebookPage
 
 //-----------------------------------------------------------------------------
 // wxNotebookPage


@@ -23,25 +25,68 @@
 
 class wxNotebookPage: public wxObject
 {
 
 class wxNotebookPage: public wxObject
 {

-  public:
+public:
+  wxNotebookPage()
+  {
+    m_id = -1;
+    m_text = "";
+    m_image = -1;
+    m_page = NULL;
+    m_client = NULL;
+    m_parent = NULL;
+  };
+
+//private:
+  int                m_id;
+  wxString           m_text;
+  int                m_image;
+  GtkNotebookPage   *m_page;
+  GtkLabel          *m_label;
+  wxWindow          *m_client;
+  GtkNotebook       *m_parent;
+};
+
+//-----------------------------------------------------------------------------
+// GTK callbacks
+//-----------------------------------------------------------------------------
+
+// page change callback
+static void gtk_notebook_page_change_callback(GtkNotebook *WXUNUSED(widget),
+                                              GtkNotebookPage *WXUNUSED(page),
+                                              gint nPage,
+                                              gpointer data)
+{
+  wxNotebook *notebook = (wxNotebook *)data;
+
+  int nOld = notebook->GetSelection();
+
+  // TODO: emulate PAGE_CHANGING event
+  wxNotebookEvent event(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGED,
+                        notebook->GetId(),
+                        nPage,
+                        nOld);
+  event.SetEventObject(notebook);
+  notebook->ProcessEvent(event);
+}
+
+static void gtk_notebook_client_size_callback( GtkWidget *WXUNUSED(widget), GtkAllocation* alloc, 
+                                               wxNotebookPage *page )
+{
+  if (!page->m_client->HasVMT()) return;
+
+/*
+  printf( "OnResize from " );
+  if (page->m_client->GetClassInfo() && page->m_client->GetClassInfo()->GetClassName())
+    printf( page->m_client->GetClassInfo()->GetClassName() );
+  printf( ".\n" );
+
+  printf( "  New: X: %d  Y: %d ", alloc->x, alloc->y );
+  printf( "  W: %d  H: %d ", alloc->width, alloc->height );
+  printf( " .\n" );
+*/

   
   

-   int                m_id;
-   wxString           m_text;
-   int                m_image;
-   void              *m_clientData;
-   GtkNotebookPage   *m_page;
-   wxPanel           *m_clientPanel;
-   
-   wxNotebookPage(void)
-   {
-     m_id = -1;
-     m_text = "";
-     m_image = -1;
-     m_clientData = NULL;
-     m_page = NULL;
-     m_clientPanel = NULL;
-   };
-   
+  page->m_client->SetSize( alloc->x, alloc->y+26,
+                           alloc->width, alloc->height );

 };
 
 //-----------------------------------------------------------------------------
 };
 
 //-----------------------------------------------------------------------------


@@ -49,119 +94,117 @@ class wxNotebookPage: public wxObject
 //-----------------------------------------------------------------------------
 
 BEGIN_EVENT_TABLE(wxNotebook, wxControl)
 //-----------------------------------------------------------------------------
 
 BEGIN_EVENT_TABLE(wxNotebook, wxControl)

+  EVT_SIZE(wxNotebook::OnSize)

 END_EVENT_TABLE()
 
 IMPLEMENT_DYNAMIC_CLASS(wxNotebook,wxControl)
 
 END_EVENT_TABLE()
 
 IMPLEMENT_DYNAMIC_CLASS(wxNotebook,wxControl)
 

-wxNotebook::wxNotebook(void)
+void wxNotebook::Init()

 {
   m_imageList = NULL;
 {
   m_imageList = NULL;

-  m_frame = NULL;

   m_pages.DeleteContents( TRUE );
   m_pages.DeleteContents( TRUE );

+  m_idHandler = 0;
+}
+
+wxNotebook::wxNotebook()
+{
+  Init();

 };
 
 };
 

-wxNotebook::wxNotebook( wxWindow *parent, const wxWindowID id, 
+wxNotebook::wxNotebook( wxWindow *parent, wxWindowID id,

       const wxPoint& pos, const wxSize& size,
       const wxPoint& pos, const wxSize& size,

-      const long style, const wxString& name )
+      long style, const wxString& name )

 {
 {

-  m_imageList = NULL;
-  m_frame = NULL;
-  m_pages.DeleteContents( TRUE );
+  Init();

   Create( parent, id, pos, size, style, name );
 };
 
   Create( parent, id, pos, size, style, name );
 };
 

-wxNotebook::~wxNotebook(void)
+wxNotebook::~wxNotebook()

 {
 {

-  if (m_imageList) delete m_imageList;
+  // don't generate change page events any more
+  if ( m_idHandler != 0 )
+    gtk_signal_disconnect(GTK_OBJECT(m_widget), m_idHandler);
+
+  if (m_imageList)
+    delete m_imageList;

   DeleteAllPages();
 };
 
   DeleteAllPages();
 };
 

-bool wxNotebook::Create(wxWindow *parent, const wxWindowID id, 
+bool wxNotebook::Create(wxWindow *parent, wxWindowID id,

       const wxPoint& pos, const wxSize& size,
       const wxPoint& pos, const wxSize& size,

-      const long style, const wxString& name )
+      long style, const wxString& name )

 {
   m_needParent = TRUE;
 {
   m_needParent = TRUE;

-  
+

   PreCreation( parent, id, pos, size, style, name );
 
   m_widget = gtk_notebook_new();
   PreCreation( parent, id, pos, size, style, name );
 
   m_widget = gtk_notebook_new();

-  
+  m_idHandler = gtk_signal_connect
+                (
+                  GTK_OBJECT(m_widget), "switch_page",
+                  GTK_SIGNAL_FUNC(gtk_notebook_page_change_callback),
+                  (gpointer)this
+                );
+

   PostCreation();
   PostCreation();

-  
+

   Show( TRUE );
   Show( TRUE );

-  
+

   return TRUE;
 };
 
   return TRUE;
 };
 

-int wxNotebook::GetSelection(void) const
+int wxNotebook::GetSelection() const

 {
 {

-  if (m_pages.Number() == 0) return -1;
+  if (m_pages.Number() == 0)
+    return -1;

 
   GtkNotebookPage *g_page = GTK_NOTEBOOK(m_widget)->cur_page;
 
   GtkNotebookPage *g_page = GTK_NOTEBOOK(m_widget)->cur_page;

-  
+

   wxNotebookPage *page = NULL;
 
   wxNode *node = m_pages.First();
   while (node)
   {
     page = (wxNotebookPage*)node->Data();
   wxNotebookPage *page = NULL;
 
   wxNode *node = m_pages.First();
   while (node)
   {
     page = (wxNotebookPage*)node->Data();

-    if (page->m_page == g_page) break;
+    if (page->m_page == g_page)
+      break;

     node = node->Next();
   };
     node = node->Next();
   };

-  
-  if (!node) wxFatalError( "Notebook error." );
-  
-  return page->m_id;
-};

 
 

-wxImageList* wxNotebook::GetImageList(void) const
-{
-  return m_imageList;
+  wxCHECK_MSG( node != NULL, -1, "wxNotebook: no selection?");
+
+  return page->m_id;

 };
 
 };
 

-int wxNotebook::GetPageCount(void) const
+int wxNotebook::GetPageCount() const

 {
   return m_pages.Number();
 };
 
 {
   return m_pages.Number();
 };
 

-int wxNotebook::GetRowCount(void) const
+int wxNotebook::GetRowCount() const

 {
   return 1;
 };
 
 {
   return 1;
 };
 

-wxString wxNotebook::GetPageText( const int page ) const
+wxString wxNotebook::GetPageText( int page ) const

 {
 {

-  wxNotebookPage *nb_page = NULL;
-
-  wxNode *node = m_pages.First();
-  while (node)
-  {
-    nb_page = (wxNotebookPage*)node->Data();
-    if (nb_page->m_id == page) break;
-    node = node->Next();
-  };
-  if (!node) return "";
-  
-  return nb_page->m_text;
+  wxNotebookPage* nb_page = GetNotebookPage(page);
+  if (nb_page)
+    return nb_page->m_text;
+  else
+    return "";

 };
 
 };
 

-int wxNotebook::GetPageImage( const int page ) const
+int wxNotebook::GetPageImage( int page ) const

 {
 {

-  wxNotebookPage *nb_page = NULL;
-
-  wxNode *node = m_pages.First();
-  while (node)
-  {
-    nb_page = (wxNotebookPage*)node->Data();
-    if (nb_page->m_id == page) break;
-    node = node->Next();
-  };
-  if (!node) return -1;
-  
-  return nb_page->m_image;
+  wxNotebookPage* nb_page = GetNotebookPage(page);
+  if (nb_page)
+    return nb_page->m_image;
+  else
+    return 0;

 };
 
 };
 

-void* wxNotebook::GetPageData( const int page ) const
+wxNotebookPage* wxNotebook::GetNotebookPage(int page) const

 {
   wxNotebookPage *nb_page = NULL;
 
 {
   wxNotebookPage *nb_page = NULL;
 


@@ -169,142 +212,109 @@ void* wxNotebook::GetPageData( const int page ) const
   while (node)
   {
     nb_page = (wxNotebookPage*)node->Data();
   while (node)
   {
     nb_page = (wxNotebookPage*)node->Data();

-    if (nb_page->m_id == page) break;
+    if (nb_page->m_id == page)
+      return nb_page;

     node = node->Next();
   };
     node = node->Next();
   };

-  if (!node) return NULL;
-  
-  return nb_page->m_clientData;
+
+  wxLogDebug( "Notebook page %d not found!", page );
+
+  return NULL;

 };
 
 };
 

-int wxNotebook::SetSelection( const int page )
+int wxNotebook::SetSelection( int page )

 {
 {

-  wxNotebookPage *nb_page = NULL;
+  int selOld = GetSelection();
+  wxNotebookPage* nb_page = GetNotebookPage(page);
+  if (!nb_page)
+    return -1;

 
 

-  wxNode *node = m_pages.First();
-  while (node)
-  {
-    nb_page = (wxNotebookPage*)node->Data();
-    if (nb_page->m_id == page) break;
-    node = node->Next();
-  };
-  if (!node) return -1;
-  

   int page_num = 0;
   GList *child = GTK_NOTEBOOK(m_widget)->children;
   while (child)
   {
   int page_num = 0;
   GList *child = GTK_NOTEBOOK(m_widget)->children;
   while (child)
   {

-    if (nb_page->m_page == (GtkNotebookPage*)child->data) break;
+    if (nb_page->m_page == (GtkNotebookPage*)child->data)
+      break;

     page_num++;
     child = child->next;
   };
     page_num++;
     child = child->next;
   };

-   
+

   if (!child) return -1;
   if (!child) return -1;

-  
+

   gtk_notebook_set_page( GTK_NOTEBOOK(m_widget), page_num );
   gtk_notebook_set_page( GTK_NOTEBOOK(m_widget), page_num );

-  
-  return page;
+
+  return selOld;

 };
 
 };
 

+void wxNotebook::AdvanceSelection(bool bForward)
+{
+  int nSel = GetSelection(),
+      nMax = GetPageCount();
+
+  if ( bForward ) {
+    SetSelection(nSel == nMax ? 0 : nSel + 1);
+  }
+  else {
+    SetSelection(nSel == 0 ? nMax : nSel - 1);
+  }
+}
+

 void wxNotebook::SetImageList( wxImageList* imageList )
 {
   m_imageList = imageList;
 };
 
 void wxNotebook::SetImageList( wxImageList* imageList )
 {
   m_imageList = imageList;
 };
 

-bool wxNotebook::SetPageText( const int page, const wxString &text )
+bool wxNotebook::SetPageText( int page, const wxString &text )

 {
 {

-  wxNotebookPage *nb_page = NULL;
+  wxNotebookPage* nb_page = GetNotebookPage(page);
+  if (!nb_page)
+    return FALSE;

 
 

-  wxNode *node = m_pages.First();
-  while (node)
-  {
-    nb_page = (wxNotebookPage*)node->Data();
-    if (nb_page->m_id == page) break;
-    node = node->Next();
-  };
-  if (!node) return FALSE;
-  

   nb_page->m_text = text;
   nb_page->m_text = text;

- 
-  // recreate
-   
+

   return TRUE;
 };
 
   return TRUE;
 };
 

-bool wxNotebook::SetPageImage( const int page, const int image )
+bool wxNotebook::SetPageImage( int page, int image )

 {
 {

-  wxNotebookPage *nb_page = NULL;
+  wxNotebookPage* nb_page = GetNotebookPage(page);
+  if (!nb_page)
+    return FALSE;

 
 

-  wxNode *node = m_pages.First();
-  while (node)
-  {
-    nb_page = (wxNotebookPage*)node->Data();
-    if (nb_page->m_id == page) break;
-    node = node->Next();
-  };
-  if (!node) return FALSE;
-  

   nb_page->m_image = image;
   nb_page->m_image = image;

- 
-  // recreate
-   
-  return TRUE;
-};

 
 

-bool wxNotebook::SetPageData( const int page, void* data )
-{
-  wxNotebookPage *nb_page = NULL;
-
-  wxNode *node = m_pages.First();
-  while (node)
-  {
-    nb_page = (wxNotebookPage*)node->Data();
-    if (nb_page->m_id == page) break;
-    node = node->Next();
-  };
-  if (!node) return FALSE;
-  
-  nb_page->m_clientData = data;
-  

   return TRUE;
 };
 
 void wxNotebook::SetPageSize( const wxSize &WXUNUSED(size) )
 {
   return TRUE;
 };
 
 void wxNotebook::SetPageSize( const wxSize &WXUNUSED(size) )
 {

+  wxFAIL_MSG("wxNotebook::SetPageSize not implemented");

 };
 
 void wxNotebook::SetPadding( const wxSize &WXUNUSED(padding) )
 {
 };
 
 void wxNotebook::SetPadding( const wxSize &WXUNUSED(padding) )
 {

- //  what's this ?
+  wxFAIL_MSG("wxNotebook::SetPadding not implemented");

 };
 
 };
 

-bool wxNotebook::DeleteAllPages(void)
+bool wxNotebook::DeleteAllPages()

 {
   wxNode *page_node = m_pages.First();
   while (page_node)
   {
     wxNotebookPage *page = (wxNotebookPage*)page_node->Data();
 {
   wxNode *page_node = m_pages.First();
   while (page_node)
   {
     wxNotebookPage *page = (wxNotebookPage*)page_node->Data();

-    
+

     DeletePage( page->m_id );
     DeletePage( page->m_id );

-    
+

     page_node = m_pages.First();
   };
     page_node = m_pages.First();
   };

-  
+

   return TRUE;
 };
 
   return TRUE;
 };
 

-bool wxNotebook::DeletePage( const int page )
+bool wxNotebook::DeletePage( int page )

 {
 {

-  wxNotebookPage *nb_page = NULL;
-
-  wxNode *node = m_pages.First();
-  while (node)
-  {
-    nb_page = (wxNotebookPage*)node->Data();
-    if (nb_page->m_id == page) break;
-    node = node->Next();
-  };
-  if (!node) return FALSE;
+  wxNotebookPage* nb_page = GetNotebookPage(page);
+  if (!nb_page) return FALSE;

 
   int page_num = 0;
   GList *child = GTK_NOTEBOOK(m_widget)->children;
 
   int page_num = 0;
   GList *child = GTK_NOTEBOOK(m_widget)->children;


@@ -314,71 +324,101 @@ bool wxNotebook::DeletePage( const int page )
     page_num++;
     child = child->next;
   };
     page_num++;
     child = child->next;
   };

-   
-  if (!child) wxFatalError( "Notebook delete error" );;
-  
-  gtk_notebook_remove_page( GTK_NOTEBOOK(m_widget), page_num );
-  
-  m_pages.DeleteObject( nb_page );
-    
-  return TRUE;
-};

 
 

-wxPanel *wxNotebook::CreatePage( const int item, const wxString &text, const int imageId, void* data )
-{
-  wxNotebookPage *page = new wxNotebookPage;
-  
-  page->m_text = text;
-  if (page->m_text.IsNull()) page->m_text = "";
-  page->m_id = item;
-  page->m_image = imageId;
-  page->m_clientData = data;
-  
-  m_frame = gtk_label_new( page->m_text );
-  gtk_misc_set_alignment( GTK_MISC(m_frame), 0.0, 0.5 );
+  wxASSERT( child );

 
 

-  page->m_clientPanel = new wxPanel( this, -1, wxPoint(0,0), wxSize(100,100) );
-    
-  m_frame = NULL;
-  
-  page->m_page = GTK_NOTEBOOK(m_widget)->cur_page;
-  
-  m_pages.Append( page );
+  delete nb_page->m_client;
+
+//  Amazingly, this is not necessary
+//  gtk_notebook_remove_page( GTK_NOTEBOOK(m_widget), page_num );

 
 

-  return page->m_clientPanel;
+  m_pages.DeleteObject( nb_page );
+
+  return TRUE;

 };
 
 };
 

-wxPanel *wxNotebook::GetPanel( const int page )
+bool wxNotebook::AddPage(wxWindow* win, const wxString& text,
+                         bool bSelect, int imageId)

 {
 {

-  wxNotebookPage *nb_page = NULL;
+  // we've created the notebook page in AddChild(). Now we just have to set
+  // the caption for the page and set the others parameters.
+
+  // first, find the page
+  wxNotebookPage *page = NULL;

 
   wxNode *node = m_pages.First();
   while (node)
   {
 
   wxNode *node = m_pages.First();
   while (node)
   {

-    nb_page = (wxNotebookPage*)node->Data();
-    if (nb_page->m_id == page) break;
+    page = (wxNotebookPage*)node->Data();
+    if ( page->m_client == win )
+      break; // found

     node = node->Next();
   };
     node = node->Next();
   };

-  if (!node) return NULL;
-  
-  return nb_page->m_clientPanel;
+
+  wxCHECK_MSG(page != NULL, FALSE,
+              "Can't add a page whose parent is not the notebook!");
+
+  // then set the attributes
+  page->m_text = text;
+  if ( page->m_text.IsEmpty() )
+    page->m_text = "";
+  page->m_image = imageId;
+  gtk_label_set(page->m_label, page->m_text);
+
+  if ( bSelect ) {
+    SetSelection(GetPageCount());
+  }
+
+  return TRUE;
+};
+
+wxWindow *wxNotebook::GetPage( int page ) const
+{
+  wxNotebookPage* nb_page = GetNotebookPage(page);
+  if (!nb_page)
+    return NULL;
+  else
+    return nb_page->m_client;

 };
 
 void wxNotebook::AddChild( wxWindow *win )
 {
 };
 
 void wxNotebook::AddChild( wxWindow *win )
 {

-  if (!m_frame) wxFatalError( "Notebook::Addchild() must not be called directly." );
+  // @@@ normally done in wxWindow::AddChild but for some reason wxNotebook
+  // case is special there (Robert?)
+  // Robert: Don't you think the code below looks different from the one
+  // in wxWindow::AddChild :-)

   
   

-  gtk_notebook_append_page( GTK_NOTEBOOK(m_widget), win->m_widget, m_frame );
-};
+  m_children.Append(win);

 
 

-//-----------------------------------------------------------------------------
-// wxTabEvent
-//-----------------------------------------------------------------------------
+  
+  wxNotebookPage *page = new wxNotebookPage();

 
 

-IMPLEMENT_DYNAMIC_CLASS(wxTabEvent, wxCommandEvent)
+  page->m_id = GetPageCount();
+  page->m_label = (GtkLabel *)gtk_label_new("Handle");
+  page->m_client = win;
+  gtk_notebook_append_page( GTK_NOTEBOOK(m_widget), win->m_widget,
+                            (GtkWidget *)page->m_label);
+  gtk_misc_set_alignment(GTK_MISC(page->m_label), 0.0, 0.5);

 
 

-wxTabEvent::wxTabEvent( WXTYPE commandType, int id ) :
-  wxCommandEvent(commandType, id)
-{
+  page->m_page = 
+    (GtkNotebookPage*) (g_list_last(GTK_NOTEBOOK(m_widget)->children)->data);
+    
+  page->m_parent = GTK_NOTEBOOK(m_widget);
+
+  gtk_signal_connect( GTK_OBJECT(m_widget), "size_allocate",
+    GTK_SIGNAL_FUNC(gtk_notebook_client_size_callback), (gpointer)page );
+    
+  if (!page->m_page)
+  {
+     wxLogFatalError( "Notebook page creation error" );
+     return;
+  }
+
+  m_pages.Append( page );

 };
 
 };
 

+//-----------------------------------------------------------------------------
+// wxNotebookEvent
+//-----------------------------------------------------------------------------
+
+IMPLEMENT_DYNAMIC_CLASS(wxNotebookEvent, wxCommandEvent)