Applied #15375 to stop event-sending in generic wxSpinCtrl ctor (eco)

[wxWidgets.git] / src / motif / statbox.cpp

diff --git a/src/motif/statbox.cpp b/src/motif/statbox.cpp
index 487d56710bf19a02b876697431a024c27a0a83eb..be5679b748de5d9d323fcac504beeeb3b6f2f645 100644 (file)
--- a/src/motif/statbox.cpp
+++ b/src/motif/statbox.cpp
@@ -1,44 +1,78 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        statbox.cpp
+// Name:        src/motif/statbox.cpp

 // Purpose:     wxStaticBox
 // Author:      Julian Smart
 // Modified by:
 // Created:     17/09/98
 // Purpose:     wxStaticBox
 // Author:      Julian Smart
 // Modified by:
 // Created:     17/09/98

-// RCS-ID:      $Id$

 // Copyright:   (c) Julian Smart
 // Copyright:   (c) Julian Smart

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

 /////////////////////////////////////////////////////////////////////////////
 
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
-#pragma implementation "statbox.h"
-#endif
+// For compilers that support precompilation, includes "wx.h".
+#include "wx/wxprec.h"

 
 #include "wx/statbox.h"
 
 #include "wx/statbox.h"

-#include "wx/utils.h"

 
 

+#ifndef WX_PRECOMP
+    #include "wx/utils.h"
+#endif
+
+#ifdef __VMS__
+#pragma message disable nosimpint
+#endif

 #include <Xm/Frame.h>
 #include <Xm/Frame.h>

-#include <Xm/Form.h>

 #include <Xm/Label.h>
 #include <Xm/Label.h>

-#include <Xm/LabelG.h>
-
-#include <wx/motif/private.h>
+#ifdef __VMS__
+#pragma message enable nosimpint
+#endif

 
 

-#if !USE_SHARED_LIBRARY
-IMPLEMENT_DYNAMIC_CLASS(wxStaticBox, wxControl)
+#include "wx/motif/private.h"

 
 BEGIN_EVENT_TABLE(wxStaticBox, wxControl)
 
 BEGIN_EVENT_TABLE(wxStaticBox, wxControl)

-       EVT_ERASE_BACKGROUND(wxStaticBox::OnEraseBackground)
+//EVT_ERASE_BACKGROUND(wxStaticBox::OnEraseBackground)

 END_EVENT_TABLE()
 
 END_EVENT_TABLE()
 

-#endif
+// ----------------------------------------------------------------------------
+// wxXmSizeKeeper
+// ----------------------------------------------------------------------------
+
+// helper class to reduce code duplication
+class wxXmSizeKeeper
+{
+    Dimension m_x, m_y;
+    Widget m_widget;
+public:
+    wxXmSizeKeeper( Widget w )
+        : m_widget( w )
+    {
+        XtVaGetValues( m_widget,
+                       XmNwidth, &m_x,
+                       XmNheight, &m_y,
+                       NULL );
+    }
+
+    void Restore()
+    {
+        int x, y;
+
+        XtVaGetValues( m_widget,
+                       XmNwidth, &x,
+                       XmNheight, &y,
+                       NULL );
+        if( x != m_x || y != m_y )
+            XtVaSetValues( m_widget,
+                           XmNwidth, m_x,
+                           XmNheight, m_y,
+                           NULL );
+    }
+};

 
 /*
  * Static box
  */
 
 /*
  * Static box
  */

- 
+

 wxStaticBox::wxStaticBox()
 {
 wxStaticBox::wxStaticBox()
 {

-    m_formWidget = (WXWidget) 0;

     m_labelWidget = (WXWidget) 0;
 }
 
     m_labelWidget = (WXWidget) 0;
 }
 


