From: Vadim Zeitlin Date: Wed, 7 Jul 1999 16:00:06 +0000 (+0000) Subject: attempts to fix wxStaticBitmap::SetBitmap() - still doesn't work X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/32d4bfd16c554ba060ccee9236abb8e7ff0e913a?ds=sidebyside attempts to fix wxStaticBitmap::SetBitmap() - still doesn't work git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@2961 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- diff --git a/include/wx/gtk/statbmp.h b/include/wx/gtk/statbmp.h index cfedc1c502..4716ffd797 100644 --- a/include/wx/gtk/statbmp.h +++ b/include/wx/gtk/statbmp.h @@ -75,6 +75,9 @@ public: } private: + // creates the new pixmap widget + void CreatePixmapWidget(); + wxBitmap m_bitmap; }; diff --git a/include/wx/gtk1/statbmp.h b/include/wx/gtk1/statbmp.h index cfedc1c502..4716ffd797 100644 --- a/include/wx/gtk1/statbmp.h +++ b/include/wx/gtk1/statbmp.h @@ -75,6 +75,9 @@ public: } private: + // creates the new pixmap widget + void CreatePixmapWidget(); + wxBitmap m_bitmap; }; diff --git a/src/gtk/app.cpp b/src/gtk/app.cpp index 4fbd74c2f0..a690ab53a4 100644 --- a/src/gtk/app.cpp +++ b/src/gtk/app.cpp @@ -46,7 +46,7 @@ //----------------------------------------------------------------------------- wxApp *wxTheApp = (wxApp *) NULL; -wxAppInitializerFunction wxApp::m_appInitFn = (wxAppInitializerFunction) NULL; +wxAppInitializerFunction wxAppBase::m_appInitFn = (wxAppInitializerFunction) NULL; #if wxUSE_THREADS extern wxList *wxPendingEvents; diff --git a/src/gtk/statbmp.cpp b/src/gtk/statbmp.cpp index d27737a420..ac1b7529b8 100644 --- a/src/gtk/statbmp.cpp +++ b/src/gtk/statbmp.cpp @@ -35,9 +35,21 @@ wxStaticBitmap::wxStaticBitmap( wxWindow *parent, wxWindowID id, const wxBitmap Create( parent, id, bitmap, pos, size, style, name ); } +void wxStaticBitmap::CreatePixmapWidget() +{ + wxCHECK_RET( m_bitmap.Ok(), "should only be called if we have a bitmap" ); + + GdkBitmap *mask = (GdkBitmap *) NULL; + if ( m_bitmap.GetMask() ) + mask = m_bitmap.GetMask()->GetBitmap(); + m_widget = gtk_pixmap_new( m_bitmap.GetPixmap(), mask ); + + PostCreation(); +} + bool wxStaticBitmap::Create( wxWindow *parent, wxWindowID id, const wxBitmap &bitmap, - const wxPoint &pos, const wxSize &size, - long style, const wxString &name ) + const wxPoint &pos, const wxSize &size, + long style, const wxString &name ) { m_needParent = TRUE; @@ -49,9 +61,7 @@ bool wxStaticBitmap::Create( wxWindow *parent, wxWindowID id, const wxBitmap &bi if (m_bitmap.Ok()) { - GdkBitmap *mask = (GdkBitmap *) NULL; - if (m_bitmap.GetMask()) mask = m_bitmap.GetMask()->GetBitmap(); - m_widget = gtk_pixmap_new( m_bitmap.GetPixmap(), mask ); + CreatePixmapWidget(); if (newSize.x == -1) newSize.x = m_bitmap.GetWidth(); if (newSize.y == -1) newSize.y = m_bitmap.GetHeight(); @@ -60,12 +70,12 @@ bool wxStaticBitmap::Create( wxWindow *parent, wxWindowID id, const wxBitmap &bi else { m_widget = gtk_label_new( "Bitmap" ); + + PostCreation(); } m_parent->DoAddChild( this ); - PostCreation(); - Show( TRUE ); return TRUE; @@ -73,10 +83,20 @@ bool wxStaticBitmap::Create( wxWindow *parent, wxWindowID id, const wxBitmap &bi void wxStaticBitmap::SetBitmap( const wxBitmap &bitmap ) { + bool hasWidget = m_bitmap.Ok(); m_bitmap = bitmap; if (m_bitmap.Ok()) { + if ( !hasWidget ) + { + gtk_widget_destroy( m_widget ); + + // recreate m_widget because we'd created a label and not a bitmap + // above + CreatePixmapWidget(); + } + GdkBitmap *mask = (GdkBitmap *) NULL; if (m_bitmap.GetMask()) mask = m_bitmap.GetMask()->GetBitmap(); gtk_pixmap_set( GTK_PIXMAP(m_widget), m_bitmap.GetPixmap(), mask ); diff --git a/src/gtk1/app.cpp b/src/gtk1/app.cpp index 4fbd74c2f0..a690ab53a4 100644 --- a/src/gtk1/app.cpp +++ b/src/gtk1/app.cpp @@ -46,7 +46,7 @@ //----------------------------------------------------------------------------- wxApp *wxTheApp = (wxApp *) NULL; -wxAppInitializerFunction wxApp::m_appInitFn = (wxAppInitializerFunction) NULL; +wxAppInitializerFunction wxAppBase::m_appInitFn = (wxAppInitializerFunction) NULL; #if wxUSE_THREADS extern wxList *wxPendingEvents; diff --git a/src/gtk1/statbmp.cpp b/src/gtk1/statbmp.cpp index d27737a420..ac1b7529b8 100644 --- a/src/gtk1/statbmp.cpp +++ b/src/gtk1/statbmp.cpp @@ -35,9 +35,21 @@ wxStaticBitmap::wxStaticBitmap( wxWindow *parent, wxWindowID id, const wxBitmap Create( parent, id, bitmap, pos, size, style, name ); } +void wxStaticBitmap::CreatePixmapWidget() +{ + wxCHECK_RET( m_bitmap.Ok(), "should only be called if we have a bitmap" ); + + GdkBitmap *mask = (GdkBitmap *) NULL; + if ( m_bitmap.GetMask() ) + mask = m_bitmap.GetMask()->GetBitmap(); + m_widget = gtk_pixmap_new( m_bitmap.GetPixmap(), mask ); + + PostCreation(); +} + bool wxStaticBitmap::Create( wxWindow *parent, wxWindowID id, const wxBitmap &bitmap, - const wxPoint &pos, const wxSize &size, - long style, const wxString &name ) + const wxPoint &pos, const wxSize &size, + long style, const wxString &name ) { m_needParent = TRUE; @@ -49,9 +61,7 @@ bool wxStaticBitmap::Create( wxWindow *parent, wxWindowID id, const wxBitmap &bi if (m_bitmap.Ok()) { - GdkBitmap *mask = (GdkBitmap *) NULL; - if (m_bitmap.GetMask()) mask = m_bitmap.GetMask()->GetBitmap(); - m_widget = gtk_pixmap_new( m_bitmap.GetPixmap(), mask ); + CreatePixmapWidget(); if (newSize.x == -1) newSize.x = m_bitmap.GetWidth(); if (newSize.y == -1) newSize.y = m_bitmap.GetHeight(); @@ -60,12 +70,12 @@ bool wxStaticBitmap::Create( wxWindow *parent, wxWindowID id, const wxBitmap &bi else { m_widget = gtk_label_new( "Bitmap" ); + + PostCreation(); } m_parent->DoAddChild( this ); - PostCreation(); - Show( TRUE ); return TRUE; @@ -73,10 +83,20 @@ bool wxStaticBitmap::Create( wxWindow *parent, wxWindowID id, const wxBitmap &bi void wxStaticBitmap::SetBitmap( const wxBitmap &bitmap ) { + bool hasWidget = m_bitmap.Ok(); m_bitmap = bitmap; if (m_bitmap.Ok()) { + if ( !hasWidget ) + { + gtk_widget_destroy( m_widget ); + + // recreate m_widget because we'd created a label and not a bitmap + // above + CreatePixmapWidget(); + } + GdkBitmap *mask = (GdkBitmap *) NULL; if (m_bitmap.GetMask()) mask = m_bitmap.GetMask()->GetBitmap(); gtk_pixmap_set( GTK_PIXMAP(m_widget), m_bitmap.GetPixmap(), mask );