Ticket #73 (closed defect)
Sizer does not produce same results under Motif as under NT
| Reported by: | anonymous | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | wxMotif | Version: | |
| Keywords: | Cc: | hockkn, depeyrot | |
| Blocked By: | Patch: | no | |
| Blocking: |
Description
I have modified the minimal program to show the problem. Under Motif controls are a different distance from static text than under NT. The static line disappears completely under Motif.
/////////////////////////////////////////////////////////////////////////////
// Name: minimal.cpp
// Purpose: Minimal wxWindows sample
// Author: Julian Smart
// Modified by:
// Created: 04/01/98
// RCS-ID: $Id: minimal.cpp,v 1.38 2000/07/15 19:50:15 cvsuser Exp $
// Copyright: (c) Julian Smart
// Licence: wxWindows licence
/////////////////////////////////////////////////////////////////////////////
// ============================================================================
// declarations
// ============================================================================
// ----------------------------------------------------------------------------
// headers
// ----------------------------------------------------------------------------
#ifdef GNUG
#pragma implementation "minimal.cpp"
#pragma interface "minimal.cpp"
#endif
// For compilers that support precompilation, includes "wx/wx.h".
#include "wx/wxprec.h"
#ifdef BORLANDC
#pragma hdrstop
#endif
// for all others, include the necessary headers (this file is usually all you
// need because it includes almost all "standard" wxWindows headers)
#ifndef WX_PRECOMP
#include "wx/wx.h"
#endif
// ----------------------------------------------------------------------------
// resources
// ----------------------------------------------------------------------------
// the application icon
#if defined(WXGTK)
#include "mondrian.xpm"
#endif
#include "wx/statline.h"
// ----------------------------------------------------------------------------
// private classes
// ----------------------------------------------------------------------------
// Define a new application type, each program should derive a class from wxApp
class MyApp : public wxApp
{
public:
// override base class virtuals
// ----------------------------
// this one is called on application startup and is a good place for the app
// initialization (doing it here and not in the ctor allows to have an error
// return: if OnInit() returns false, the application terminates)
virtual bool OnInit();
};
// Define a new frame type: this is going to be our main frame
class MyFrame : public wxFrame
{
public:
// ctor(s)
MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size);
// event handlers (these functions should _not_ be virtual)
void OnQuit(wxCommandEvent& event);
void OnAbout(wxCommandEvent& event);
void OnDialog(wxCommandEvent& event);
private:
// any class wishing to process wxWindows events must use this macro
DECLARE_EVENT_TABLE()
};
// ----------------------------------------------------------------------------
// constants
// ----------------------------------------------------------------------------
// IDs for the controls and the menu commands
enum
{
// menu items
Minimal_Quit = 1,
Minimal_About = 2,
Minimal_Dialog
};
// ----------------------------------------------------------------------------
// event tables and other macros for wxWindows
// ----------------------------------------------------------------------------
// the event tables connect the wxWindows events with the functions (event
// handlers) which process them. It can be also done at run-time, but for the
// simple menu events like this the static method is much simpler.
BEGIN_EVENT_TABLE(MyFrame, wxFrame)
EVT_MENU(Minimal_Quit, MyFrame::OnQuit)
EVT_MENU(Minimal_About, MyFrame::OnAbout)
EVT_MENU(Minimal_Dialog, MyFrame::OnDialog)
END_EVENT_TABLE()
// Create a new application object: this macro will allow wxWindows to create
// the application object during program execution (it's better than using a
// static object for many reasons) and also declares the accessor function
// wxGetApp() which will return the reference of the right type (i.e. MyApp and
// not wxApp)
IMPLEMENT_APP(MyApp)
// ============================================================================
// implementation
// ============================================================================
// ----------------------------------------------------------------------------
// the application class
// ----------------------------------------------------------------------------
// 'Main program' equivalent: the program execution "starts" here
bool MyApp::OnInit()
{
// create the main application window
MyFrame *frame = new MyFrame("Minimal wxWindows App",
wxPoint(50, 50), wxSize(450, 340));
// and show it (the frames, unlike simple controls, are not shown when
// created initially)
frame->Show(TRUE);
// success: wxApp::OnRun() will be called which will enter the main message
// loop and the application will run. If we returned FALSE here, the
// application would exit immediately.
return TRUE;
}
// ----------------------------------------------------------------------------
// main frame
// ----------------------------------------------------------------------------
// frame constructor
MyFrame::MyFrame(const wxString& title, const wxPoint& pos, const wxSize& size)
: wxFrame((wxFrame *)NULL, -1, title, pos, size)
{
#ifdef WXMAC
// we need this in order to allow the about menu relocation, since ABOUT is
// not the default id of the about menu
wxApp::s_macAboutMenuItemId = Minimal_About;
#endif
// set the frame icon
SetIcon(wxICON(mondrian));
// create a menu bar
wxMenu *menuFile = new wxMenu("", wxMENU_TEAROFF);
// the "About" item should be in the help menu
wxMenu *helpMenu = new wxMenu;
helpMenu->Append(Minimal_About, "&About...\tCtrl-A", "Show about dialog");
menuFile->Append(Minimal_Dialog, "&Calculate\tAlt-C", "Calculate results");
menuFile->Append(Minimal_Quit, "E&xit\tAlt-X", "Quit this program");
// now append the freshly created menu to the menu bar...
wxMenuBar *menuBar = new wxMenuBar();
menuBar->Append(menuFile, "&File");
menuBar->Append(helpMenu, "&Help");
// ... and attach this menu bar to the frame
SetMenuBar(menuBar);
#if wxUSE_STATUSBAR
// create a status bar just for fun (by default with 1 pane only)
CreateStatusBar(2);
SetStatusText("Welcome to wxWindows!");
#endif // wxUSE_STATUSBAR
}
// event handlers
void MyFrame::OnQuit(wxCommandEvent& WXUNUSED(event))
{
// TRUE is to force the frame to close
Close(TRUE);
}
void MyFrame::OnAbout(wxCommandEvent& WXUNUSED(event))
{
wxString msg;
msg.Printf( _T("This is the about dialog of minimal sample.\n")
_T("Welcome to %s"), wxVERSION_STRING);
wxMessageBox(msg, "About Minimal", wxOK | wxICON_INFORMATION, this);
}
//-----------------------------------------------------------------------------
void MyFrame::OnDialog(wxCommandEvent& WXUNUSED(event))
{
wxFrame *frame = new wxFrame (this, -1, "");
wxStaticBox *box = new wxStaticBox (frame, -1, "");
wxStaticBoxSizer *topsizer = new wxStaticBoxSizer (box, wxVERTICAL);
topsizer->Add
(new wxStaticText (frame, -1, "Output file"),
1, wxALL, 5);
topsizer->Add
(new wxTextCtrl (frame, -1),
1, wxLEFT | wxRIGHT | wxGROW, 5);
topsizer->Add
(new wxStaticText (frame, -1, "Description"),
1, wxALL, 5);
topsizer->Add
(new wxTextCtrl (frame, -1),
1, wxLEFT | wxRIGHT | wxGROW, 5);
wxBoxSizer *scriptsizer;
topsizer->Add
(scriptsizer = new wxBoxSizer (wxHORIZONTAL),
0, wxTOP | wxLEFT | wxRIGHT | wxGROW, 5);
scriptsizer->Add
(new wxStaticText (frame, -1, "Script name"),
1, wxTOP, 8);
const wxString choices[] = {"Delete script", "Keep script", "Append script",
"Script only"};
scriptsizer->Add
(new wxChoice (frame, -1, wxDefaultPosition,
wxDefaultSize, 4, choices),
0, wxTOP | wxALIGN_RIGHT, 5);
topsizer->Add
(new wxTextCtrl (frame, -1),
0, wxLEFT | wxTOP | wxRIGHT | wxGROW, 5);
// action area buttons
topsizer->Add
(new wxStaticLine (frame, -1),
0, wxALL | wxGROW, 5);
box = new wxStaticBox (frame, -1, "");
wxStaticBoxSizer *actionsizer;
topsizer->Add
(actionsizer = new wxStaticBoxSizer (box, wxHORIZONTAL),
0, wxALL | wxGROW, 5);
actionsizer->Add
(new wxButton (frame, -1, "Recalculate"),
1, wxLEFT | wxRIGHT, 3);
actionsizer->Add
(new wxButton (frame, -1, "Later"),
1, wxLEFT | wxRIGHT, 3);
actionsizer->Add
(new wxButton (frame, -1, "Cancel"),
1, wxLEFT | wxRIGHT, 3);
actionsizer->Add
(new wxButton (frame, -1, "Help"),
1, wxLEFT | wxRIGHT, 3);
frame->SetAutoLayout (TRUE);
topsizer->Fit (frame);
topsizer->SetSizeHints (frame);
frame->SetSizer (topsizer);
frame->Show (TRUE);
}
//-----------------------------------------------------------------------------
