X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/dd61c39df27bfb6eff6864854a9819fc5a0cc2b4..6cef0db28018fd2644ee4e38af715872e5242459:/src/mac/carbon/choice.cpp diff --git a/src/mac/carbon/choice.cpp b/src/mac/carbon/choice.cpp index f2173eb03c..c395a39d19 100644 --- a/src/mac/carbon/choice.cpp +++ b/src/mac/carbon/choice.cpp @@ -17,13 +17,14 @@ #ifndef WX_PRECOMP #include "wx/menu.h" + #include "wx/dcclient.h" #endif #include "wx/mac/uma.h" extern MenuHandle NewUniqueMenu() ; -IMPLEMENT_DYNAMIC_CLASS(wxChoice, wxControl) +IMPLEMENT_DYNAMIC_CLASS(wxChoice, wxControlWithItems) wxChoice::~wxChoice() @@ -49,11 +50,17 @@ bool wxChoice::Create(wxWindow *parent, const wxValidator& validator, const wxString& name ) { - wxCArrayString chs( choices ); + if ( !Create( parent, id, pos, size, 0, NULL, style, validator, name ) ) + return false; + + Append( choices ); + + if ( !choices.empty() ) + SetSelection( 0 ); - return Create( - parent, id, pos, size, chs.GetCount(), chs.GetStrings(), - style, validator, name ); + SetInitialSize( size ); + + return true; } bool wxChoice::Create(wxWindow *parent, @@ -90,17 +97,14 @@ bool wxChoice::Create(wxWindow *parent, m_strings = wxArrayString( 1 ); #endif - for ( int i = 0; i < n; i++ ) - { - Append( choices[i] ); - } + Append(n, choices); // Set the first item as being selected if (n > 0) SetSelection( 0 ); // Needed because it is a wxControlWithItems - SetBestSize( size ); + SetInitialSize( size ); return true; } @@ -108,54 +112,45 @@ bool wxChoice::Create(wxWindow *parent, // ---------------------------------------------------------------------------- // adding/deleting items to/from the list // ---------------------------------------------------------------------------- -int wxChoice::DoAppend( const wxString& item ) -{ -#if wxUSE_STL - wxArrayString::iterator insertPoint; - unsigned int index; - if (GetWindowStyle() & wxCB_SORT) - { - insertPoint = std::lower_bound( m_strings.begin(), m_strings.end(), item ); - index = insertPoint - m_strings.begin(); - } - else +int wxChoice::DoInsertItems(const wxArrayStringsAdapter & items, + unsigned int pos, + void **clientData, wxClientDataType type) +{ + const unsigned int numItems = items.GetCount(); + for( unsigned int i = 0; i < numItems; ++i, ++pos ) { - insertPoint = m_strings.end(); - index = m_strings.size(); - } - - m_strings.insert( insertPoint, item ); -#else - unsigned int index = m_strings.Add( item ); -#endif - - m_datas.Insert( NULL , index ); - UMAInsertMenuItem( MAC_WXHMENU( m_macPopUpMenuHandle ), item, m_font.GetEncoding(), index ); - DoSetItemClientData( index, NULL ); - m_peer->SetMaximum( GetCount() ); - - return index; -} + unsigned int idx; -int wxChoice::DoInsert( const wxString& item, unsigned int pos ) -{ - wxCHECK_MSG( !(GetWindowStyle() & wxCB_SORT), -1, wxT("wxChoice::DoInsert: can't insert into sorted list") ); - wxCHECK_MSG( IsValidInsert(pos), -1, wxT("wxChoice::DoInsert: invalid index") ); +#if wxUSE_STL + if ( IsSorted() ) + { + wxArrayString::iterator + insertPoint = std::lower_bound( m_strings.begin(), m_strings.end(), items[i] ); + idx = insertPoint - m_strings.begin(); + m_strings.insert( insertPoint, items[i] ); + } + else +#endif // wxUSE_STL + { + idx = pos; + m_strings.Insert( items[i], idx ); + } - if (pos == GetCount()) - return DoAppend( item ); + UMAInsertMenuItem(MAC_WXHMENU( m_macPopUpMenuHandle ), + items[i], + GetFont().GetEncoding(), + idx); + m_datas.Insert( NULL, idx ); + AssignNewItemClientData(idx, clientData, i, type); + } - UMAInsertMenuItem( MAC_WXHMENU( m_macPopUpMenuHandle ), item, m_font.GetEncoding(), pos ); - m_strings.Insert( item, pos ); - m_datas.Insert( NULL, pos ); - DoSetItemClientData( pos, NULL ); m_peer->SetMaximum( GetCount() ); - return pos; + return pos - 1; } -void wxChoice::Delete(unsigned int n) +void wxChoice::DoDeleteOneItem(unsigned int n) { wxCHECK_RET( IsValid(n) , wxT("wxChoice::Delete: invalid index") ); @@ -168,9 +163,8 @@ void wxChoice::Delete(unsigned int n) m_peer->SetMaximum( GetCount() ) ; } -void wxChoice::Clear() +void wxChoice::DoClear() { - FreeData(); for ( unsigned int i = 0 ; i < GetCount() ; i++ ) { ::DeleteMenuItem( MAC_WXHMENU(m_macPopUpMenuHandle) , 1 ) ; @@ -181,18 +175,6 @@ void wxChoice::Clear() m_peer->SetMaximum( 0 ) ; } -void wxChoice::FreeData() -{ - if ( HasClientObjectData() ) - { - unsigned int count = GetCount(); - for ( unsigned int n = 0; n < count; n++ ) - { - delete GetClientObject( n ); - } - } -} - // ---------------------------------------------------------------------------- // selection // ---------------------------------------------------------------------------- @@ -219,7 +201,7 @@ int wxChoice::FindString( const wxString& s, bool bCase ) const { #if !wxUSE_STL // Avoid assert for non-default args passed to sorted array Index - if ( HasFlag(wxCB_SORT) ) + if ( IsSorted() ) bCase = true; #endif @@ -260,16 +242,6 @@ void * wxChoice::DoGetItemClientData(unsigned int n) const return (void *)m_datas[n]; } -void wxChoice::DoSetItemClientObject(unsigned int n, wxClientData* clientData) -{ - DoSetItemClientData(n, clientData); -} - -wxClientData* wxChoice::DoGetItemClientObject(unsigned int n) const -{ - return (wxClientData*)DoGetItemClientData( n ) ; -} - wxInt32 wxChoice::MacControlHit( WXEVENTHANDLERREF WXUNUSED(handler) , WXEVENTREF WXUNUSED(event) ) { wxCommandEvent event( wxEVT_COMMAND_CHOICE_SELECTED, m_windowId ); @@ -299,48 +271,22 @@ wxSize wxChoice::DoGetBestSize() const int lbHeight = 20; int wLine; -#if TARGET_CARBON - long metric ; + SInt32 metric ; GetThemeMetric( kThemeMetricPopupButtonHeight , &metric ); lbHeight = metric ; -#endif { - wxMacPortStateHelper st( UMAGetWindowPort( (WindowRef) MacGetTopLevelWindowRef() ) ) ; - if ( m_font.Ok() ) - { - ::TextFont( m_font.MacGetFontNum() ) ; - ::TextSize( m_font.MacGetFontSize() ) ; - ::TextFace( m_font.MacGetFontStyle() ) ; - } - else - { - ::TextFont( kFontIDMonaco ) ; - ::TextSize( 9 ) ; - ::TextFace( 0 ) ; - } + wxClientDC dc(const_cast(this)); // Find the widest line for(unsigned int i = 0; i < GetCount(); i++) { wxString str(GetString(i)); -#if wxUSE_UNICODE - Point bounds = { 0, 0 } ; - SInt16 baseline ; - - ::GetThemeTextDimensions( wxMacCFStringHolder( str , m_font.GetEncoding() ) , - kThemeCurrentPortFont, - kThemeStateActive, - false, - &bounds, - &baseline ); - - wLine = bounds.h ; -#else - wLine = ::TextWidth( str.c_str() , 0 , str.length() ) ; -#endif + wxCoord width, height ; + dc.GetTextExtent( str , &width, &height); + wLine = width ; lbWidth = wxMax( lbWidth, wLine ) ; } @@ -348,8 +294,11 @@ wxSize wxChoice::DoGetBestSize() const // Add room for the popup arrow lbWidth += 2 * lbHeight ; - // And just a bit more - int cx = ::TextWidth( "X" , 0 , 1 ) ; + wxCoord width, height ; + dc.GetTextExtent( wxT("X"), &width, &height); + int cx = width ; + lbHeight += 4; + lbWidth += cx ; }