wxImageList ctor now takes the same arguments as the MSW version (but it's

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

diff --git a/src/gtk1/notebook.cpp b/src/gtk1/notebook.cpp
index 54ad8ca5eea334edcaf56d2caa05bf4c0b53e477..0c0f939051a3fb7062328026becf0468e2b4ccd5 100644 (file)
--- a/src/gtk1/notebook.cpp
+++ b/src/gtk1/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,31 @@
 #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"
+
+//-----------------------------------------------------------------------------
+// GTK callbacks
+//-----------------------------------------------------------------------------
+
+// page change callback
+static void gtk_notebook_page_change_callback(GtkNotebook *widget,
+                                              GtkNotebookPage *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);
+}

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


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

-  public:
-  
-   int                m_id;
-   wxString           m_text;
-   int                m_image;
-   void              *m_clientData;
-   GtkNotebookPage   *m_page;
-   wxWindow          *m_clientPanel;
-   
-   wxNotebookPage(void)
-   {
-     m_id = -1;
-     m_text = "";
-     m_image = -1;
-     m_clientData = NULL;
-     m_page = NULL;
-     m_clientPanel = NULL;
-   };
-   
+public:
+  wxNotebookPage()
+  {
+    m_id = -1;
+    m_text = "";
+    m_image = -1;
+    m_page = NULL;
+    m_clientPanel = NULL;
+  };
+
+//private:
+  int                m_id;
+  wxString           m_text;
+  int                m_image;
+  GtkNotebookPage   *m_page;
+  GtkLabel          *m_label;
+  wxWindow          *m_clientPanel;

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


@@ -49,87 +72,99 @@ 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, const wxWindowID id,

       const wxPoint& pos, const wxSize& size,
       const long style, const wxString& name )
 {
       const wxPoint& pos, const wxSize& size,
       const 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, const wxWindowID id,

       const wxPoint& pos, const wxSize& size,
       const long style, const wxString& name )
 {
   m_needParent = TRUE;
       const wxPoint& pos, const wxSize& size,
       const long style, const wxString& name )
 {
   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 = GetNotebookPage(page);
   if (nb_page)
 {
   wxNotebookPage* nb_page = GetNotebookPage(page);
   if (nb_page)


@@ -138,22 +173,13 @@ wxString wxNotebook::GetPageText( const int page ) const
     return "";
 };
 
     return "";
 };
 

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

 {
   wxNotebookPage* nb_page = GetNotebookPage(page);
   if (nb_page)
     return nb_page->m_image;
   else
 {
   wxNotebookPage* nb_page = GetNotebookPage(page);
   if (nb_page)
     return nb_page->m_image;
   else

-    return NULL;
-};
-
-void* wxNotebook::GetPageData( const int page ) const
-{
-  wxNotebookPage* nb_page = GetNotebookPage(page);
-  if (nb_page)
-    return nb_page->m_clientData;
-  else
-    return NULL;
+    return 0;

 };
 
 wxNotebookPage* wxNotebook::GetNotebookPage(int page) const
 };
 
 wxNotebookPage* wxNotebook::GetNotebookPage(int page) const


