X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/de160b06478f7b007a28bd84f60fc59c0683ee09..6cef0db28018fd2644ee4e38af715872e5242459:/src/mac/carbon/combobox.cpp?ds=inline diff --git a/src/mac/carbon/combobox.cpp b/src/mac/carbon/combobox.cpp index 734e631e28..4db3c45a02 100644 --- a/src/mac/carbon/combobox.cpp +++ b/src/mac/carbon/combobox.cpp @@ -20,6 +20,7 @@ #include "wx/menu.h" #include "wx/containr.h" #include "wx/toplevel.h" + #include "wx/textctrl.h" #endif #include "wx/mac/uma.h" @@ -49,16 +50,11 @@ MenuHandle NewUniqueMenu() // ---------------------------------------------------------------------------- // the margin between the text control and the choice -#if TARGET_API_MAC_OSX // margin should be bigger on OS X due to blue highlight // around text control. static const wxCoord MARGIN = 4; // this is the border a focus rect on OSX is needing static const int TEXTFOCUSBORDER = 3 ; -#else -static const wxCoord MARGIN = 2; -static const int TEXTFOCUSBORDER = 0 ; -#endif // ---------------------------------------------------------------------------- @@ -87,14 +83,14 @@ protected: NavEvent.SetWindowChange(false); // Get the parent of the combo and have it process the navigation? - if (m_cb->GetParent()->GetEventHandler()->ProcessEvent(NavEvent)) + if (m_cb->GetParent()->HandleWindowEvent(NavEvent)) return; } // send the event to the combobox class in case the user has bound EVT_CHAR wxKeyEvent kevt(event); kevt.SetEventObject(m_cb); - if (m_cb->GetEventHandler()->ProcessEvent(kevt)) + if (m_cb->HandleWindowEvent(kevt)) // If the event was handled and not skipped then we're done return; @@ -107,7 +103,7 @@ protected: // This will invoke the dialog default action, // such as the clicking the default button. - if (!m_cb->GetEventHandler()->ProcessEvent( event )) + if (!m_cb->HandleWindowEvent( event )) { wxTopLevelWindow *tlw = wxDynamicCast(wxGetTopLevelParent(this), wxTopLevelWindow); if ( tlw && tlw->GetDefaultItem() ) @@ -132,7 +128,7 @@ protected: { event.SetEventObject(m_cb); event.SetId(m_cb->GetId()); - if (! m_cb->GetEventHandler()->ProcessEvent(event)) + if (! m_cb->HandleWindowEvent(event)) event.Skip(); } @@ -140,7 +136,7 @@ protected: { event.SetEventObject(m_cb); event.SetId(m_cb->GetId()); - if (! m_cb->GetEventHandler()->ProcessEvent(event)) + if (! m_cb->HandleWindowEvent(event)) event.Skip(); } @@ -148,7 +144,7 @@ protected: { event.SetEventObject(m_cb); event.SetId(m_cb->GetId()); - if (! m_cb->GetEventHandler()->ProcessEvent(event)) + if (! m_cb->HandleWindowEvent(event)) event.Skip(); } @@ -230,9 +226,6 @@ END_EVENT_TABLE() wxComboBox::~wxComboBox() { - // delete client objects - FreeData(); - // delete the controls now, don't leave them alive even though they would // still be eventually deleted by our parent - but it will be too late, the // user code expects them to be gone now @@ -347,10 +340,13 @@ bool wxComboBox::Create(wxWindow *parent, const wxValidator& validator, const wxString& name) { - wxCArrayString chs( choices ); + if ( !Create( parent, id, value, pos, size, 0, NULL, + style, validator, name ) ) + return false; - return Create( parent, id, value, pos, size, chs.GetCount(), - chs.GetStrings(), style, validator, name ); + Append(choices); + + return true; } bool wxComboBox::Create(wxWindow *parent, @@ -388,10 +384,7 @@ bool wxComboBox::Create(wxWindow *parent, DoSetSize(pos.x, pos.y, csize.x, csize.y); - for ( int i = 0 ; i < n ; i++ ) - { - m_choice->DoAppend( choices[ i ] ); - } + Append( n, choices ); // Needed because it is a wxControlWithItems SetInitialSize(size); @@ -425,6 +418,16 @@ void wxComboBox::SetValue(const wxString& value) m_text->SetValue( value ); } +void wxComboBox::WriteText(const wxString& text) +{ + m_text->WriteText(text); +} + +void wxComboBox::GetSelection(long *from, long *to) const +{ + m_text->GetSelection(from, to); +} + // Clipboard operations void wxComboBox::Copy() @@ -510,14 +513,12 @@ void wxComboBox::SetSelection(long from, long to) m_text->SetSelection(from,to); } -int wxComboBox::DoAppend(const wxString& item) -{ - return m_choice->DoAppend( item ) ; -} - -int wxComboBox::DoInsert(const wxString& item, unsigned int pos) +int wxComboBox::DoInsertItems(const wxArrayStringsAdapter& items, + unsigned int pos, + void **clientData, + wxClientDataType type) { - return m_choice->DoInsert( item , pos ) ; + return m_choice->DoInsertItems(items, pos, clientData, type); } void wxComboBox::DoSetItemClientData(unsigned int n, void* clientData) @@ -530,40 +531,24 @@ void* wxComboBox::DoGetItemClientData(unsigned int n) const return m_choice->DoGetItemClientData( n ) ; } -void wxComboBox::DoSetItemClientObject(unsigned int n, wxClientData* clientData) +wxClientDataType wxComboBox::GetClientDataType() const { - return m_choice->DoSetItemClientObject(n, clientData); + return m_choice->GetClientDataType(); } -wxClientData* wxComboBox::DoGetItemClientObject(unsigned int n) const +void wxComboBox::SetClientDataType(wxClientDataType clientDataItemsType) { - return m_choice->DoGetItemClientObject( n ) ; -} - -void wxComboBox::FreeData() -{ - if ( HasClientObjectData() ) - { - unsigned int count = GetCount(); - for ( unsigned int n = 0; n < count; n++ ) - { - SetClientObject( n, NULL ); - } - } + m_choice->SetClientDataType(clientDataItemsType); } -void wxComboBox::Delete(unsigned int n) +void wxComboBox::DoDeleteOneItem(unsigned int n) { - // force client object deletion - if( HasClientObjectData() ) - SetClientObject( n, NULL ); - m_choice->Delete( n ); + m_choice->DoDeleteOneItem( n ); } -void wxComboBox::Clear() +void wxComboBox::DoClear() { - FreeData(); - m_choice->Clear(); + m_choice->DoClear(); } int wxComboBox::GetSelection() const @@ -576,7 +561,7 @@ void wxComboBox::SetSelection(int n) m_choice->SetSelection( n ); if ( m_text != NULL ) - m_text->SetValue(GetString(n)); + m_text->SetValue(n != wxNOT_FOUND ? GetString(n) : wxString(wxEmptyString)); } int wxComboBox::FindString(const wxString& s, bool bCase) const