// ----------------------------------------------------------------------------
IMPLEMENT_DYNAMIC_CLASS(wxListbook, wxBookCtrlBase)
-IMPLEMENT_DYNAMIC_CLASS(wxListbookEvent, wxNotifyEvent)
-const wxEventType wxEVT_COMMAND_LISTBOOK_PAGE_CHANGING = wxNewEventType();
-const wxEventType wxEVT_COMMAND_LISTBOOK_PAGE_CHANGED = wxNewEventType();
+wxDEFINE_EVENT( wxEVT_LISTBOOK_PAGE_CHANGING, wxBookCtrlEvent );
+wxDEFINE_EVENT( wxEVT_LISTBOOK_PAGE_CHANGED, wxBookCtrlEvent );
BEGIN_EVENT_TABLE(wxListbook, wxBookCtrlBase)
EVT_SIZE(wxListbook::OnSize)
// wxListbook creation
// ----------------------------------------------------------------------------
-void wxListbook::Init()
-{
- m_selection = wxNOT_FOUND;
-}
-
bool
wxListbook::Create(wxWindow *parent,
wxWindowID id,
wxID_ANY,
wxDefaultPosition,
wxDefaultSize,
- wxLC_ICON | wxLC_SINGLE_SEL |
- (IsVertical() ? wxLC_ALIGN_LEFT : wxLC_ALIGN_TOP)
+ GetListCtrlFlags()
);
+ if ( GetListView()->InReportView() )
+ GetListView()->InsertColumn(0, wxS("Pages"));
+
#ifdef __WXMSW__
// On XP with themes enabled the GetViewRect used in GetControllerSize() to
// determine the space needed for the list view will incorrectly return
}
// ----------------------------------------------------------------------------
-// wxListbook geometry management
+// wxListCtrl flags
// ----------------------------------------------------------------------------
-wxSize wxListbook::GetControllerSize() const
+long wxListbook::GetListCtrlFlags() const
{
- const wxSize sizeClient = GetClientSize(),
- sizeBorder = m_bookctrl->GetSize() - m_bookctrl->GetClientSize(),
- sizeList = GetListView()->GetViewRect().GetSize() + sizeBorder;
-
- wxSize size;
-
- if ( IsVertical() )
+ // We'd like to always use wxLC_ICON mode but it doesn't work with the
+ // native wxListCtrl under MSW unless we do have icons for all the items,
+ // so we can't use it if we have no image list. In this case we'd like to
+ // use wxLC_LIST mode because it works correctly for both horizontally and
+ // vertically laid out controls, but MSW native wxListCtrl insists on
+ // creating multiple columns if there are too many items and there doesn't
+ // seem anything to do about it, so we have to use wxLC_REPORT mode in this
+ // case there.
+
+ long flags = IsVertical() ? wxLC_ALIGN_LEFT : wxLC_ALIGN_TOP;
+ if ( GetImageList() )
{
- size.x = sizeClient.x;
- size.y = sizeList.y;
+ flags |= wxLC_ICON;
}
- else // left/right aligned
+ else // No images.
{
- size.x = sizeList.x;
- size.y = sizeClient.y;
+#ifdef __WXMSW__
+ if ( !IsVertical() )
+ {
+ // Notice that we intentionally overwrite the alignment flags here
+ // by not using "|=", alignment isn't used for report view.
+ flags = wxLC_REPORT | wxLC_NO_HEADER;
+ }
+ else
+#endif // __WXMSW__
+ {
+ flags |= wxLC_LIST;
+ }
}
- return size;
+ // Use single selection in any case.
+ return flags | wxLC_SINGLE_SEL;
}
+// ----------------------------------------------------------------------------
+// wxListbook geometry management
+// ----------------------------------------------------------------------------
+
void wxListbook::OnSize(wxSizeEvent& event)
{
// arrange the icons before calling SetClientSize(), otherwise it wouldn't
// under MSW, we'd finish with an ugly looking list control with both
// vertical and horizontal scrollbar (with one of them being added because
// the other one is not accounted for in client size computations)
- wxListView *list = GetListView();
- if (list) list->Arrange();
- wxBookCtrlBase::OnSize(event);
+ wxListView * const list = GetListView();
+ if ( list )
+ list->Arrange();
+
+ event.Skip();
}
int wxListbook::HitTest(const wxPoint& pt, long *flags) const
return pagePos;
}
-wxSize wxListbook::CalcSizeFromPage(const wxSize& sizePage) const
+void wxListbook::UpdateSize()
{
- // we need to add the size of the list control and the border between
- const wxSize sizeList = GetControllerSize();
-
- wxSize size = sizePage;
- if ( IsVertical() )
- {
- size.y += sizeList.y + GetInternalBorder();
- }
- else // left/right aligned
- {
- size.x += sizeList.x + GetInternalBorder();
- }
-
- return size;
+ // we should find a more elegant way to force a layout than generating this
+ // dummy event
+ wxSizeEvent sz(GetSize(), GetId());
+ GetEventHandler()->ProcessEvent(sz);
}
-
// ----------------------------------------------------------------------------
// accessing the pages
// ----------------------------------------------------------------------------
return GetListView()->GetItemText(n);
}
-int wxListbook::GetPageImage(size_t WXUNUSED(n)) const
+int wxListbook::GetPageImage(size_t n) const
{
- wxFAIL_MSG( _T("wxListbook::GetPageImage() not implemented") );
+ wxListItem item;
+ item.SetId(n);
- return wxNOT_FOUND;
+ if (GetListView()->GetItem(item))
+ {
+ return item.GetImage();
+ }
+ else
+ {
+ return wxNOT_FOUND;
+ }
}
bool wxListbook::SetPageImage(size_t n, int imageId)
void wxListbook::SetImageList(wxImageList *imageList)
{
- GetListView()->SetImageList(imageList, wxIMAGE_LIST_NORMAL);
+ const long flagsOld = GetListCtrlFlags();
wxBookCtrlBase::SetImageList(imageList);
+
+ const long flagsNew = GetListCtrlFlags();
+
+ wxListView * const list = GetListView();
+
+ // We may need to change the list control mode if the image list presence
+ // has changed.
+ if ( flagsNew != flagsOld )
+ {
+ // Preserve the selection which is lost when changing the mode
+ const int oldSel = GetSelection();
+
+ list->SetWindowStyleFlag(flagsNew);
+ if ( list->InReportView() )
+ list->InsertColumn(0, wxS("Pages"));
+
+ // Restore selection
+ if ( oldSel != wxNOT_FOUND )
+ SetSelection(oldSel);
+ }
+
+ list->SetImageList(imageList, wxIMAGE_LIST_NORMAL);
}
// ----------------------------------------------------------------------------
GetListView()->Focus(newsel);
}
-int wxListbook::GetSelection() const
+wxBookCtrlEvent* wxListbook::CreatePageChangingEvent() const
{
- return m_selection;
+ return new wxBookCtrlEvent(wxEVT_LISTBOOK_PAGE_CHANGING, m_windowId);
}
-wxBookCtrlBaseEvent* wxListbook::CreatePageChangingEvent() const
+void wxListbook::MakeChangedEvent(wxBookCtrlEvent &event)
{
- return new wxListbookEvent(wxEVT_COMMAND_LISTBOOK_PAGE_CHANGING, m_windowId);
-}
-
-void wxListbook::MakeChangedEvent(wxBookCtrlBaseEvent &event)
-{
- event.SetEventType(wxEVT_COMMAND_LISTBOOK_PAGE_CHANGED);
+ event.SetEventType(wxEVT_LISTBOOK_PAGE_CHANGED);
}
GetListView()->Focus(m_selection);
}
- // some page should be selected: either this one or the first one if there
- // is still no selection
- int selNew = -1;
- if ( bSelect )
- selNew = n;
- else if ( m_selection == -1 )
- selNew = 0;
-
- if ( selNew != m_selection )
+ if ( !DoSetSelectionAfterInsertion(n, bSelect) )
page->Hide();
- if ( selNew != -1 )
- SetSelection(selNew);
+ UpdateSize();
- wxSizeEvent sz(GetSize(), GetId());
- GetEventHandler()->ProcessEvent(sz);
-
return true;
}
wxWindow *wxListbook::DoRemovePage(size_t page)
{
- const size_t page_count = GetPageCount();
wxWindow *win = wxBookCtrlBase::DoRemovePage(page);
if ( win )
{
GetListView()->DeleteItem(page);
- if (m_selection >= (int)page)
- {
- // force new sel valid if possible
- int sel = m_selection - 1;
- if (page_count == 1)
- sel = wxNOT_FOUND;
- else if ((page_count == 2) || (sel == -1))
- sel = 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 ((sel != wxNOT_FOUND) && (sel != m_selection))
- SetSelection(sel);
- }
+ DoSetSelectionAfterRemoval(page);
GetListView()->Arrange();
- if (GetPageCount() == 0)
- {
- wxSizeEvent sz(GetSize(), GetId());
- GetEventHandler()->ProcessEvent(sz);
- }
+ UpdateSize();
}
return win;
if (!wxBookCtrlBase::DeleteAllPages())
return false;
- m_selection = -1;
-
- wxSizeEvent sz(GetSize(), GetId());
- GetEventHandler()->ProcessEvent(sz);
+ UpdateSize();
return true;
}
projects / wxWidgets.git / blobdiff