Ticket #73 (closed defect)

Opened 8 years ago

Last modified 3 years ago

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) defined(WXMOTIF)

#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);

}

//-----------------------------------------------------------------------------

Change History

Changed 3 years ago by hockkn

Basically deprecated platform, ancient report... closing.

Note: See TracTickets for help on using tickets.