src/gtk/notebook.cpp

   1 /////////////////////////////////////////////////////////////////////////////
   2 // Name:        src/gtk/notebook.cpp
   3 // Purpose:
   4 // Author:      Robert Roebling
   5 // Id:          $Id$
   6 // Copyright:   (c) 1998 Robert Roebling, Vadim Zeitlin
   7 // Licence:     wxWindows licence
   8 /////////////////////////////////////////////////////////////////////////////
   9
  10 // For compilers that support precompilation, includes "wx.h".
  11 #include "wx/wxprec.h"
  12
  13 #if wxUSE_NOTEBOOK
  14
  15 #include "wx/notebook.h"
  16
  17 #ifndef WX_PRECOMP
  18     #include "wx/intl.h"
  19     #include "wx/log.h"
  20     #include "wx/utils.h"
  21     #include "wx/panel.h"
  22     #include "wx/msgdlg.h"
  23     #include "wx/bitmap.h"
  24 #endif
  25
  26 #include "wx/imaglist.h"
  27 #include "wx/fontutil.h"
  28
  29 // FIXME: Use GtkImage instead of GtkPixmap. Don't use gtk_container_border_width
  30 #include <gtk/gtkversion.h>
  31 #ifdef GTK_DISABLE_DEPRECATED
  32 #undef GTK_DISABLE_DEPRECATED
  33 #endif
  34
  35 #include "wx/gtk/private.h"
  36 #include "wx/gtk/win_gtk.h"
  37
  38 #include <gdk/gdkkeysyms.h>
  39
  40 // ----------------------------------------------------------------------------
  41 // events
  42 // ----------------------------------------------------------------------------
  43
  44 DEFINE_EVENT_TYPE(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGED)
  45 DEFINE_EVENT_TYPE(wxEVT_COMMAND_NOTEBOOK_PAGE_CHANGING)
  46
  47 //-----------------------------------------------------------------------------
  48 // data
  49 //-----------------------------------------------------------------------------
  50
  51 extern bool g_blockEventsOnDrag;
  52
  53 //-----------------------------------------------------------------------------
  54 // wxGtkNotebookPage
  55 //-----------------------------------------------------------------------------
  56
  57 // VZ: this is rather ugly as we keep the pages themselves in an array (it
  58 //     allows us to have quite a few functions implemented in the base class)
  59 //     but the page data is kept in a separate list, so we must maintain them
  60 //     in sync manually... of course, the list had been there before the base
  61 //     class which explains it but it still would be nice to do something
  62 //     about this one day
  63
  64 class wxGtkNotebookPage: public wxObject
  65 {
  66 public:
  67     wxGtkNotebookPage()
  68     {
  69         m_image = -1;
  70         m_page = (GtkNotebookPage *) NULL;
  71         m_box = (GtkWidget *) NULL;
  72     }
  73
  74     wxString           m_text;
  75     int                m_image;
  76     GtkNotebookPage   *m_page;
  77     GtkLabel          *m_label;
  78     GtkWidget         *m_box;     // in which the label and image are packed
  79 };
  80
  81
  82 #include "wx/listimpl.cpp"
  83 WX_DEFINE_LIST(wxGtkNotebookPagesList)
  84
  85
  86 //-----------------------------------------------------------------------------
  87 // "switch_page"
  88 //-----------------------------------------------------------------------------
  89
  90 extern "C" {
  91 static void gtk_notebook_page_change_callback(GtkNotebook *WXUNUSED(widget),
  92                                               GtkNotebookPage *WXUNUSED(gpage),
  93                                               guint page,
  94                                               wxNotebook *notebook )
  95 {
  96     // are you trying to call SetSelection() from a notebook event handler?
  97     // you shouldn't!
  98     wxCHECK_RET( !notebook->m_inSwitchPage,
  99                  _T("gtk_notebook_page_change_callback reentered") );
 100
 101     notebook->m_inSwitchPage = true;
 102     if (g_isIdle)
 103         wxapp_install_idle_handler();
 104
 105     int old = notebook->GetSelection();
 106
 107
 108     if (notebook->m_skipNextPageChangeEvent)
 109     {
 110         // this event was programatically generated by ChangeSelection() and thus must
 111         // be skipped
 112         notebook->m_skipNextPageChangeEvent = false;
 113
 114         // make wxNotebook::GetSelection() return the correct (i.e. consistent
 115         // with wxNotebookEvent::GetSelection()) value even though the page is
 116         // not really changed in GTK+
 117         notebook->m_selection = page;
 118     }
 119     else
 120     {
 121         if ( !notebook->SendPageChangingEvent(page) )
 122         {
 123             // program doesn't allow the page change
 124             g_signal_stop_emission_by_name(notebook->m_widget, "switch_page");
 125         }
 126         else // change allowed
 127         {
 128             // make wxNotebook::GetSelection() return the correct (i.e. consistent
 129             // with wxNotebookEvent::GetSelection()) value even though the page is
 130             // not really changed in GTK+
 131             notebook->m_selection = page;
 132
 133             notebook->SendPageChangedEvent(old);
 134         }
 135     }
 136
 137     notebook->m_inSwitchPage = false;
 138 }
 139 }
 140
 141 //-----------------------------------------------------------------------------
 142 // "size_allocate"
 143 //-----------------------------------------------------------------------------
 144
 145 extern "C" {
 146 static void gtk_page_size_callback( GtkWidget *WXUNUSED(widget), GtkAllocation* alloc, wxWindow *win )
 147 {
 148     if (g_isIdle)
 149         wxapp_install_idle_handler();
 150
 151     if ((win->m_x == alloc->x) &&
 152         (win->m_y == alloc->y) &&
 153         (win->m_width == alloc->width) &&
 154         (win->m_height == alloc->height))
 155     {
 156         return;
 157     }
 158
 159     win->SetSize( alloc->x, alloc->y, alloc->width, alloc->height );
 160
 161     /* GTK 1.2 up to version 1.2.5 is broken so that we have to call allocate
 162        here in order to make repositioning after resizing to take effect. */
 163     if ((gtk_major_version == 1) &&
 164         (gtk_minor_version == 2) &&
 165         (gtk_micro_version < 6) &&
 166         (win->m_wxwindow) &&
 167         (GTK_WIDGET_REALIZED(win->m_wxwindow)))
 168     {
 169         gtk_widget_size_allocate( win->m_wxwindow, alloc );
 170     }
 171 }
 172 }
 173
 174 //-----------------------------------------------------------------------------
 175 // "realize" from m_widget
 176 //-----------------------------------------------------------------------------
 177
 178 extern "C" {
 179 static void
 180 gtk_notebook_realized_callback( GtkWidget * WXUNUSED(widget), wxWindow *win )
 181 {
 182     if (g_isIdle)
 183         wxapp_install_idle_handler();
 184
 185     /* GTK 1.2 up to version 1.2.5 is broken so that we have to call a queue_resize
 186        here in order to make repositioning before showing to take effect. */
 187     gtk_widget_queue_resize( win->m_widget );
 188 }
 189 }
 190
 191 //-----------------------------------------------------------------------------
 192 // "key_press_event"
 193 //-----------------------------------------------------------------------------
 194
 195 extern "C" {
 196 static gboolean
 197 gtk_notebook_key_press_callback( GtkWidget   *widget,
 198                                  GdkEventKey *gdk_event,
 199                                  wxNotebook  *notebook )
 200 {
 201     // don't need to install idle handler, its done from "event" signal
 202
 203     if (!notebook->m_hasVMT) return FALSE;
 204     if (g_blockEventsOnDrag) return FALSE;
 205
 206     /* win is a control: tab can be propagated up */
 207     if ((gdk_event->keyval == GDK_Left) || (gdk_event->keyval == GDK_Right))
 208     {
 209         int page;
 210         int nMax = notebook->GetPageCount();
 211         if ( nMax-- ) // decrement it to get the last valid index
 212         {
 213             int nSel = notebook->GetSelection();
 214
 215             // change selection wrapping if it becomes invalid
 216             page = (gdk_event->keyval != GDK_Left) ? nSel == nMax ? 0
 217                                        : nSel + 1
 218                         : nSel == 0 ? nMax
 219                                     : nSel - 1;
 220         }
 221         else // notebook is empty, no next page
 222         {
 223             return FALSE;
 224         }
 225
 226         // m_selection = page;
 227         gtk_notebook_set_current_page( GTK_NOTEBOOK(widget), page );
 228
 229         return TRUE;
 230     }
 231
 232     /* win is a control: tab can be propagated up */
 233     if ((gdk_event->keyval == GDK_Tab) || (gdk_event->keyval == GDK_ISO_Left_Tab))
 234     {
 235         int sel = notebook->GetSelection();
 236         if (sel == -1)
 237             return TRUE;
 238         wxGtkNotebookPage *nb_page = notebook->GetNotebookPage(sel);
 239         wxCHECK_MSG( nb_page, FALSE, _T("invalid selection in wxNotebook") );
 240
 241         wxNavigationKeyEvent event;
 242         event.SetEventObject( notebook );
 243         /* GDK reports GDK_ISO_Left_Tab for SHIFT-TAB */
 244         event.SetDirection( (gdk_event->keyval == GDK_Tab) );
 245         /* CTRL-TAB changes the (parent) window, i.e. switch notebook page */
 246         event.SetWindowChange( (gdk_event->state & GDK_CONTROL_MASK) ||
 247                                (gdk_event->keyval == GDK_Left) || (gdk_event->keyval == GDK_Right) );
 248         event.SetCurrentFocus( notebook );
 249
 250         wxNotebookPage *client = notebook->GetPage(sel);
 251         if ( !client->GetEventHandler()->ProcessEvent( event ) )
 252         {
 253              client->SetFocus();
 254         }
 255
 256         return TRUE;
 257     }
 258
 259     return FALSE;
 260 }
 261 }
 262
 263 //-----------------------------------------------------------------------------
 264 // InsertChild callback for wxNotebook
 265 //-----------------------------------------------------------------------------
 266
 267 static void wxInsertChildInNotebook( wxNotebook* parent, wxWindow* child )
 268 {
 269     // Hack Alert! (Part I): This sets the notebook as the parent of the child
 270     // widget, and takes care of some details such as updating the state and
 271     // style of the child to reflect its new location.  We do this early
 272     // because without it GetBestSize (which is used to set the initial size
 273     // of controls if an explicit size is not given) will often report
 274     // incorrect sizes since the widget's style context is not fully known.
 275     // See bug #901694 for details
 276     // (http://sourceforge.net/tracker/?func=detail&aid=901694&group_id=9863&atid=109863)
 277     gtk_widget_set_parent(child->m_widget, parent->m_widget);
 278
 279     // NOTE: This should be considered a temporary workaround until we can
 280     // work out the details and implement delaying the setting of the initial
 281     // size of widgets until the size is really needed.
 282 }
 283
 284 //-----------------------------------------------------------------------------
 285 // wxNotebook
 286 //-----------------------------------------------------------------------------
 287
 288 IMPLEMENT_DYNAMIC_CLASS(wxNotebook,wxControl)
 289
 290 BEGIN_EVENT_TABLE(wxNotebook, wxControl)
 291     EVT_NAVIGATION_KEY(wxNotebook::OnNavigationKey)
 292 END_EVENT_TABLE()
 293
 294 void wxNotebook::Init()
 295 {
 296     m_padding = 0;
 297     m_inSwitchPage = false;
 298     m_skipNextPageChangeEvent = false;
 299
 300     m_imageList = (wxImageList *) NULL;
 301     m_selection = -1;
 302     m_themeEnabled = true;
 303 }
 304
 305 wxNotebook::wxNotebook()
 306 {
 307     Init();
 308 }
 309
 310 wxNotebook::wxNotebook( wxWindow *parent, wxWindowID id,
 311       const wxPoint& pos, const wxSize& size,
 312       long style, const wxString& name )
 313 {
 314     Init();
 315     Create( parent, id, pos, size, style, name );
 316 }
 317
 318 wxNotebook::~wxNotebook()
 319 {
 320     DeleteAllPages();
 321 }
 322
 323 bool wxNotebook::Create(wxWindow *parent, wxWindowID id,
 324                         const wxPoint& pos, const wxSize& size,
 325                         long style, const wxString& name )
 326 {
 327     m_needParent = true;
 328     m_acceptsFocus = true;
 329     m_insertCallback = (wxInsertChildFunction)wxInsertChildInNotebook;
 330
 331     if ( (style & wxBK_ALIGN_MASK) == wxBK_DEFAULT )
 332         style |= wxBK_TOP;
 333
 334     if (!PreCreation( parent, pos, size ) ||
 335         !CreateBase( parent, id, pos, size, style, wxDefaultValidator, name ))
 336     {
 337         wxFAIL_MSG( wxT("wxNoteBook creation failed") );
 338         return false;
 339     }
 340
 341
 342     m_widget = gtk_notebook_new();
 343
 344     gtk_notebook_set_scrollable( GTK_NOTEBOOK(m_widget), 1 );
 345
 346     g_signal_connect (m_widget, "switch_page",
 347                       G_CALLBACK (gtk_notebook_page_change_callback), this);
 348
 349     m_parent->DoAddChild( this );
 350
 351     if (m_windowStyle & wxBK_RIGHT)
 352         gtk_notebook_set_tab_pos( GTK_NOTEBOOK(m_widget), GTK_POS_RIGHT );
 353     if (m_windowStyle & wxBK_LEFT)
 354         gtk_notebook_set_tab_pos( GTK_NOTEBOOK(m_widget), GTK_POS_LEFT );
 355     if (m_windowStyle & wxBK_BOTTOM)
 356         gtk_notebook_set_tab_pos( GTK_NOTEBOOK(m_widget), GTK_POS_BOTTOM );
 357
 358     g_signal_connect (m_widget, "key_press_event",
 359                       G_CALLBACK (gtk_notebook_key_press_callback), this);
 360
 361     PostCreation(size);
 362
 363     g_signal_connect (m_widget, "realize",
 364                       G_CALLBACK (gtk_notebook_realized_callback), this);
 365
 366     return true;
 367 }
 368
 369 int wxNotebook::GetSelection() const
 370 {
 371     wxCHECK_MSG( m_widget != NULL, -1, wxT("invalid notebook") );
 372
 373     if ( m_selection == -1 )
 374     {
 375         GList *nb_pages = GTK_NOTEBOOK(m_widget)->children;
 376
 377         if (g_list_length(nb_pages) != 0)
 378         {
 379             GtkNotebook *notebook = GTK_NOTEBOOK(m_widget);
 380
 381             gpointer cur = notebook->cur_page;
 382             if ( cur != NULL )
 383             {
 384                 wxConstCast(this, wxNotebook)->m_selection =
 385                     g_list_index( nb_pages, cur );
 386             }
 387         }
 388     }
 389
 390     return m_selection;
 391 }
 392
 393 wxString wxNotebook::GetPageText( size_t page ) const
 394 {
 395     wxCHECK_MSG( m_widget != NULL, wxEmptyString, wxT("invalid notebook") );
 396
 397     wxGtkNotebookPage* nb_page = GetNotebookPage(page);
 398     if (nb_page)
 399         return nb_page->m_text;
 400     else
 401         return wxEmptyString;
 402 }
 403
 404 int wxNotebook::GetPageImage( size_t page ) const
 405 {
 406     wxCHECK_MSG( m_widget != NULL, -1, wxT("invalid notebook") );
 407
 408     wxGtkNotebookPage* nb_page = GetNotebookPage(page);
 409     if (nb_page)
 410         return nb_page->m_image;
 411     else
 412         return -1;
 413 }
 414
 415 wxGtkNotebookPage* wxNotebook::GetNotebookPage( int page ) const
 416 {
 417     wxCHECK_MSG( m_widget != NULL, (wxGtkNotebookPage*) NULL, wxT("invalid notebook") );
 418
 419     wxCHECK_MSG( page < (int)m_pagesData.GetCount(), (wxGtkNotebookPage*) NULL, wxT("invalid notebook index") );
 420
 421     return m_pagesData.Item(page)->GetData();
 422 }
 423
 424 int wxNotebook::DoSetSelection( size_t page, int flags )
 425 {
 426     wxCHECK_MSG( m_widget != NULL, -1, wxT("invalid notebook") );
 427
 428     wxCHECK_MSG( page < m_pagesData.GetCount(), -1, wxT("invalid notebook index") );
 429
 430     int selOld = GetSelection();
 431
 432     if ( !(flags & SetSelection_SendEvent) )
 433         m_skipNextPageChangeEvent = true;
 434
 435     // cache the selection
 436     m_selection = page;
 437     gtk_notebook_set_current_page( GTK_NOTEBOOK(m_widget), page );
 438
 439 #ifdef __WXDEBUG__
 440     if ( !(flags & SetSelection_SendEvent) )
 441     {
 442         // gtk_notebook_set_current_page will emit the switch-page signal which will be
 443         // caught by our gtk_notebook_page_change_callback which should have reset the
 444         // flag to false:
 445         wxASSERT(!m_skipNextPageChangeEvent);
 446     }
 447 #endif // __WXDEBUG__
 448
 449     wxNotebookPage *client = GetPage(page);
 450     if ( client )
 451         client->SetFocus();
 452
 453     return selOld;
 454 }
 455
 456 bool wxNotebook::SetPageText( size_t page, const wxString &text )
 457 {
 458     wxCHECK_MSG( m_widget != NULL, false, wxT("invalid notebook") );
 459
 460     wxGtkNotebookPage* nb_page = GetNotebookPage(page);
 461
 462     wxCHECK_MSG( nb_page, false, wxT("SetPageText: invalid page index") );
 463
 464     nb_page->m_text = text;
 465
 466     gtk_label_set_text( nb_page->m_label, wxGTK_CONV( nb_page->m_text ) );
 467
 468     return true;
 469 }
 470
 471 bool wxNotebook::SetPageImage( size_t page, int image )
 472 {
 473     /* HvdH 28-12-98: now it works, but it's a bit of a kludge */
 474
 475     wxGtkNotebookPage* nb_page = GetNotebookPage(page);
 476
 477     if (!nb_page) return false;
 478
 479     /* Optimization posibility: return immediately if image unchanged.
 480      * Not enabled because it may break existing (stupid) code that
 481      * manipulates the imagelist to cycle images */
 482
 483     /* if (image == nb_page->m_image) return true; */
 484
 485     /* For different cases:
 486        1) no image -> no image
 487        2) image -> no image
 488        3) no image -> image
 489        4) image -> image */
 490
 491     if (image == -1 && nb_page->m_image == -1)
 492         return true; /* Case 1): Nothing to do. */
 493
 494     GtkWidget *pixmapwid = (GtkWidget*) NULL;
 495
 496     if (nb_page->m_image != -1)
 497     {
 498         /* Case 2) or 4). There is already an image in the gtkhbox. Let's find it */
 499
 500         GList *child = gtk_container_get_children(GTK_CONTAINER(nb_page->m_box));
 501         while (child)
 502         {
 503             if (GTK_IS_PIXMAP(child->data))
 504             {
 505                 pixmapwid = GTK_WIDGET(child->data);
 506                 break;
 507             }
 508             child = child->next;
 509         }
 510
 511         /* We should have the pixmap widget now */
 512         wxASSERT(pixmapwid != NULL);
 513
 514         if (image == -1)
 515         {
 516             /* If there's no new widget, just remove the old from the box */
 517             gtk_container_remove(GTK_CONTAINER(nb_page->m_box), pixmapwid);
 518             nb_page->m_image = -1;
 519
 520             return true; /* Case 2) */
 521         }
 522     }
 523
 524     /* Only cases 3) and 4) left */
 525     wxASSERT( m_imageList != NULL ); /* Just in case */
 526
 527     /* Construct the new pixmap */
 528     const wxBitmap *bmp = m_imageList->GetBitmapPtr(image);
 529     GdkPixmap *pixmap = bmp->GetPixmap();
 530     GdkBitmap *mask = (GdkBitmap*) NULL;
 531     if ( bmp->GetMask() )
 532     {
 533         mask = bmp->GetMask()->GetBitmap();
 534     }
 535
 536     if (pixmapwid == NULL)
 537     {
 538         /* Case 3) No old pixmap. Create a new one and prepend it to the hbox */
 539         pixmapwid = gtk_pixmap_new (pixmap, mask );
 540
 541         /* CHECKME: Are these pack flags okay? */
 542         gtk_box_pack_start(GTK_BOX(nb_page->m_box), pixmapwid, FALSE, FALSE, m_padding);
 543         gtk_widget_show(pixmapwid);
 544     }
 545     else
 546     {
 547         /* Case 4) Simply replace the pixmap */
 548         gtk_pixmap_set(GTK_PIXMAP(pixmapwid), pixmap, mask);
 549     }
 550
 551     nb_page->m_image = image;
 552
 553     return true;
 554 }
 555
 556 void wxNotebook::SetPageSize( const wxSize &WXUNUSED(size) )
 557 {
 558     wxFAIL_MSG( wxT("wxNotebook::SetPageSize not implemented") );
 559 }
 560
 561 void wxNotebook::SetPadding( const wxSize &padding )
 562 {
 563     wxCHECK_RET( m_widget != NULL, wxT("invalid notebook") );
 564
 565     m_padding = padding.GetWidth();
 566
 567     int i;
 568     for (i=0; i<int(GetPageCount()); i++)
 569     {
 570         wxGtkNotebookPage* nb_page = GetNotebookPage(i);
 571         wxASSERT(nb_page != NULL);
 572
 573         if (nb_page->m_image != -1)
 574         {
 575             // gtk_box_set_child_packing sets padding on BOTH sides
 576             // icon provides left padding, label provides center and right
 577             int image = nb_page->m_image;
 578             SetPageImage(i,-1);
 579             SetPageImage(i,image);
 580         }
 581         wxASSERT(nb_page->m_label);
 582         gtk_box_set_child_packing(GTK_BOX(nb_page->m_box),
 583                                   GTK_WIDGET(nb_page->m_label),
 584                                   FALSE, FALSE, m_padding, GTK_PACK_END);
 585     }
 586 }
 587
 588 void wxNotebook::SetTabSize(const wxSize& WXUNUSED(sz))
 589 {
 590     wxFAIL_MSG( wxT("wxNotebook::SetTabSize not implemented") );
 591 }
 592
 593 bool wxNotebook::DeleteAllPages()
 594 {
 595     wxCHECK_MSG( m_widget != NULL, false, wxT("invalid notebook") );
 596
 597     while (m_pagesData.GetCount() > 0)
 598         DeletePage( m_pagesData.GetCount()-1 );
 599
 600     wxASSERT_MSG( GetPageCount() == 0, _T("all pages must have been deleted") );
 601
 602     InvalidateBestSize();
 603     return wxNotebookBase::DeleteAllPages();
 604 }
 605
 606 wxNotebookPage *wxNotebook::DoRemovePage( size_t page )
 607 {
 608     if ( m_selection != -1 && (size_t)m_selection >= page )
 609     {
 610         // the index will become invalid after the page is deleted
 611         m_selection = -1;
 612     }
 613
 614     wxNotebookPage *client = wxNotebookBase::DoRemovePage(page);
 615     if ( !client )
 616         return NULL;
 617
 618     gtk_widget_ref( client->m_widget );
 619     gtk_widget_unrealize( client->m_widget );
 620     gtk_widget_unparent( client->m_widget );
 621
 622     // gtk_notebook_remove_page() sends "switch_page" signal with some strange
 623     // new page index (when deleting selected page 0, new page is 1 although,
 624     // clearly, the selection should stay 0), so suppress this
 625     g_signal_handlers_disconnect_by_func (m_widget,
 626                                           (gpointer) gtk_notebook_page_change_callback,
 627                                           this);
 628
 629     gtk_notebook_remove_page( GTK_NOTEBOOK(m_widget), page );
 630
 631     g_signal_connect (m_widget, "switch_page",
 632                       G_CALLBACK (gtk_notebook_page_change_callback), this);
 633
 634     wxGtkNotebookPage* p = GetNotebookPage(page);
 635     m_pagesData.DeleteObject(p);
 636     delete p;
 637
 638     return client;
 639 }
 640
 641 bool wxNotebook::InsertPage( size_t position,
 642                              wxNotebookPage* win,
 643                              const wxString& text,
 644                              bool select,
 645                              int imageId )
 646 {
 647     wxCHECK_MSG( m_widget != NULL, false, wxT("invalid notebook") );
 648
 649     wxCHECK_MSG( win->GetParent() == this, false,
 650                wxT("Can't add a page whose parent is not the notebook!") );
 651
 652     wxCHECK_MSG( position <= GetPageCount(), false,
 653                  _T("invalid page index in wxNotebookPage::InsertPage()") );
 654
 655     // Hack Alert! (Part II): See above in wxInsertChildInNotebook callback
 656     // why this has to be done.  NOTE: using gtk_widget_unparent here does not
 657     // work as it seems to undo too much and will cause errors in the
 658     // gtk_notebook_insert_page below, so instead just clear the parent by
 659     // hand here.
 660     win->m_widget->parent = NULL;
 661
 662     // don't receive switch page during addition
 663     g_signal_handlers_disconnect_by_func (m_widget,
 664                                           (gpointer) gtk_notebook_page_change_callback,
 665                                           this);
 666
 667     if (m_themeEnabled)
 668         win->SetThemeEnabled(true);
 669
 670     GtkNotebook *notebook = GTK_NOTEBOOK(m_widget);
 671
 672     wxGtkNotebookPage *nb_page = new wxGtkNotebookPage();
 673
 674     if ( position == GetPageCount() )
 675         m_pagesData.Append( nb_page );
 676     else
 677         m_pagesData.Insert( position, nb_page );
 678
 679     m_pages.Insert(win, position);
 680
 681     nb_page->m_box = gtk_hbox_new( FALSE, 1 );
 682     gtk_container_border_width( GTK_CONTAINER(nb_page->m_box), 2 );
 683
 684     g_signal_connect (win->m_widget, "size_allocate",
 685                       G_CALLBACK (gtk_page_size_callback), win);
 686
 687     gtk_notebook_insert_page( notebook, win->m_widget, nb_page->m_box, position );
 688
 689     nb_page->m_page = (GtkNotebookPage*) g_list_last(notebook->children)->data;
 690
 691     /* set the label image */
 692     nb_page->m_image = imageId;
 693
 694     if (imageId != -1)
 695     {
 696         wxASSERT( m_imageList != NULL );
 697
 698         const wxBitmap *bmp = m_imageList->GetBitmapPtr(imageId);
 699         GdkPixmap *pixmap = bmp->GetPixmap();
 700         GdkBitmap *mask = (GdkBitmap*) NULL;
 701         if ( bmp->GetMask() )
 702         {
 703             mask = bmp->GetMask()->GetBitmap();
 704         }
 705
 706         GtkWidget *pixmapwid = gtk_pixmap_new (pixmap, mask );
 707
 708         gtk_box_pack_start(GTK_BOX(nb_page->m_box), pixmapwid, FALSE, FALSE, m_padding);
 709
 710         gtk_widget_show(pixmapwid);
 711     }
 712
 713     /* set the label text */
 714
 715     nb_page->m_text = wxStripMenuCodes(text);
 716     if (nb_page->m_text.empty()) nb_page->m_text = wxEmptyString;
 717
 718     nb_page->m_label = GTK_LABEL( gtk_label_new(wxGTK_CONV(nb_page->m_text)) );
 719     gtk_box_pack_end( GTK_BOX(nb_page->m_box), GTK_WIDGET(nb_page->m_label), FALSE, FALSE, m_padding );
 720
 721     /* apply current style */
 722     GtkRcStyle *style = CreateWidgetStyle();
 723     if ( style )
 724     {
 725         gtk_widget_modify_style(GTK_WIDGET(nb_page->m_label), style);
 726         gtk_rc_style_unref(style);
 727     }
 728
 729     /* show the label */
 730     gtk_widget_show( GTK_WIDGET(nb_page->m_label) );
 731     if (select && (m_pagesData.GetCount() > 1))
 732     {
 733       SetSelection( position );
 734     }
 735
 736     g_signal_connect (m_widget, "switch_page",
 737                       G_CALLBACK (gtk_notebook_page_change_callback), this);
 738
 739     InvalidateBestSize();
 740     return true;
 741 }
 742
 743 // helper for HitTest(): check if the point lies inside the given widget which
 744 // is the child of the notebook whose position and border size are passed as
 745 // parameters
 746 static bool
 747 IsPointInsideWidget(const wxPoint& pt, GtkWidget *w,
 748                     gint x, gint y, gint border = 0)
 749 {
 750     return
 751         (pt.x >= w->allocation.x - x - border) &&
 752         (pt.x <= w->allocation.x - x + border + w->allocation.width) &&
 753         (pt.y >= w->allocation.y - y - border) &&
 754         (pt.y <= w->allocation.y - y + border + w->allocation.height);
 755 }
 756
 757 int wxNotebook::HitTest(const wxPoint& pt, long *flags) const
 758 {
 759     const gint x = m_widget->allocation.x;
 760     const gint y = m_widget->allocation.y;
 761
 762     const size_t count = GetPageCount();
 763     size_t i = 0;
 764
 765     GtkNotebook * notebook = GTK_NOTEBOOK(m_widget);
 766     if (gtk_notebook_get_scrollable(notebook))
 767         i = g_list_position( notebook->children, notebook->first_tab );
 768
 769     for ( ; i < count; i++ )
 770     {
 771         wxGtkNotebookPage* nb_page = GetNotebookPage(i);
 772         GtkWidget *box = nb_page->m_box;
 773
 774         const gint border = gtk_container_get_border_width(GTK_CONTAINER(box));
 775
 776         if ( IsPointInsideWidget(pt, box, x, y, border) )
 777         {
 778             // ok, we're inside this tab -- now find out where, if needed
 779             if ( flags )
 780             {
 781                 GtkWidget *pixmap = NULL;
 782
 783                 GList *children = gtk_container_get_children(GTK_CONTAINER(box));
 784                 for ( GList *child = children; child; child = child->next )
 785                 {
 786                     if ( GTK_IS_PIXMAP(child->data) )
 787                     {
 788                         pixmap = GTK_WIDGET(child->data);
 789                         break;
 790                     }
 791                 }
 792
 793                 if ( children )
 794                     g_list_free(children);
 795
 796                 if ( pixmap && IsPointInsideWidget(pt, pixmap, x, y) )
 797                 {
 798                     *flags = wxBK_HITTEST_ONICON;
 799                 }
 800                 else if ( IsPointInsideWidget(pt, GTK_WIDGET(nb_page->m_label), x, y) )
 801                 {
 802                     *flags = wxBK_HITTEST_ONLABEL;
 803                 }
 804                 else
 805                 {
 806                     *flags = wxBK_HITTEST_ONITEM;
 807                 }
 808             }
 809
 810             return i;
 811         }
 812     }
 813
 814     if ( flags )
 815     {
 816         *flags = wxBK_HITTEST_NOWHERE;
 817         wxWindowBase * page = GetCurrentPage();
 818         if ( page )
 819         {
 820             // rect origin is in notebook's parent coordinates
 821             wxRect rect = page->GetRect();
 822
 823             // adjust it to the notebook's coordinates
 824             wxPoint pos = GetPosition();
 825             rect.x -= pos.x;
 826             rect.y -= pos.y;
 827             if ( rect.Contains( pt ) )
 828                 *flags |= wxBK_HITTEST_ONPAGE;
 829         }
 830     }
 831
 832     return wxNOT_FOUND;
 833 }
 834
 835 void wxNotebook::OnNavigationKey(wxNavigationKeyEvent& event)
 836 {
 837     if (event.IsWindowChange())
 838         AdvanceSelection( event.GetDirection() );
 839     else
 840         event.Skip();
 841 }
 842
 843 #if wxUSE_CONSTRAINTS
 844
 845 // override these 2 functions to do nothing: everything is done in OnSize
 846 void wxNotebook::SetConstraintSizes( bool WXUNUSED(recurse) )
 847 {
 848     // don't set the sizes of the pages - their correct size is not yet known
 849     wxControl::SetConstraintSizes(false);
 850 }
 851
 852 bool wxNotebook::DoPhase( int WXUNUSED(nPhase) )
 853 {
 854     return true;
 855 }
 856
 857 #endif
 858
 859 void wxNotebook::DoApplyWidgetStyle(GtkRcStyle *style)
 860 {
 861     gtk_widget_modify_style(m_widget, style);
 862     size_t cnt = m_pagesData.GetCount();
 863     for (size_t i = 0; i < cnt; i++)
 864         gtk_widget_modify_style(GTK_WIDGET(GetNotebookPage(i)->m_label), style);
 865 }
 866
 867 GdkWindow *wxNotebook::GTKGetWindow(wxArrayGdkWindows& windows) const
 868 {
 869     windows.push_back(m_widget->window);
 870     windows.push_back(GTK_NOTEBOOK(m_widget)->event_window);
 871
 872     return NULL;
 873 }
 874
 875 // static
 876 wxVisualAttributes
 877 wxNotebook::GetClassDefaultAttributes(wxWindowVariant WXUNUSED(variant))
 878 {
 879     return GetDefaultAttributesFromGTKWidget(gtk_notebook_new);
 880 }
 881
 882 //-----------------------------------------------------------------------------
 883 // wxNotebookEvent
 884 //-----------------------------------------------------------------------------
 885
 886 IMPLEMENT_DYNAMIC_CLASS(wxNotebookEvent, wxNotifyEvent)
 887
 888 #endif