@@ -49,154 +83,82 @@ bool wxStaticBox::Create(wxWindow *parent, wxWindowID id,
            long style,
            const wxString& name)
 {
            long style,
            const wxString& name)
 {

-    m_formWidget = (WXWidget) 0;
+    if( !CreateControl( parent, id, pos, size, style,
+                        wxDefaultValidator, name ) )
+        return false;

     m_labelWidget = (WXWidget) 0;
     m_labelWidget = (WXWidget) 0;

-    m_backgroundColour = parent->GetBackgroundColour();
-    m_foregroundColour = parent->GetForegroundColour();
-    m_windowFont = parent->GetFont();
-
-    SetName(name);
-
-    if (parent) parent->AddChild(this);
-
-    if ( id == -1 )
-           m_windowId = (int)NewControlId();
-    else
-           m_windowId = id;
-
-    m_windowStyle = style;
-
-    bool hasLabel = (!label.IsNull() && !label.IsEmpty()) ;
+    PreCreation();

 
     Widget parentWidget = (Widget) parent->GetClientWidget();
 
 
     Widget parentWidget = (Widget) parent->GetClientWidget();
 

-    Widget formWidget = XtVaCreateManagedWidget ((char*) (const char*) name,
-                                       xmFormWidgetClass, parentWidget,
-                                       XmNmarginHeight, 0,
-                                       XmNmarginWidth, 0,
-                                       NULL);
+    m_mainWidget = XtVaCreateManagedWidget ("staticboxframe",
+            xmFrameWidgetClass, parentWidget,
+            // MBN: why override default?
+            // XmNshadowType, XmSHADOW_IN,
+            NULL);

 
 

-
-    if (hasLabel)
+    if (!label.empty())

     {
     {

-        XmFontList fontList = (XmFontList) m_windowFont.GetFontList(1.0, XtDisplay(parentWidget));
-
-        wxString label1(wxStripMenuCodes(label));
-        XmString text = XmStringCreateSimple ((char*) (const char*) label1);
-        m_labelWidget = (WXWidget) XtVaCreateManagedWidget ((char*) (const char*) label1,
-                                            xmLabelWidgetClass, formWidget,
-                                             XmNfontList, fontList,
-                                            XmNlabelString, text,
-                                            NULL);
-        XmStringFree (text);
+        wxString label1(GetLabelText(label));
+        wxXmString text(label1);
+        Display* dpy = XtDisplay( parentWidget );
+
+        m_labelWidget = (WXWidget) XtVaCreateManagedWidget ("staticboxlabel",
+                xmLabelWidgetClass, (Widget)m_mainWidget,
+                wxFont::GetFontTag(), m_font.GetFontTypeC(dpy),
+                XmNlabelString, text(),
+#if wxCHECK_MOTIF_VERSION( 2, 0 )
+                XmNframeChildType, XmFRAME_TITLE_CHILD,
+#else
+                XmNchildType, XmFRAME_TITLE_CHILD,
+#endif
+                NULL);

     }
 
     }
 

-    Widget frameWidget = XtVaCreateManagedWidget ("frame",
-                                       xmFrameWidgetClass, formWidget,
-                                        XmNshadowType, XmSHADOW_IN,
-//                                     XmNmarginHeight, 0,
-//                                     XmNmarginWidth, 0,
-                                       NULL);
-
-    if (hasLabel)
-        XtVaSetValues ((Widget) m_labelWidget,
-                      XmNtopAttachment, XmATTACH_FORM,
-                      XmNleftAttachment, XmATTACH_FORM,
-                      XmNrightAttachment, XmATTACH_FORM,
-                      XmNalignment, XmALIGNMENT_BEGINNING,
-                      NULL);
-
-    XtVaSetValues (frameWidget,
-           XmNtopAttachment, hasLabel ? XmATTACH_WIDGET : XmATTACH_FORM,
-                    XmNtopWidget, hasLabel ? (Widget) m_labelWidget : formWidget,
-                    XmNbottomAttachment, XmATTACH_FORM,
-                    XmNleftAttachment, XmATTACH_FORM,
-                    XmNrightAttachment, XmATTACH_FORM,
-                    NULL);
-
-    m_mainWidget = (Widget) formWidget;
-
-    SetCanAddEventHandler(TRUE);
-    AttachWidget (parent, m_mainWidget, (WXWidget) frameWidget, pos.x, pos.y, size.x, size.y);
-    ChangeBackgroundColour();
-
-    return TRUE;
+    PostCreation();
+    AttachWidget (parent, m_mainWidget, NULL, pos.x, pos.y, size.x, size.y);
+
+    return true;

 }
 
 }
 

