X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/159e6235b2bd2c9e991e9380622c9cdc21b0ead6..cf4ce62c26bcbd5a707e3054f611d0164617b012:/src/generic/choicbkg.cpp diff --git a/src/generic/choicbkg.cpp b/src/generic/choicbkg.cpp index 26f6759282..dc93e2b292 100644 --- a/src/generic/choicbkg.cpp +++ b/src/generic/choicbkg.cpp @@ -1,5 +1,5 @@ /////////////////////////////////////////////////////////////////////////////// -// Name: generic/choicbkg.cpp +// Name: src/generic/choicbkg.cpp // Purpose: generic implementation of wxChoicebook // Author: Vadim Zeitlin // Modified by: Wlodzimierz ABX Skiba from generic/listbkg.cpp @@ -26,10 +26,15 @@ #if wxUSE_CHOICEBOOK -#include "wx/choice.h" #include "wx/choicebk.h" + +#ifndef WX_PRECOMP + #include "wx/settings.h" + #include "wx/choice.h" + #include "wx/sizer.h" +#endif + #include "wx/imaglist.h" -#include "wx/settings.h" // ---------------------------------------------------------------------------- // various wxWidgets macros @@ -42,16 +47,13 @@ // event table // ---------------------------------------------------------------------------- -IMPLEMENT_DYNAMIC_CLASS(wxChoicebook, wxControl) -IMPLEMENT_DYNAMIC_CLASS(wxChoicebookEvent, wxNotifyEvent) +IMPLEMENT_DYNAMIC_CLASS(wxChoicebook, wxBookCtrlBase) -const wxEventType wxEVT_COMMAND_CHOICEBOOK_PAGE_CHANGING = wxNewEventType(); -const wxEventType wxEVT_COMMAND_CHOICEBOOK_PAGE_CHANGED = wxNewEventType(); -const int wxID_CHOICEBOOKCHOICE = wxNewId(); +wxDEFINE_EVENT( wxEVT_COMMAND_CHOICEBOOK_PAGE_CHANGING, wxBookCtrlEvent ); +wxDEFINE_EVENT( wxEVT_COMMAND_CHOICEBOOK_PAGE_CHANGED, wxBookCtrlEvent ); BEGIN_EVENT_TABLE(wxChoicebook, wxBookCtrlBase) - EVT_SIZE(wxChoicebook::OnSize) - EVT_CHOICE(wxID_CHOICEBOOKCHOICE, wxChoicebook::OnChoiceSelected) + EVT_CHOICE(wxID_ANY, wxChoicebook::OnChoiceSelected) END_EVENT_TABLE() // ============================================================================ @@ -64,7 +66,6 @@ END_EVENT_TABLE() void wxChoicebook::Init() { - m_choice = NULL; m_selection = wxNOT_FOUND; } @@ -76,9 +77,9 @@ wxChoicebook::Create(wxWindow *parent, long style, const wxString& name) { - if ( (style & wxCHB_ALIGN_MASK) == wxCHB_DEFAULT ) + if ( (style & wxBK_ALIGN_MASK) == wxBK_DEFAULT ) { - style |= wxCHB_TOP; + style |= wxBK_TOP; } // no border for this control, it doesn't look nice together with @@ -90,14 +91,23 @@ wxChoicebook::Create(wxWindow *parent, wxDefaultValidator, name) ) return false; - m_choice = new wxChoice + m_bookctrl = new wxChoice ( this, - wxID_CHOICEBOOKCHOICE, + wxID_ANY, wxDefaultPosition, wxDefaultSize ); + wxSizer* mainSizer = new wxBoxSizer(IsVertical() ? wxVERTICAL : wxHORIZONTAL); + + if (style & wxBK_RIGHT || style & wxBK_BOTTOM) + mainSizer->Add(0, 0, 1, wxEXPAND, 0); + + m_controlSizer = new wxBoxSizer(IsVertical() ? wxHORIZONTAL : wxVERTICAL); + m_controlSizer->Add(m_bookctrl, 1, (IsVertical() ? wxALIGN_CENTRE_VERTICAL : wxALIGN_CENTRE) |wxGROW, 0); + mainSizer->Add(m_controlSizer, 0, (IsVertical() ? (int) wxGROW : (int) wxALIGN_CENTRE_VERTICAL)|wxALL, m_controlMargin); + SetSizer(mainSizer); return true; } @@ -105,10 +115,10 @@ wxChoicebook::Create(wxWindow *parent, // wxChoicebook geometry management // ---------------------------------------------------------------------------- -wxSize wxChoicebook::GetChoiceSize() const +wxSize wxChoicebook::GetControllerSize() const { const wxSize sizeClient = GetClientSize(), - sizeChoice = m_choice->GetBestFittingSize(); + sizeChoice = m_controlSizer->CalcMin(); wxSize size; if ( IsVertical() ) @@ -125,98 +135,23 @@ wxSize wxChoicebook::GetChoiceSize() const return size; } -wxRect wxChoicebook::GetPageRect() const -{ - const wxSize sizeChoice = m_choice->GetBestFittingSize(); - - wxPoint pt; - wxRect rectPage(pt, GetClientSize()); - switch ( GetWindowStyle() & wxCHB_ALIGN_MASK ) - { - default: - wxFAIL_MSG( _T("unexpected wxChoicebook alignment") ); - // fall through - - case wxCHB_TOP: - rectPage.y = sizeChoice.y + GetInternalBorder(); - // fall through - - case wxCHB_BOTTOM: - rectPage.height -= sizeChoice.y + GetInternalBorder(); - break; - - case wxCHB_LEFT: - rectPage.x = sizeChoice.x + GetInternalBorder(); - // fall through - - case wxCHB_RIGHT: - rectPage.width -= sizeChoice.x + GetInternalBorder(); - break; - } - - return rectPage; -} - -void wxChoicebook::OnSize(wxSizeEvent& event) -{ - event.Skip(); - - if ( !m_choice ) - { - // we're not fully created yet - return; - } - - // resize the choice control and the page area to fit inside our new size - const wxSize sizeClient = GetClientSize(), - sizeChoice = GetChoiceSize(); - - wxPoint posChoice; - switch ( GetWindowStyle() & wxCHB_ALIGN_MASK ) - { - default: - wxFAIL_MSG( _T("unexpected wxChoicebook alignment") ); - // fall through - - case wxCHB_TOP: - case wxCHB_LEFT: - // posChoice is already ok - break; - - case wxCHB_BOTTOM: - posChoice.y = sizeClient.y - sizeChoice.y; - break; - - case wxCHB_RIGHT: - posChoice.x = sizeClient.x - sizeChoice.x; - break; - } - - m_choice->Move(posChoice); - m_choice->SetSize(sizeChoice); - - // resize the currently shown page - if ( m_selection != wxNOT_FOUND ) - { - wxWindow *page = m_pages[m_selection]; - wxCHECK_RET( page, _T("NULL page in wxChoicebook?") ); - page->SetSize(GetPageRect()); - } -} - wxSize wxChoicebook::CalcSizeFromPage(const wxSize& sizePage) const { // we need to add the size of the choice control and the border between - const wxSize sizeChoice = GetChoiceSize(); + const wxSize sizeChoice = GetControllerSize(); wxSize size = sizePage; if ( IsVertical() ) { + if ( sizeChoice.x > sizePage.x ) + size.x = sizeChoice.x; size.y += sizeChoice.y + GetInternalBorder(); } else // left/right aligned { size.x += sizeChoice.x + GetInternalBorder(); + if ( sizeChoice.y > sizePage.y ) + size.y = sizeChoice.y; } return size; @@ -229,14 +164,14 @@ wxSize wxChoicebook::CalcSizeFromPage(const wxSize& sizePage) const bool wxChoicebook::SetPageText(size_t n, const wxString& strText) { - m_choice->SetString(n, strText); + GetChoiceCtrl()->SetString(n, strText); return true; } wxString wxChoicebook::GetPageText(size_t n) const { - return m_choice->GetString(n); + return GetChoiceCtrl()->GetString(n); } int wxChoicebook::GetPageImage(size_t WXUNUSED(n)) const @@ -254,9 +189,16 @@ bool wxChoicebook::SetPageImage(size_t WXUNUSED(n), int WXUNUSED(imageId)) } // ---------------------------------------------------------------------------- -// image list stuff +// miscellaneous other stuff // ---------------------------------------------------------------------------- +void wxChoicebook::DoSetWindowVariant(wxWindowVariant variant) +{ + wxBookCtrlBase::DoSetWindowVariant(variant); + if (m_bookctrl) + m_bookctrl->SetWindowVariant(variant); +} + void wxChoicebook::SetImageList(wxImageList *imageList) { // TODO: can be implemented in form of static bitmap near choice control @@ -273,39 +215,14 @@ int wxChoicebook::GetSelection() const return m_selection; } -int wxChoicebook::SetSelection(size_t n) +wxBookCtrlEvent* wxChoicebook::CreatePageChangingEvent() const { - wxCHECK_MSG( IS_VALID_PAGE(n), wxNOT_FOUND, - wxT("invalid page index in wxChoicebook::SetSelection()") ); - - const int oldSel = m_selection; - - if ( int(n) != m_selection ) - { - wxChoicebookEvent event(wxEVT_COMMAND_CHOICEBOOK_PAGE_CHANGING, m_windowId); - event.SetSelection(n); - event.SetOldSelection(m_selection); - event.SetEventObject(this); - if ( !GetEventHandler()->ProcessEvent(event) || event.IsAllowed() ) - { - if ( m_selection != wxNOT_FOUND ) - m_pages[m_selection]->Hide(); - - wxWindow *page = m_pages[n]; - page->SetSize(GetPageRect()); - page->Show(); - - // change m_selection now to ignore the selection change event - m_selection = n; - m_choice->Select(n); - - // program allows the page change - event.SetEventType(wxEVT_COMMAND_CHOICEBOOK_PAGE_CHANGED); - (void)GetEventHandler()->ProcessEvent(event); - } - } + return new wxBookCtrlEvent(wxEVT_COMMAND_CHOICEBOOK_PAGE_CHANGING, m_windowId); +} - return oldSel; +void wxChoicebook::MakeChangedEvent(wxBookCtrlEvent &event) +{ + event.SetEventType(wxEVT_COMMAND_CHOICEBOOK_PAGE_CHANGED); } // ---------------------------------------------------------------------------- @@ -322,7 +239,7 @@ wxChoicebook::InsertPage(size_t n, if ( !wxBookCtrlBase::InsertPage(n, page, text, bSelect, imageId) ) return false; - m_choice->Insert(text, n); + GetChoiceCtrl()->Insert(text, n); // if the inserted page is before the selected one, we must update the // index of the selected page @@ -330,7 +247,7 @@ wxChoicebook::InsertPage(size_t n, { // one extra page added m_selection++; - m_choice->Select(m_selection); + GetChoiceCtrl()->Select(m_selection); } // some page should be selected: either this one or the first one if there @@ -347,34 +264,33 @@ wxChoicebook::InsertPage(size_t n, if ( selNew != wxNOT_FOUND ) SetSelection(selNew); - InvalidateBestSize(); return true; } wxWindow *wxChoicebook::DoRemovePage(size_t page) { - const size_t page_count = GetPageCount(); wxWindow *win = wxBookCtrlBase::DoRemovePage(page); if ( win ) { - m_choice->Delete(page); + GetChoiceCtrl()->Delete(page); - if (m_selection >= (int)page) + if ( m_selection >= (int)page ) { - // force new sel valid if possible - int sel = m_selection - 1; - if (page_count == 1) + // ensure that the selection is valid + int sel; + if ( GetPageCount() == 0 ) sel = wxNOT_FOUND; - else if ((page_count == 2) || (sel == -1)) - sel = 0; + else + sel = m_selection ? m_selection - 1 : 0; - // force sel invalid if deleting current page - don't try to hide it - m_selection = (m_selection == (int)page) ? wxNOT_FOUND : m_selection - 1; + // if deleting current page we shouldn't try to hide it + m_selection = m_selection == (int)page ? wxNOT_FOUND + : m_selection - 1; - if ((sel != wxNOT_FOUND) && (sel != m_selection)) + if ( sel != wxNOT_FOUND && sel != m_selection ) SetSelection(sel); - } + } } return win; @@ -383,7 +299,8 @@ wxWindow *wxChoicebook::DoRemovePage(size_t page) bool wxChoicebook::DeleteAllPages() { - m_choice->Clear(); + m_selection = wxNOT_FOUND; + GetChoiceCtrl()->Clear(); return wxBookCtrlBase::DeleteAllPages(); } @@ -393,6 +310,12 @@ bool wxChoicebook::DeleteAllPages() void wxChoicebook::OnChoiceSelected(wxCommandEvent& eventChoice) { + if ( eventChoice.GetEventObject() != m_bookctrl ) + { + eventChoice.Skip(); + return; + } + const int selNew = eventChoice.GetSelection(); if ( selNew == m_selection ) @@ -407,7 +330,7 @@ void wxChoicebook::OnChoiceSelected(wxCommandEvent& eventChoice) // change wasn't allowed, return to previous state if (m_selection != selNew) - m_choice->Select(m_selection); + GetChoiceCtrl()->Select(m_selection); } #endif // wxUSE_CHOICEBOOK