// 1) Added capability for wxWizardPage to accept resources
// 2) Added "Help" button handler stub
// 3) Fixed ShowPage() bug on displaying bitmaps
+// Robert Vazan (sizers)
// Created: 15.08.99
// RCS-ID: $Id$
// Copyright: (c) 1999 Vadim Zeitlin <zeitlin@dptmaths.ens-cachan.fr>
// headers
// ----------------------------------------------------------------------------
-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
#pragma implementation "wizardg.h"
#endif
#endif //WX_PRECOMP
#include "wx/statline.h"
+#include "wx/sizer.h"
#include "wx/wizard.h"
+// ----------------------------------------------------------------------------
+// wxWizardSizer
+// ----------------------------------------------------------------------------
+
+class wxWizardSizer : public wxSizer
+{
+public:
+ wxWizardSizer(wxWizard *owner);
+
+ void RecalcSizes();
+ wxSize CalcMin();
+
+ wxSize GetMaxChildSize();
+ int Border() const;
+
+private:
+ wxSize SiblingSize(wxSizerItem *child);
+
+ wxWizard *m_owner;
+ bool m_childSizeValid;
+ wxSize m_childSize;
+};
+
// ----------------------------------------------------------------------------
// event tables and such
// ----------------------------------------------------------------------------
END_EVENT_TABLE()
IMPLEMENT_DYNAMIC_CLASS(wxWizard, wxDialog)
+
+/*
+ TODO PROPERTIES :
+ wxWizard
+ extstyle
+ title
+*/
+
IMPLEMENT_ABSTRACT_CLASS(wxWizardPage, wxPanel)
IMPLEMENT_DYNAMIC_CLASS(wxWizardPageSimple, wxWizardPage)
IMPLEMENT_DYNAMIC_CLASS(wxWizardEvent, wxNotifyEvent)
if ( resource != NULL )
{
#if wxUSE_WX_RESOURCES
- if ( !LoadFromResource(this, resource) )
+#if 0
+ if ( !LoadFromResource(this, resource) )
{
wxFAIL_MSG(wxT("wxWizardPage LoadFromResource failed!!!!"));
}
+#endif
#endif // wxUSE_RESOURCES
}
{
return m_next;
}
+
// ----------------------------------------------------------------------------
-// generic wxWizard implementation
+// wxWizardSizer
// ----------------------------------------------------------------------------
-void wxWizard::Init()
+wxWizardSizer::wxWizardSizer(wxWizard *owner)
+ : m_owner(owner)
{
- m_posWizard = wxDefaultPosition;
- m_page = (wxWizardPage *)NULL;
- m_btnPrev = m_btnNext = NULL;
- m_statbmp = NULL;
+ m_childSizeValid = false;
}
-bool wxWizard::Create(wxWindow *parent,
- int id,
- const wxString& title,
- const wxBitmap& bitmap,
- const wxPoint& pos)
+void wxWizardSizer::RecalcSizes()
{
- m_posWizard = pos;
- m_bitmap = bitmap ;
-
- // just create the dialog itself here, the controls will be created in
- // DoCreateControls() called later when we know our final size
- m_page = (wxWizardPage *)NULL;
- m_btnPrev = m_btnNext = NULL;
- m_statbmp = NULL;
-
- return wxDialog::Create(parent, id, title, pos);
+ // Effect of this function depends on m_owner->m_page and
+ // it should be called whenever it changes (wxWizard::ShowPage)
+ if ( m_owner->m_page )
+ {
+ m_owner->m_page->SetSize(m_position.x,m_position.y, m_size.x,m_size.y);
+ }
}
-void wxWizard::DoCreateControls()
+wxSize wxWizardSizer::CalcMin()
{
- // do nothing if the controls were already created
- if ( WasCreated() )
- return;
-
- // constants defining the dialog layout
- // ------------------------------------
-
- // these constants define the position of the upper left corner of the
- // bitmap or the page in the wizard
- static const int X_MARGIN = 10;
- static const int Y_MARGIN = 10;
-
- // margin between the bitmap and the panel
- static const int BITMAP_X_MARGIN = 15;
-
- // margin between the bitmap and the static line
- static const int BITMAP_Y_MARGIN = 15;
-
- // margin between the static line and the buttons
- static const int SEPARATOR_LINE_MARGIN = 15;
-
- // margin between "Next >" and "Cancel" buttons
- static const int BUTTON_MARGIN = 10;
+ return m_owner->GetPageSize();
+}
- // margin between Back and Next buttons
-#ifdef __WXMAC__
- static const int BACKNEXT_MARGIN = 10;
-#else
- static const int BACKNEXT_MARGIN = 0;
+wxSize wxWizardSizer::GetMaxChildSize()
+{
+#if !defined(__WXDEBUG__)
+ if ( m_childSizeValid )
+ return m_childSize;
#endif
- // default width and height of the page
- static const int DEFAULT_PAGE_WIDTH = 270;
- static const int DEFAULT_PAGE_HEIGHT = 290;
+ wxSize maxOfMin;
+ wxSizerItemList::compatibility_iterator childNode;
- // create controls
- // ---------------
+ for(childNode = m_children.GetFirst(); childNode;
+ childNode = childNode->GetNext())
+ {
+ wxSizerItem *child = childNode->GetData();
+ maxOfMin.IncTo(child->CalcMin());
+ maxOfMin.IncTo(SiblingSize(child));
+ }
- wxSize sizeBtn = wxButton::GetDefaultSize();
+#ifdef __WXDEBUG__
+ if ( m_childSizeValid && m_childSize != maxOfMin )
+ {
+ wxFAIL_MSG( _T("Size changed in wxWizard::GetPageAreaSizer()")
+ _T("after RunWizard().\n")
+ _T("Did you forget to call GetSizer()->Fit(this) ")
+ _T("for some page?")) ;
- // the global dialog layout is: a row of buttons at the bottom (aligned to
- // the right), the static line above them, the bitmap (if any) on the left
- // of the upper part of the dialog and the panel in the remaining space
- m_x = X_MARGIN;
- m_y = Y_MARGIN;
+ return m_childSize;
+ }
+#endif // __WXDEBUG__
- int defaultHeight;
- if ( m_bitmap.Ok() )
+ if ( m_owner->m_started )
{
- m_statbmp = new wxStaticBitmap(this, -1, m_bitmap, wxPoint(m_x, m_y));
+ m_childSizeValid = true;
+ m_childSize = maxOfMin;
+ }
+
+ return maxOfMin;
+}
- m_x += m_bitmap.GetWidth() + BITMAP_X_MARGIN;
+int wxWizardSizer::Border() const
+{
+ if ( m_owner->m_calledSetBorder )
+ return m_owner->m_border;
- defaultHeight = m_bitmap.GetHeight();
- }
- else
- {
- m_statbmp = (wxStaticBitmap *)NULL;
+ return m_children.IsEmpty() ? 5 : 0;
+}
- defaultHeight = DEFAULT_PAGE_HEIGHT;
+wxSize wxWizardSizer::SiblingSize(wxSizerItem *child)
+{
+ wxSize maxSibling;
+
+ if ( child->IsWindow() )
+ {
+ wxWizardPage *page = wxDynamicCast(child->GetWindow(), wxWizardPage);
+ if ( page )
+ {
+ for ( wxWizardPage *sibling = page->GetNext();
+ sibling;
+ sibling = sibling->GetNext() )
+ {
+ if ( sibling->GetSizer() )
+ {
+ maxSibling.IncTo(sibling->GetSizer()->CalcMin());
+ }
+ }
+ }
}
+
+ return maxSibling;
+}
- // use default size if none given and also make sure that the dialog is
- // not less than the default size
- m_height = m_sizePage.y == -1 ? defaultHeight : m_sizePage.y;
- m_width = m_sizePage.x == -1 ? DEFAULT_PAGE_WIDTH : m_sizePage.x;
- if ( m_height < defaultHeight )
- m_height = defaultHeight;
- if ( m_width < DEFAULT_PAGE_WIDTH )
- m_width = DEFAULT_PAGE_WIDTH;
+// ----------------------------------------------------------------------------
+// generic wxWizard implementation
+// ----------------------------------------------------------------------------
- int x = X_MARGIN;
- int y = m_y + m_height + BITMAP_Y_MARGIN;
+void wxWizard::Init()
+{
+ m_posWizard = wxDefaultPosition;
+ m_page = (wxWizardPage *)NULL;
+ m_btnPrev = m_btnNext = NULL;
+ m_statbmp = NULL;
+ m_sizerBmpAndPage = NULL;
+ m_sizerPage = NULL;
+ m_calledSetBorder = false;
+ m_border = 0;
+ m_started = false;
+}
-#if wxUSE_STATLINE
- (void)new wxStaticLine(this, -1, wxPoint(x, y),
- wxSize(m_x + m_width - x, 2));
-#endif // wxUSE_STATLINE
+bool wxWizard::Create(wxWindow *parent,
+ int id,
+ const wxString& title,
+ const wxBitmap& bitmap,
+ const wxPoint& pos,
+ long style)
+{
+ bool result = wxDialog::Create(parent,id,title,pos,wxDefaultSize,style);
+
+ m_posWizard = pos;
+ m_bitmap = bitmap ;
- x = m_x + m_width - 3*sizeBtn.x - BUTTON_MARGIN - BACKNEXT_MARGIN;
- y += SEPARATOR_LINE_MARGIN;
+ DoCreateControls();
+
+ return result;
+}
- if (GetExtraStyle() & wxWIZARD_EX_HELPBUTTON)
+void wxWizard::AddBitmapRow(wxBoxSizer *mainColumn)
+{
+ m_sizerBmpAndPage = new wxBoxSizer(wxHORIZONTAL);
+ mainColumn->Add(
+ m_sizerBmpAndPage,
+ 1, // Vertically stretchable
+ wxEXPAND // Horizonal stretching, no border
+ );
+ mainColumn->Add(0,5,
+ 0, // No vertical stretching
+ wxEXPAND // No border, (mostly useless) horizontal stretching
+ );
+
+#if wxUSE_STATBMP
+ if ( m_bitmap.Ok() )
{
- x -= sizeBtn.x;
- x -= BUTTON_MARGIN ;
-
- (void)new wxButton(this, wxID_HELP, _("&Help"), wxPoint(x, y), sizeBtn);
- x += sizeBtn.x;
- x += BUTTON_MARGIN ;
+ m_statbmp = new wxStaticBitmap(this, -1, m_bitmap);
+ m_sizerBmpAndPage->Add(
+ m_statbmp,
+ 0, // No horizontal stretching
+ wxALL, // Border all around, top alignment
+ 5 // Border width
+ );
+ m_sizerBmpAndPage->Add(
+ 5,0,
+ 0, // No horizontal stretching
+ wxEXPAND // No border, (mostly useless) vertical stretching
+ );
}
+#endif
- m_btnPrev = new wxButton(this, wxID_BACKWARD, _("< &Back"), wxPoint(x, y), sizeBtn);
+ // Added to m_sizerBmpAndPage in FinishLayout
+ m_sizerPage = new wxWizardSizer(this);
+}
- x += sizeBtn.x;
- x += BACKNEXT_MARGIN;
+void wxWizard::AddStaticLine(wxBoxSizer *mainColumn)
+{
+#if wxUSE_STATLINE
+ mainColumn->Add(
+ new wxStaticLine(this, -1),
+ 0, // Vertically unstretchable
+ wxEXPAND | wxALL, // Border all around, horizontally stretchable
+ 5 // Border width
+ );
+ mainColumn->Add(0,5,
+ 0, // No vertical stretching
+ wxEXPAND // No border, (mostly useless) horizontal stretching
+ );
+#else
+ (void)mainColumn;
+#endif // wxUSE_STATLINE
+}
- m_btnNext = new wxButton(this, wxID_FORWARD, _("&Next >"), wxPoint(x, y), sizeBtn);
+void wxWizard::AddBackNextPair(wxBoxSizer *buttonRow)
+{
+ wxASSERT_MSG( m_btnNext && m_btnPrev,
+ _T("You must create the buttons before calling ")
+ _T("wxWizard::AddBackNextPair") );
- x += sizeBtn.x + BUTTON_MARGIN;
- (void)new wxButton(this, wxID_CANCEL, _("&Cancel"), wxPoint(x, y), sizeBtn);
+ // margin between Back and Next buttons
+#ifdef __WXMAC__
+ static const int BACKNEXT_MARGIN = 10;
+#else
+ static const int BACKNEXT_MARGIN = 0;
+#endif
- // position and size the dialog
- // ----------------------------
+ wxBoxSizer *backNextPair = new wxBoxSizer(wxHORIZONTAL);
+ buttonRow->Add(
+ backNextPair,
+ 0, // No horizontal stretching
+ wxALL, // Border all around
+ 5 // Border width
+ );
+
+ backNextPair->Add(m_btnPrev);
+ backNextPair->Add(BACKNEXT_MARGIN,0,
+ 0, // No horizontal stretching
+ wxEXPAND // No border, (mostly useless) vertical stretching
+ );
+ backNextPair->Add(m_btnNext);
+}
- SetClientSize(m_x + m_width + X_MARGIN,
- m_y + m_height + BITMAP_Y_MARGIN +
- SEPARATOR_LINE_MARGIN + sizeBtn.y + Y_MARGIN);
+void wxWizard::AddButtonRow(wxBoxSizer *mainColumn)
+{
+ // the order in which the buttons are created determines the TAB order - at least under MSWindows...
+ // although the 'back' button appears before the 'next' button, a more userfriendly tab order is
+ // to activate the 'next' button first (create the next button before the back button).
+ // The reason is: The user will repeatedly enter information in the wizard pages and then wants to
+ // press 'next'. If a user uses mostly the keyboard, he would have to skip the 'back' button
+ // everytime. This is annoying. There is a second reason: RETURN acts as TAB. If the 'next'
+ // button comes first in the TAB order, the user can enter information very fast using the RETURN
+ // key to TAB to the next entry field and page. This would not be possible, if the 'back' button
+ // was created before the 'next' button.
+
+ wxBoxSizer *buttonRow = new wxBoxSizer(wxHORIZONTAL);
+ mainColumn->Add(
+ buttonRow,
+ 0, // Vertically unstretchable
+ wxALIGN_RIGHT // Right aligned, no border
+ );
+
+ // Desired TAB order is 'next', 'cancel', 'help', 'back'. This makes the 'back' button the last control on the page.
+ // Create the buttons in the right order...
+ m_btnNext = new wxButton(this, wxID_FORWARD, _("&Next >"));
+ wxButton *btnCancel=new wxButton(this, wxID_CANCEL, _("&Cancel"));
+ wxButton *btnHelp=0;
+ if (GetExtraStyle() & wxWIZARD_EX_HELPBUTTON)
+ btnHelp=new wxButton(this, wxID_HELP, _("&Help"));
+ m_btnPrev = new wxButton(this, wxID_BACKWARD, _("< &Back"));
+
+ if (btnHelp)
+ buttonRow->Add(
+ btnHelp,
+ 0, // Horizontally unstretchable
+ wxALL, // Border all around, top aligned
+ 5 // Border width
+ );
+
+ AddBackNextPair(buttonRow);
+
+ buttonRow->Add(
+ btnCancel,
+ 0, // Horizontally unstretchable
+ wxALL, // Border all around, top aligned
+ 5 // Border width
+ );
+}
- if ( m_posWizard == wxDefaultPosition )
- {
- CentreOnScreen();
- }
+void wxWizard::DoCreateControls()
+{
+ // do nothing if the controls were already created
+ if ( WasCreated() )
+ return;
+
+ // wxWindow::SetSizer will be called at end
+ wxBoxSizer *windowSizer = new wxBoxSizer(wxVERTICAL);
+
+ wxBoxSizer *mainColumn = new wxBoxSizer(wxVERTICAL);
+ windowSizer->Add(
+ mainColumn,
+ 1, // Vertical stretching
+ wxALL | wxEXPAND, // Border all around, horizontal stretching
+ 5 // Border width
+ );
+
+ AddBitmapRow(mainColumn);
+ AddStaticLine(mainColumn);
+ AddButtonRow(mainColumn);
+
+ // wxWindow::SetSizer should be followed by wxWindow::Fit, but
+ // this is done in FinishLayout anyway so why duplicate it
+ SetSizer(windowSizer);
}
void wxWizard::SetPageSize(const wxSize& size)
{
- // otherwise it will have no effect now as it's too late...
- wxASSERT_MSG( !WasCreated(), _T("should be called before RunWizard()!") );
-
+ wxCHECK_RET(!m_started,wxT("wxWizard::SetPageSize after RunWizard"));
m_sizePage = size;
}
-void wxWizard::FitToPage(const wxWizardPage *page)
+void wxWizard::FinishLayout()
{
- // otherwise it will have no effect now as it's too late...
- wxASSERT_MSG( !WasCreated(), _T("should be called before RunWizard()!") );
+ m_sizerBmpAndPage->Add(
+ m_sizerPage,
+ 1, // Horizontal stretching
+ wxEXPAND | wxALL, // Vertically stretchable
+ m_sizerPage->Border()
+ );
+
+ GetSizer()->SetSizeHints(this);
+ if ( m_posWizard == wxDefaultPosition )
+ CentreOnScreen();
+}
- wxSize sizeMax;
+void wxWizard::FitToPage(const wxWizardPage *page)
+{
+ wxCHECK_RET(!m_started,wxT("wxWizard::FitToPage after RunWizard"));
+
while ( page )
{
wxSize size = page->GetBestSize();
- if ( size.x > sizeMax.x )
- sizeMax.x = size.x;
-
- if ( size.y > sizeMax.y )
- sizeMax.y = size.y;
+ m_sizePage.IncTo(size);
page = page->GetNext();
}
-
- if ( sizeMax.x > m_sizePage.x )
- m_sizePage.x = sizeMax.x;
-
- if ( sizeMax.y > m_sizePage.y )
- m_sizePage.y = sizeMax.y;
}
bool wxWizard::ShowPage(wxWizardPage *page, bool goingForward)
{
// terminate successfully
EndModal(wxID_OK);
- if ( !IsModal() )
- {
- wxWizardEvent event(wxEVT_WIZARD_FINISHED, GetId(),FALSE, 0);
- (void)GetEventHandler()->ProcessEvent(event);
- }
+
+ // and notify the user code (this is especially useful for modeless
+ // wizards)
+ wxWizardEvent event(wxEVT_WIZARD_FINISHED, GetId(), FALSE, 0);
+ (void)GetEventHandler()->ProcessEvent(event);
+
return TRUE;
}
// position and show the new page
(void)m_page->TransferDataToWindow();
- m_page->SetSize(m_x, m_y, m_width, m_height);
+
+ // wxWizardSizer::RecalcSizes wants to be called when m_page changes
+ m_sizerPage->RecalcSizes();
// check if bitmap needs to be updated
// update default flag as well
bmpIsDefault = FALSE;
}
+#if wxUSE_STATBMP
// change the bitmap if:
// 1) a default bitmap was selected in constructor
// 2) this page was constructed with a bitmap
bmp = m_page->GetBitmap();
m_statbmp->SetBitmap(bmp);
}
+#endif
// and update the buttons state
m_btnPrev->Enable(HasPrevPage(m_page));
bool wxWizard::RunWizard(wxWizardPage *firstPage)
{
wxCHECK_MSG( firstPage, FALSE, wxT("can't run empty wizard") );
-
- DoCreateControls();
-
+
+ // Set before FinishLayout to enable wxWizardSizer::GetMaxChildSize
+ m_started = true;
+
+ // This cannot be done sooner, because user can change layout options
+ // up to this moment
+ FinishLayout();
+
// can't return FALSE here because there is no old page
(void)ShowPage(firstPage, TRUE /* forward */);
wxSize wxWizard::GetPageSize() const
{
- // make sure that the controls are created because otherwise m_width and
- // m_height would be both still -1
- wxConstCast(this, wxWizard)->DoCreateControls();
+ wxSize pageSize(GetManualPageSize());
+ pageSize.IncTo(m_sizerPage->GetMaxChildSize());
+ return pageSize;
+}
- return wxSize(m_width, m_height);
+wxSizer *wxWizard::GetPageAreaSizer() const
+{
+ return m_sizerPage;
}
-void wxWizard::OnCancel(wxCommandEvent& WXUNUSED(event))
+void wxWizard::SetBorder(int border)
+{
+ wxCHECK_RET(!m_started,wxT("wxWizard::SetBorder after RunWizard"));
+
+ m_calledSetBorder = true;
+ m_border = border;
+}
+
+wxSize wxWizard::GetManualPageSize() const
+{
+ // default width and height of the page
+ static const int DEFAULT_PAGE_WIDTH = 270;
+ static const int DEFAULT_PAGE_HEIGHT = 290;
+
+ wxSize totalPageSize(DEFAULT_PAGE_WIDTH,DEFAULT_PAGE_HEIGHT);
+
+ totalPageSize.IncTo(m_sizePage);
+
+ if ( m_statbmp )
+ {
+ totalPageSize.IncTo(wxSize(0, m_bitmap.GetHeight()));
+ }
+
+ return totalPageSize;
+}
+
+void wxWizard::OnCancel(wxCommandEvent& WXUNUSED(eventUnused))
{
// this function probably can never be called when we don't have an active
// page, but a small extra check won't hurt
// ask the current page first: notice that we do it before calling
// GetNext/Prev() because the data transfered from the controls of the page
// may change the value returned by these methods
- if ( m_page && !m_page->TransferDataFromWindow() )
+ if ( m_page && (!m_page->Validate() || !m_page->TransferDataFromWindow()) )
{
// the page data is incorrect, don't do anything
return;
if ( !(GetExtraStyle() & wxWS_EX_BLOCK_EVENTS) )
{
// the event will be propagated anyhow
+ event.Skip();
return;
}
// our public interface
// ----------------------------------------------------------------------------
-#ifdef WXWIN_COMPATIBILITY_2_2
+#if WXWIN_COMPATIBILITY_2_2
/* static */
wxWizard *wxWizardBase::Create(wxWindow *parent,