AuroraMiddleware/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Account for largest item in wxGenericListCtrl::GetBestSize().

Browse Source 
Don't just return a hard-coded value but at least return something big
enough to show the largest item in the control in non-report mode.

This fixes the appearance of wxListbook which simply truncated its items
before if they didn't fit in 80 pixels horizontally.

Also switch to implementing DoGetBestClientSize() instead of DoGetBestSize()
as this method doesn't account for the control borders (it does account for
the scrollbars however).

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@64884 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin 2010-07-11 10:44:08 +00:00
parent 89b799cccf
commit 3dea8ba7ac
2 changed files with 57 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
include/wx/generic/listctrl.h
View File

@ -221,7 +221,7 @@ protected:
virtual void DoClientToScreen( int *x, int *y ) const;
virtual void DoScreenToClient( int *x, int *y ) const;
virtual wxSize DoGetBestSize() const;
virtual wxSize DoGetBestClientSize() const;
// return the text for the given column of the given item
virtual wxString OnGetItemText(long item, long column) const;

61
src/generic/listctrl.cpp
View File

@ -5078,12 +5078,63 @@ void wxGenericListCtrl::SetFocus()
m_mainWin->SetFocus();
}
wxSize wxGenericListCtrl::DoGetBestSize() const
wxSize wxGenericListCtrl::DoGetBestClientSize() const
{
// Something is better than nothing...
// 100x80 is what the MSW version will get from the default
// wxControl::DoGetBestSize
return wxSize(100, 80);
// Something is better than nothing even if this is completely arbitrary.
wxSize sizeBest(100, 80);
if ( !InReportView() )
{
// Ensure that our minimal width is at least big enough to show all our
// items. This is important for wxListbook to size itself correctly.
// Remember the offset of the first item: this corresponds to the
// margins around the item so we will add it to the minimal size below
// to ensure that we have equal margins on all sides.
wxPoint ofs;
// We can iterate over all items as there shouldn't be too many of them
// in non-report view. If it ever becomes a problem, we could examine
// just the first few items probably, the determination of the best
// size is less important if we will need scrollbars anyhow.
for ( int n = 0; n < GetItemCount(); n++ )
{
const wxRect itemRect = m_mainWin->GetLineRect(n);
if ( !n )
{
// Remember the position of the first item as all the rest are
// offset by at least this number of pixels too.
ofs = itemRect.GetPosition();
}
sizeBest.IncTo(itemRect.GetSize());
}
sizeBest.IncBy(2*ofs);
// If we have the scrollbars we need to account for them too. And to
// make sure the scrollbars status is up to date we need to call this
// function to set them.
m_mainWin->RecalculatePositions(true /* no refresh */);
// Unfortunately we can't use wxWindow::HasScrollbar() here as we need
// to use m_mainWin client/virtual size for determination of whether we
// use scrollbars and not the size of this window itself. Maybe that
// function should be extended to work correctly in the case when our
// scrollbars manage a different window from this one but currently it
// doesn't work.
const wxSize sizeClient = m_mainWin->GetClientSize();
const wxSize sizeVirt = m_mainWin->GetVirtualSize();
if ( sizeVirt.x > sizeClient.x /* HasScrollbar(wxHORIZONTAL) */ )
sizeBest.y += wxSystemSettings::GetMetric(wxSYS_HSCROLL_Y);
if ( sizeVirt.y > sizeClient.y /* HasScrollbar(wxVERTICAL) */ )
sizeBest.x += wxSystemSettings::GetMetric(wxSYS_VSCROLL_X);
}
return sizeBest;
}
// ----------------------------------------------------------------------------