@@ -164,99 +190,106 @@ wxNotebookPage* wxNotebook::GetNotebookPage(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;
+
+  wxLogDebug("Notebook page %d not found!", page);
+
+  return NULL;

 };
 
 };
 

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

 {
 {

+  int selOld = GetSelection();

   wxNotebookPage* nb_page = GetNotebookPage(page);
   wxNotebookPage* nb_page = GetNotebookPage(page);

-  if (!nb_page) return -1;
-  
+  if (!nb_page)
+    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 = GetNotebookPage(page);
 {
   wxNotebookPage* nb_page = GetNotebookPage(page);

-  if (!nb_page) return FALSE;
-  
+  if (!nb_page)
+    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, const int image )

 {
   wxNotebookPage* nb_page = GetNotebookPage(page);
   if (!nb_page)
     return FALSE;
 {
   wxNotebookPage* nb_page = GetNotebookPage(page);
   if (!nb_page)
     return FALSE;

-  
-  nb_page->m_image = image;
- 
-  // recreate
-   
-  return TRUE;
-};

 
 

-bool wxNotebook::SetPageData( const int page, void* data )
-{
-  wxNotebookPage* nb_page = GetNotebookPage(page);
-  if (!nb_page) return FALSE;
+  nb_page->m_image = image;

 
 

-  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 = GetNotebookPage(page);
   if (!nb_page) return FALSE;
 {
   wxNotebookPage* nb_page = GetNotebookPage(page);
   if (!nb_page) return FALSE;


@@ -271,73 +304,108 @@ bool wxNotebook::DeletePage( const int page )
   };
 
   wxASSERT( child );
   };
 
   wxASSERT( child );

-        
+

   delete nb_page->m_clientPanel;
   delete nb_page->m_clientPanel;

-  
+

 //  Amazingly, this is not necessary
 //  gtk_notebook_remove_page( GTK_NOTEBOOK(m_widget), page_num );
 //  Amazingly, this is not necessary
 //  gtk_notebook_remove_page( GTK_NOTEBOOK(m_widget), page_num );

-  
+

   m_pages.DeleteObject( nb_page );
   m_pages.DeleteObject( nb_page );

-    
+

   return TRUE;
 };
 
   return TRUE;
 };
 

-bool wxNotebook::AddPage( const int item, const wxString &text, wxWindow* win, const int imageId, void* data )
+bool wxNotebook::AddPage(wxWindow* win, const wxString& text,
+                         bool bSelect, int imageId)

 {
 {

-  wxNotebookPage *page = new wxNotebookPage;
+  // 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)
+  {
+    page = (wxNotebookPage*)node->Data();
+    if ( page->m_clientPanel == win )
+      break; // found
+    node = node->Next();
+  };
+
+  wxCHECK_MSG(page != NULL, FALSE,
+              "Can't add a page whose parent is not the notebook!");
+
+  // then set the attributes

   page->m_text = text;
   page->m_text = text;

-  if (page->m_text.IsNull()) page->m_text = "";
-  page->m_id = item;
+  if ( page->m_text.IsEmpty() )
+    page->m_text = "";

   page->m_image = imageId;
   page->m_image = imageId;

-  page->m_clientData = data;
-  
-  m_frame = gtk_label_new( page->m_text );
-  gtk_notebook_append_page( GTK_NOTEBOOK(m_widget), win->m_widget, m_frame );
-
-  gtk_misc_set_alignment( GTK_MISC(m_frame), 0.0, 0.5 );
+  gtk_label_set(page->m_label, page->m_text);

 
 

-  page->m_clientPanel = win;
-//  page->m_clientPanel = new wxPanel( this, -1, wxPoint(0,0), wxSize(100,100) );
-    
-  m_frame = NULL;
-  
-  page->m_page = (GtkNotebookPage*)( g_list_last( GTK_NOTEBOOK(m_widget)->children )->data );
-  
-  if (!page->m_page)
-  {
-     wxFatalError( "Notebook page creation error" );
-     return FALSE;
+  if ( bSelect ) {
+    SetSelection(GetPageCount());

   }
 
   }
 

-  m_pages.Append( page );
-

   return TRUE;
 };
 
   return TRUE;
 };
 

-wxWindow *wxNotebook::GetPageWindow( const int page ) const
+wxWindow *wxNotebook::GetPage( int page ) const

 {
   wxNotebookPage* nb_page = GetNotebookPage(page);
 {
   wxNotebookPage* nb_page = GetNotebookPage(page);

-  if (!nb_page) return NULL;
-  
-  return nb_page->m_clientPanel;
+  if (!nb_page)
+    return NULL;
+  else
+    return nb_page->m_clientPanel;

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

-  if (!m_frame) wxFatalError( "Notebook::Addchild() must not be called directly." );
-  
-//  gtk_notebook_append_page( GTK_NOTEBOOK(m_widget), win->m_widget, m_frame );
-};
+  // @@@ normally done in wxWindow::AddChild but for some reason wxNotebook
+  // case is special there (Robert?)
+  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("no caption");
+  page->m_clientPanel = 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
+                 );
+
+  if (!page->m_page)
+  {
+     wxLogFatalError( "Notebook page creation error" );
+     return;
+  }
+
+  m_pages.Append( page );

 };
 
 };
 

+void wxNotebook::OnSize(wxSizeEvent& event)
+{
+  // forward this event to all pages
+  wxNode *node = m_pages.First();
+  while (node)
+  {
+    wxNotebookPage *page = (wxNotebookPage*)node->Data();
+    // @@@@ This -50 is completely wrong - instead, we should substract
+    //      the height of the tabs
+    page->m_clientPanel->SetSize(event.GetSize().GetX(),
+                                 event.GetSize().GetY() - 50);
+
+    node = node->Next();
+  };
+}
+
+//-----------------------------------------------------------------------------
+// wxNotebookEvent
+//-----------------------------------------------------------------------------
+
+IMPLEMENT_DYNAMIC_CLASS(wxNotebookEvent, wxCommandEvent)