-void wxStaticBox::SetLabel(const wxString& label)
+wxStaticBox::~wxStaticBox()

 {
 {

-    if (!m_labelWidget)
-        return;
+   DetachWidget(m_mainWidget);
+   XtDestroyWidget((Widget) m_mainWidget);

 
 

-    if (!label.IsNull())
-    {
-        wxString label1(wxStripMenuCodes(label));
-
-        XmString text = XmStringCreateSimple ((char*) (const char*) label1);
-        XtVaSetValues ((Widget) m_labelWidget,
-                    XmNlabelString, text,
-                    XmNlabelType, XmSTRING,
-                    NULL);
-        XmStringFree (text);
-    }
+   m_mainWidget = (WXWidget) 0;
+   m_labelWidget = (WXWidget) 0;

 }
 
 }
 

-wxString wxStaticBox::GetLabel() const
+void wxStaticBox::SetLabel( const wxString& label )

 {
 {

-    if (!m_labelWidget)
-        return wxEmptyString;
+    wxXmSizeKeeper sk( (Widget)GetMainWidget() );

 
 

-    XmString text = 0;
-    char *s;
-    XtVaGetValues ((Widget) m_labelWidget,
-                XmNlabelString, &text,
-                NULL);
+    wxStaticBoxBase::SetLabel( label );

 
 

-    if (!text)
-        return wxEmptyString;
-
-    if (XmStringGetLtoR (text, XmSTRING_DEFAULT_CHARSET, &s))
-    {
-        wxString str(s);
-        XtFree (s);
-        return str;
-    }
-    else
-    {
-        return wxEmptyString;
-    }
+    sk.Restore();

 }
 
 }
 

-void wxStaticBox::SetSize(int x, int y, int width, int height, int sizeFlags)
+void wxStaticBox::GetBordersForSizer(int *borderTop, int *borderOther) const

 {
 {

-    wxControl::SetSize (x, y, width, height, sizeFlags);
+    Dimension shadow, border;

 
 

-    if (m_labelWidget)
-    {
-        Dimension xx, yy;
-        XtVaGetValues ((Widget) m_labelWidget, XmNwidth, &xx, XmNheight, &yy, NULL);
-
-        if (width > -1)
-            XtVaSetValues ((Widget) m_mainWidget, XmNwidth, width,
-                   NULL);
-        if (height > -1)
-            XtVaSetValues ((Widget) m_mainWidget, XmNheight, height - yy,
-                   NULL);
-    }
-}
+    XtVaGetValues( (Widget) GetMainWidget(),
+                   XmNshadowThickness, &shadow,
+                   XmNborderWidth, &border,
+                   NULL);

 
 

-void wxStaticBox::ChangeFont(bool keepOriginalSize)
-{
-    wxWindow::ChangeFont(keepOriginalSize);
-}
+    *borderOther = shadow + border;

 
 

-void wxStaticBox::ChangeBackgroundColour()
-{
-    wxWindow::ChangeBackgroundColour();
-}
+    if( GetLabelWidget() )
+    {
+        XtWidgetGeometry preferred;
+        XtQueryGeometry( (Widget) GetLabelWidget(), NULL, &preferred );

 
 

-void wxStaticBox::ChangeForegroundColour()
-{
-    wxWindow::ChangeForegroundColour();
+        *borderTop = preferred.height;
+    }
+    else
+    {
+        *borderTop = shadow;
+    }

 }
 }

-