diff --git a/docs/changes.txt b/docs/changes.txt index e98ae23307..ad002b9a29 100644 --- a/docs/changes.txt +++ b/docs/changes.txt @@ -188,6 +188,7 @@ wxMac: - Fixed cursor for wxBusyCursor and wxContextHelp. - Fixed wxListCtrl to respect items' non-default fonts. +- wxListCtrl::SetColumnWidth() now supports wxLIST_AUTOSIZE. 2.8.7 diff --git a/include/wx/mac/carbon/listctrl.h b/include/wx/mac/carbon/listctrl.h index 6b5b7e4fbc..1af13e0ae6 100644 --- a/include/wx/mac/carbon/listctrl.h +++ b/include/wx/mac/carbon/listctrl.h @@ -419,6 +419,9 @@ protected: // keep track of inserted/deleted columns int m_count; // for virtual lists, store item count + +private: + int CalcColumnAutoWidth(int col) const; private: DECLARE_EVENT_TABLE() diff --git a/src/mac/carbon/listctrl_mac.cpp b/src/mac/carbon/listctrl_mac.cpp index 9ba30ee093..77e3ee4b91 100644 --- a/src/mac/carbon/listctrl_mac.cpp +++ b/src/mac/carbon/listctrl_mac.cpp @@ -1052,9 +1052,10 @@ bool wxListCtrl::SetColumnWidth(int col, int width) if (m_dbImpl) { - int mywidth = width; - if (width == wxLIST_AUTOSIZE || width == wxLIST_AUTOSIZE_USEHEADER) - mywidth = 150; + if ( width == wxLIST_AUTOSIZE_USEHEADER ) + { + width = 150; // FIXME + } if (col == -1) { @@ -1066,17 +1067,22 @@ bool wxListCtrl::SetColumnWidth(int col, int width) colInfo.SetWidth(width); SetColumn(column, colInfo); + const int mywidth = (width == wxLIST_AUTOSIZE) + ? CalcColumnAutoWidth(column) : width; m_dbImpl->SetColumnWidth(column, mywidth); } } else { + if ( width == wxLIST_AUTOSIZE ) + width = CalcColumnAutoWidth(col); + wxListItem colInfo; GetColumn(col, colInfo); - + colInfo.SetWidth(width); SetColumn(col, colInfo); - m_dbImpl->SetColumnWidth(col, mywidth); + m_dbImpl->SetColumnWidth(col, width); } return true; } @@ -3339,5 +3345,36 @@ void wxMacListCtrlItem::SetColumnInfo( unsigned int column, wxListItem* item ) } } +int wxListCtrl::CalcColumnAutoWidth(int col) const +{ + int width = 0; + + for ( int i = 0; i < GetItemCount(); i++ ) + { + wxListItem info; + info.SetMask(wxLIST_MASK_TEXT | wxLIST_MASK_IMAGE); + info.SetId(i); + info.SetColumn(col); + GetItem(info); + + const wxFont font = info.GetFont(); + + int w = 0; + if ( font.IsOk() ) + GetTextExtent(info.GetText(), &w, NULL, NULL, NULL, &font); + else + GetTextExtent(info.GetText(), &w, NULL); + + w += 2 * kItemPadding; + + if ( info.GetImage() != -1 ) + w += kIconWidth; + + width = wxMax(width, w); + } + + return width; +} + #endif // wxUSE_LISTCTRL