X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/8d700f716ff2a278f26adba71e03e5405c2113e5..7564225f8bff6e6c2ed27695836035fa85a81189:/src/mac/carbon/listctrl_mac.cpp diff --git a/src/mac/carbon/listctrl_mac.cpp b/src/mac/carbon/listctrl_mac.cpp index 16981635c9..5450de5164 100644 --- a/src/mac/carbon/listctrl_mac.cpp +++ b/src/mac/carbon/listctrl_mac.cpp @@ -1,5 +1,5 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: src/msw/listctrl.cpp +// Name: src/mac/listctrl_mac.cpp // Purpose: wxListCtrl // Author: Julian Smart // Modified by: Agron Selimaj @@ -26,20 +26,17 @@ #if wxUSE_LISTCTRL +#include "wx/listctrl.h" + #ifndef WX_PRECOMP - #include "wx/app.h" #include "wx/intl.h" - #include "wx/log.h" - #include "wx/settings.h" - #include "wx/dcclient.h" - #include "wx/textctrl.h" #endif #include "wx/mac/uma.h" #include "wx/imaglist.h" -#include "wx/listctrl.h" #include "wx/sysopt.h" +#include "wx/timer.h" #define wxMAC_ALWAYS_USE_GENERIC_LISTCTRL wxT("mac.listctrl.always_use_generic") @@ -119,37 +116,90 @@ IMPLEMENT_DYNAMIC_CLASS(wxListEvent, wxNotifyEvent) WX_DECLARE_EXPORTED_LIST(wxListItem, wxListItemList); #include "wx/listimpl.cpp" WX_DEFINE_LIST(wxListItemList) +WX_DEFINE_LIST(wxColumnList) + +// so we can check for column clicks +static const EventTypeSpec eventList[] = +{ + { kEventClassControl, kEventControlHit }, + { kEventClassControl, kEventControlDraw } +}; + +static pascal OSStatus wxMacListCtrlEventHandler( EventHandlerCallRef handler , EventRef event , void *data ) +{ + OSStatus result = eventNotHandledErr ; + + wxMacCarbonEvent cEvent( event ) ; + + ControlRef controlRef ; + cEvent.GetParameter( kEventParamDirectObject , &controlRef ) ; + + wxListCtrl *window = (wxListCtrl*) data ; + wxListEvent le( wxEVT_COMMAND_LIST_COL_CLICK, window->GetId() ); + le.SetEventObject( window ); + + switch ( GetEventKind( event ) ) + { + // check if the column was clicked on and fire an event if so + case kEventControlHit : + { + ControlPartCode result = cEvent.GetParameter(kEventParamControlPart, typeControlPartCode) ; + if (result == kControlButtonPart){ + DataBrowserPropertyID col; + GetDataBrowserSortProperty(controlRef, &col); + int column = col - kMinColumnId; + le.m_col = column; + window->GetEventHandler()->ProcessEvent( le ); + } + result = CallNextEventHandler(handler, event); + break; + } + case kEventControlDraw: + { + CGContextRef context = cEvent.GetParameter(kEventParamCGContextRef, typeCGContextRef) ; + window->MacSetDrawingContext(context); + result = CallNextEventHandler(handler, event); + window->MacSetDrawingContext(NULL); + break; + } + default : + break ; + } + + + return result ; +} + +DEFINE_ONE_SHOT_HANDLER_GETTER( wxMacListCtrlEventHandler ) class wxMacListCtrlItem : public wxMacListBoxItem { public: wxMacListCtrlItem(); - + virtual void Notification(wxMacDataItemBrowserControl *owner , DataBrowserItemNotification message, DataBrowserItemDataRef itemData ) const; - + virtual void SetColumnInfo( unsigned int column, wxListItem* item ); virtual wxListItem* GetColumnInfo( unsigned int column ); virtual bool HasColumnInfo( unsigned int column ); - + virtual void SetColumnTextValue( unsigned int column, const wxString& text ); - virtual const wxString& GetColumnTextValue( unsigned int column ); - + virtual const wxString& GetColumnTextValue( unsigned int column ); + virtual int GetColumnImageValue( unsigned int column ); virtual void SetColumnImageValue( unsigned int column, int imageIndex ); - - virtual OSStatus GetSetData( wxMacDataItemBrowserControl *owner , - DataBrowserPropertyID property, - DataBrowserItemDataRef itemData, - bool changeValue ); - virtual ~wxMacListCtrlItem(); protected: wxListItemList m_rowItems; }; +DataBrowserDrawItemUPP gDataBrowserDrawItemUPP = NULL; +//DataBrowserEditItemUPP gDataBrowserEditItemUPP = NULL; +DataBrowserHitTestUPP gDataBrowserHitTestUPP = NULL; + // TODO: Make a better name!! class wxMacDataBrowserListCtrlControl : public wxMacDataItemBrowserControl { @@ -159,22 +209,298 @@ public: // create a list item (can be a subclass of wxMacListBoxItem) - virtual wxMacListCtrlItem* CreateItem(); - + virtual wxMacDataItem* CreateItem(); + virtual void MacInsertItem( unsigned int n, wxListItem* item ); virtual void MacSetColumnInfo( unsigned int row, unsigned int column, wxListItem* item ); virtual void MacGetColumnInfo( unsigned int row, unsigned int column, wxListItem& item ); virtual void UpdateState(wxMacDataItem* dataItem, wxListItem* item); - + protected: + // we need to override to provide specialized handling for virtual wxListCtrls + virtual OSStatus GetSetItemData(DataBrowserItemID itemID, + DataBrowserPropertyID property, + DataBrowserItemDataRef itemData, + Boolean changeValue ); + + virtual void ItemNotification( + DataBrowserItemID itemID, + DataBrowserItemNotification message, + DataBrowserItemDataRef itemData); + + virtual Boolean CompareItems(DataBrowserItemID itemOneID, + DataBrowserItemID itemTwoID, + DataBrowserPropertyID sortProperty); + + static pascal void DataBrowserDrawItemProc(ControlRef browser, + DataBrowserItemID item, + DataBrowserPropertyID property, + DataBrowserItemState itemState, + const Rect *theRect, + SInt16 gdDepth, + Boolean colorDevice); + + virtual void DrawItem(DataBrowserItemID itemID, + DataBrowserPropertyID property, + DataBrowserItemState itemState, + const Rect *itemRect, + SInt16 gdDepth, + Boolean colorDevice); + + static pascal Boolean DataBrowserEditTextProc(ControlRef browser, + DataBrowserItemID item, + DataBrowserPropertyID property, + CFStringRef theString, + Rect *maxEditTextRect, + Boolean *shrinkToFit); + + static pascal Boolean DataBrowserHitTestProc(ControlRef browser, + DataBrowserItemID itemID, + DataBrowserPropertyID property, + const Rect *theRect, + const Rect *mouseRect) { return true; } + + virtual bool ConfirmEditText(DataBrowserItemID item, + DataBrowserPropertyID property, + CFStringRef theString, + Rect *maxEditTextRect, + Boolean *shrinkToFit); + + + wxClientDataType m_clientDataItemsType; + bool m_isVirtual; + +}; + +class wxMacListCtrlEventDelegate : public wxEvtHandler +{ +public: + wxMacListCtrlEventDelegate( wxListCtrl* list, int id ); + virtual bool ProcessEvent( wxEvent& event ); + +private: + wxListCtrl* m_list; + int m_id; +}; + +wxMacListCtrlEventDelegate::wxMacListCtrlEventDelegate( wxListCtrl* list, int id ) +{ + m_list = list; + m_id = id; +} + +bool wxMacListCtrlEventDelegate::ProcessEvent( wxEvent& event ) +{ + // even though we use a generic list ctrl underneath, make sure + // we present ourselves as wxListCtrl. + event.SetEventObject( m_list ); + event.SetId( m_id ); + + if ( !event.IsKindOf( CLASSINFO( wxCommandEvent ) ) ) + { + if (m_list->GetEventHandler()->ProcessEvent( event )) + return true; + } + return wxEvtHandler::ProcessEvent(event); +} + +//----------------------------------------------------------------------------- +// wxListCtrlRenameTimer (internal) +//----------------------------------------------------------------------------- + +class wxListCtrlRenameTimer: public wxTimer +{ +private: + wxListCtrl *m_owner; + +public: + wxListCtrlRenameTimer( wxListCtrl *owner ); + void Notify(); +}; +//----------------------------------------------------------------------------- +// wxListCtrlTextCtrlWrapper: wraps a wxTextCtrl to make it work for inline editing +//----------------------------------------------------------------------------- + +class wxListCtrlTextCtrlWrapper : public wxEvtHandler +{ +public: + // NB: text must be a valid object but not Create()d yet + wxListCtrlTextCtrlWrapper(wxListCtrl *owner, + wxTextCtrl *text, + long itemEdit); + + wxTextCtrl *GetText() const { return m_text; } + + void AcceptChangesAndFinish(); + +protected: + void OnChar( wxKeyEvent &event ); + void OnKeyUp( wxKeyEvent &event ); + void OnKillFocus( wxFocusEvent &event ); + + bool AcceptChanges(); + void Finish(); + +private: + wxListCtrl *m_owner; + wxTextCtrl *m_text; + wxString m_startValue; + long m_itemEdited; + bool m_finished; + bool m_aboutToFinish; + + DECLARE_EVENT_TABLE() }; -// TODO: This gives errors, find out why. -//BEGIN_EVENT_TABLE(wxListCtrl, wxControl) -// EVT_PAINT(wxListCtrl::OnPaint) -//END_EVENT_TABLE() +//----------------------------------------------------------------------------- +// wxListCtrlRenameTimer (internal) +//----------------------------------------------------------------------------- + +wxListCtrlRenameTimer::wxListCtrlRenameTimer( wxListCtrl *owner ) +{ + m_owner = owner; +} + +void wxListCtrlRenameTimer::Notify() +{ + m_owner->OnRenameTimer(); +} + +//----------------------------------------------------------------------------- +// wxListCtrlTextCtrlWrapper (internal) +//----------------------------------------------------------------------------- + +BEGIN_EVENT_TABLE(wxListCtrlTextCtrlWrapper, wxEvtHandler) + EVT_CHAR (wxListCtrlTextCtrlWrapper::OnChar) + EVT_KEY_UP (wxListCtrlTextCtrlWrapper::OnKeyUp) + EVT_KILL_FOCUS (wxListCtrlTextCtrlWrapper::OnKillFocus) +END_EVENT_TABLE() + +wxListCtrlTextCtrlWrapper::wxListCtrlTextCtrlWrapper(wxListCtrl *owner, + wxTextCtrl *text, + long itemEdit) + : m_startValue(owner->GetItemText(itemEdit)), + m_itemEdited(itemEdit) +{ + m_owner = owner; + m_text = text; + m_finished = false; + m_aboutToFinish = false; + + wxRect rectLabel; + owner->GetItemRect(itemEdit, rectLabel); + + m_text->Create(owner, wxID_ANY, m_startValue, + wxPoint(rectLabel.x+8,rectLabel.y), + wxSize(rectLabel.width,rectLabel.height)); + m_text->SetFocus(); + + m_text->PushEventHandler(this); +} + +void wxListCtrlTextCtrlWrapper::Finish() +{ + if ( !m_finished ) + { + m_finished = true; + + m_text->RemoveEventHandler(this); + m_owner->FinishEditing(m_text); + + wxPendingDelete.Append( this ); + } +} + +bool wxListCtrlTextCtrlWrapper::AcceptChanges() +{ + const wxString value = m_text->GetValue(); + + if ( value == m_startValue ) + // nothing changed, always accept + return true; + + if ( !m_owner->OnRenameAccept(m_itemEdited, value) ) + // vetoed by the user + return false; + + // accepted, do rename the item + m_owner->SetItemText(m_itemEdited, value); + + return true; +} + +void wxListCtrlTextCtrlWrapper::AcceptChangesAndFinish() +{ + m_aboutToFinish = true; + + // Notify the owner about the changes + AcceptChanges(); + + // Even if vetoed, close the control (consistent with MSW) + Finish(); +} + +void wxListCtrlTextCtrlWrapper::OnChar( wxKeyEvent &event ) +{ + switch ( event.m_keyCode ) + { + case WXK_RETURN: + AcceptChangesAndFinish(); + break; + + case WXK_ESCAPE: + m_owner->OnRenameCancelled( m_itemEdited ); + Finish(); + break; + + default: + event.Skip(); + } +} + +void wxListCtrlTextCtrlWrapper::OnKeyUp( wxKeyEvent &event ) +{ + if (m_finished) + { + event.Skip(); + return; + } + + // auto-grow the textctrl: + wxSize parentSize = m_owner->GetSize(); + wxPoint myPos = m_text->GetPosition(); + wxSize mySize = m_text->GetSize(); + int sx, sy; + m_text->GetTextExtent(m_text->GetValue() + _T("MM"), &sx, &sy); + if (myPos.x + sx > parentSize.x) + sx = parentSize.x - myPos.x; + if (mySize.x > sx) + sx = mySize.x; + m_text->SetSize(sx, wxDefaultCoord); + + event.Skip(); +} + +void wxListCtrlTextCtrlWrapper::OnKillFocus( wxFocusEvent &event ) +{ + if ( !m_finished && !m_aboutToFinish ) + { + if ( !AcceptChanges() ) + m_owner->OnRenameCancelled( m_itemEdited ); + + Finish(); + } + + // We must let the native text control handle focus + event.Skip(); +} + +BEGIN_EVENT_TABLE(wxListCtrl, wxControl) + EVT_LEFT_DOWN(wxListCtrl::OnLeftDown) + EVT_LEFT_DCLICK(wxListCtrl::OnDblClick) +END_EVENT_TABLE() // ============================================================================ // implementation @@ -194,7 +520,7 @@ void wxListCtrl::Init() m_imageListNormal = NULL; m_imageListSmall = NULL; m_imageListState = NULL; - + // keep track of if we created our own image lists, or if they were assigned // to us. m_ownsImageListNormal = m_ownsImageListSmall = m_ownsImageListState = false; @@ -203,6 +529,83 @@ void wxListCtrl::Init() m_textCtrl = NULL; m_genericImpl = NULL; m_dbImpl = NULL; + m_compareFunc = NULL; + m_compareFuncData = 0; + m_colsInfo = wxColumnList(); + m_textColor = wxNullColour; + m_bgColor = wxNullColour; + m_textctrlWrapper = NULL; + m_current = -1; + m_renameTimer = new wxListCtrlRenameTimer( this ); +} + +class wxGenericListCtrlHook : public wxGenericListCtrl +{ +public: + wxGenericListCtrlHook(wxListCtrl* parent, + wxWindowID id, + const wxPoint& pos, + const wxSize& size, + long style, + const wxValidator& validator, + const wxString& name) + : wxGenericListCtrl(parent, id, pos, size, style, validator, name), + m_nativeListCtrl(parent) + { + } + +protected: + virtual wxListItemAttr * OnGetItemAttr(long item) const + { + return m_nativeListCtrl->OnGetItemAttr(item); + } + + virtual int OnGetItemImage(long item) const + { + return m_nativeListCtrl->OnGetItemImage(item); + } + + virtual int OnGetItemColumnImage(long item, long column) const + { + return m_nativeListCtrl->OnGetItemColumnImage(item, column); + } + + virtual wxString OnGetItemText(long item, long column) const + { + return m_nativeListCtrl->OnGetItemText(item, column); + } + + wxListCtrl* m_nativeListCtrl; + +}; + +void wxListCtrl::OnLeftDown(wxMouseEvent& event) +{ + if ( m_textctrlWrapper ) + { + m_current = -1; + m_textctrlWrapper->AcceptChangesAndFinish(); + } + + int hitResult; + long current = HitTest(event.GetPosition(), hitResult); + if ((current == m_current) && + (hitResult == wxLIST_HITTEST_ONITEM) && + HasFlag(wxLC_EDIT_LABELS) ) + { + m_renameTimer->Start( 100, true ); + } + else + { + m_current = current; + } + event.Skip(); +} + +void wxListCtrl::OnDblClick(wxMouseEvent& event) +{ + m_current = -1; + event.Skip(); } bool wxListCtrl::Create(wxWindow *parent, @@ -217,41 +620,56 @@ bool wxListCtrl::Create(wxWindow *parent, // for now, we'll always use the generic list control for ICON and LIST views, // because they dynamically change the number of columns on resize. // Also, allow the user to set it to use the list ctrl as well. - if ( (wxSystemOptions::HasOption( wxMAC_ALWAYS_USE_GENERIC_LISTCTRL ) + // Also, use generic list control in VIRTUAL mode. + if ( (wxSystemOptions::HasOption( wxMAC_ALWAYS_USE_GENERIC_LISTCTRL ) && (wxSystemOptions::GetOptionInt( wxMAC_ALWAYS_USE_GENERIC_LISTCTRL ) == 1)) || - (style & wxLC_ICON) || (style & wxLC_SMALL_ICON) || (style & wxLC_LIST)) + (style & wxLC_ICON) || (style & wxLC_SMALL_ICON) || (style & wxLC_LIST) ) { m_macIsUserPane = true; - - if ( !wxWindow::Create(parent, id, pos, size, style, name) ) + + if ( !wxWindow::Create(parent, id, pos, size, style & wxNO_BORDER, name) ) return false; - m_genericImpl = new wxGenericListCtrl(this, id, pos, size, style, validator, name); + + // since the generic control is a child, make sure we position it at 0, 0 + m_genericImpl = new wxGenericListCtrlHook(this, id, wxPoint(0, 0), size, style, validator, name); + m_genericImpl->PushEventHandler( new wxMacListCtrlEventDelegate( this, GetId() ) ); return true; } - + else { m_macIsUserPane = false; - + if ( !wxWindow::Create(parent, id, pos, size, style, name) ) return false; m_dbImpl = new wxMacDataBrowserListCtrlControl( this, pos, size, style ); m_peer = m_dbImpl; MacPostControlCreate( pos, size ); + + InstallControlEventHandler( m_peer->GetControlRef() , GetwxMacListCtrlEventHandlerUPP(), + GetEventTypeCount(eventList), eventList, this, + (EventHandlerRef *)&m_macListCtrlEventHandler); } - + return true; } wxListCtrl::~wxListCtrl() -{ +{ + if (m_genericImpl) + { + m_genericImpl->PopEventHandler(/* deleteHandler = */ true); + } + if (m_ownsImageListNormal) delete m_imageListNormal; if (m_ownsImageListSmall) delete m_imageListSmall; if (m_ownsImageListState) delete m_imageListState; + + delete m_renameTimer; } // ---------------------------------------------------------------------------- @@ -289,7 +707,8 @@ void wxListCtrl::SetWindowStyleFlag(long flag) { m_windowStyle = flag; - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->SetWindowStyleFlag(flag); } @@ -297,11 +716,56 @@ void wxListCtrl::SetWindowStyleFlag(long flag) } } -void wxListCtrl::DoSetSize( int x, int y, int width, int height, int sizeFlags ){ +void wxListCtrl::DoSetSize( int x, int y, int width, int height, int sizeFlags ) +{ wxControl::DoSetSize(x, y, width, height, sizeFlags); - + + if (m_genericImpl) + m_genericImpl->SetSize(0, 0, width, height, sizeFlags); +} + +wxSize wxListCtrl::DoGetBestSize() const +{ + return wxWindow::DoGetBestSize(); +} + +bool wxListCtrl::SetFont(const wxFont& font) +{ + bool rv = true; + rv = wxControl::SetFont(font); + if (m_genericImpl) + rv = m_genericImpl->SetFont(font); + return rv; +} + +bool wxListCtrl::SetForegroundColour(const wxColour& colour) +{ + bool rv = true; + if (m_genericImpl) + rv = m_genericImpl->SetForegroundColour(colour); + if (m_dbImpl) + SetTextColour(colour); + return rv; +} + +bool wxListCtrl::SetBackgroundColour(const wxColour& colour) +{ + bool rv = true; + if (m_genericImpl) + rv = m_genericImpl->SetBackgroundColour(colour); + if (m_dbImpl) + m_bgColor = colour; + return rv; +} + +wxColour wxListCtrl::GetBackgroundColour() +{ if (m_genericImpl) - m_genericImpl->SetSize(x, y, width, height, sizeFlags); + return m_genericImpl->GetBackgroundColour(); + if (m_dbImpl) + return m_bgColor; + + return wxNullColour; } // ---------------------------------------------------------------------------- @@ -315,24 +779,29 @@ bool wxListCtrl::GetColumn(int col, wxListItem& item) const return m_genericImpl->GetColumn(col, item); bool success = true; - - if ( item.m_mask & wxLIST_MASK_TEXT ) - { - } - if ( item.m_mask & wxLIST_MASK_FORMAT ) + if (m_dbImpl) { - } - if ( item.m_mask & wxLIST_MASK_IMAGE ) - { - } + wxColumnList::compatibility_iterator node = m_colsInfo.Item( col ); + wxASSERT_MSG( node, _T("invalid column index in wxMacListCtrlItem") ); + wxListItem* column = node->GetData(); - if ( (item.m_mask & wxLIST_MASK_TEXT) ) - { + long mask = column->GetMask(); + if (mask & wxLIST_MASK_TEXT) + item.SetText(column->GetText()); + if (mask & wxLIST_MASK_DATA) + item.SetData(column->GetData()); + if (mask & wxLIST_MASK_IMAGE) + item.SetImage(column->GetImage()); + if (mask & wxLIST_MASK_STATE) + item.SetState(column->GetState()); + if (mask & wxLIST_MASK_FORMAT) + item.SetAlign(column->GetAlign()); + if (mask & wxLIST_MASK_WIDTH) + item.SetWidth(column->GetWidth()); } - return success; } @@ -341,16 +810,79 @@ bool wxListCtrl::SetColumn(int col, wxListItem& item) { if (m_genericImpl) return m_genericImpl->SetColumn(col, item); - - return false; + + if (m_dbImpl) + { + if ( col >= (int)m_colsInfo.GetCount() ) + { + wxListItem* listItem = new wxListItem(item); + m_colsInfo.Append( listItem ); + } + else + { + wxListItem listItem; + GetColumn( col, listItem ); + long mask = item.GetMask(); + if (mask & wxLIST_MASK_TEXT) + listItem.SetText(item.GetText()); + if (mask & wxLIST_MASK_DATA) + listItem.SetData(item.GetData()); + if (mask & wxLIST_MASK_IMAGE) + listItem.SetImage(item.GetImage()); + if (mask & wxLIST_MASK_STATE) + listItem.SetState(item.GetState()); + if (mask & wxLIST_MASK_FORMAT) + listItem.SetAlign(item.GetAlign()); + if (mask & wxLIST_MASK_WIDTH) + listItem.SetWidth(item.GetWidth()); + } + + // change the appearance in the databrowser. + DataBrowserListViewHeaderDesc columnDesc; + columnDesc.version=kDataBrowserListViewLatestHeaderDesc; + verify_noerr( m_dbImpl->GetHeaderDesc( kMinColumnId + col, &columnDesc ) ); + + /* + if (item.GetMask() & wxLIST_MASK_TEXT) + { + wxFontEncoding enc; + if ( m_font.Ok() ) + enc = m_font.GetEncoding(); + else + enc = wxLocale::GetSystemEncoding(); + wxMacCFStringHolder cfTitle; + cfTitle.Assign( item.GetText() , enc ); + if(columnDesc.titleString) + CFRelease(columnDesc.titleString); + columnDesc.titleString = cfTitle; + } + */ + + if (item.GetMask() & wxLIST_MASK_IMAGE && item.GetImage() != -1 ) + { + columnDesc.btnContentInfo.contentType = kControlContentIconRef; + wxImageList* imageList = GetImageList(wxIMAGE_LIST_SMALL); + if (imageList && imageList->GetImageCount() > 0 ) + { + wxBitmap bmp = imageList->GetBitmap( item.GetImage() ); + IconRef icon = bmp.GetBitmapData()->GetIconRef(); + columnDesc.btnContentInfo.u.iconRef = icon; + } + } + + verify_noerr( m_dbImpl->SetHeaderDesc( kMinColumnId + col, &columnDesc ) ); + + } + return true; } int wxListCtrl::GetColumnCount() const { if (m_genericImpl) return m_genericImpl->GetColumnCount(); - - if (m_dbImpl){ + + if (m_dbImpl) + { UInt32 count; m_dbImpl->GetColumnCount(&count); return count; @@ -364,8 +896,9 @@ int wxListCtrl::GetColumnWidth(int col) const { if (m_genericImpl) return m_genericImpl->GetColumnWidth(col); - - if (m_dbImpl){ + + if (m_dbImpl) + { return m_dbImpl->GetColumnWidth(col); } @@ -377,16 +910,26 @@ bool wxListCtrl::SetColumnWidth(int col, int width) { if (m_genericImpl) return m_genericImpl->SetColumnWidth(col, width); - - // TODO: This is setting the width of the first column - // to the entire window width; investigate why - // this is. - - //if (m_dbImpl){ - // m_dbImpl->SetColumnWidth(col+1, width); - // return true; - //} - + + if (m_dbImpl) + { + int mywidth = width; + if (width == wxLIST_AUTOSIZE || width == wxLIST_AUTOSIZE_USEHEADER) + mywidth = 150; + + if (col == -1) + { + for (int column = 0; column < GetColumnCount(); column++) + { + m_dbImpl->SetColumnWidth(col, mywidth); + } + } + else{ + m_dbImpl->SetColumnWidth(col, mywidth); + } + return true; + } + return false; } @@ -398,19 +941,20 @@ int wxListCtrl::GetCountPerPage() const { if (m_genericImpl) return m_genericImpl->GetCountPerPage(); - - if (m_dbImpl){ - } - - return 1; -} + + if (m_dbImpl) + { + } + + return 1; +} // Gets the edit control for editing labels. wxTextCtrl* wxListCtrl::GetEditControl() const { if (m_genericImpl) return m_genericImpl->GetEditControl(); - + return NULL; } @@ -419,9 +963,24 @@ bool wxListCtrl::GetItem(wxListItem& info) const { if (m_genericImpl) return m_genericImpl->GetItem(info); - + if (m_dbImpl) - m_dbImpl->MacGetColumnInfo(info.m_itemId, info.m_col, info); + { + if (!IsVirtual()) + m_dbImpl->MacGetColumnInfo(info.m_itemId, info.m_col, info); + else + { + info.SetText( OnGetItemText(info.m_itemId, info.m_col) ); + info.SetImage( OnGetItemColumnImage(info.m_itemId, info.m_col) ); + wxListItemAttr* attrs = OnGetItemAttr( info.m_itemId ); + if (attrs) + { + info.SetFont( attrs->GetFont() ); + info.SetBackgroundColour( attrs->GetBackgroundColour() ); + info.SetTextColour( attrs->GetTextColour() ); + } + } + } bool success = true; return success; } @@ -462,7 +1021,7 @@ int wxListCtrl::GetItemState(long item, long stateMask) const { if (m_genericImpl) return m_genericImpl->GetItemState(item, stateMask); - + wxListItem info; info.m_mask = wxLIST_MASK_STATE; @@ -516,7 +1075,7 @@ wxString wxListCtrl::GetItemText(long item) const { if (m_genericImpl) return m_genericImpl->GetItemText(item); - + wxListItem info; info.m_mask = wxLIST_MASK_TEXT; @@ -532,7 +1091,7 @@ void wxListCtrl::SetItemText(long item, const wxString& str) { if (m_genericImpl) return m_genericImpl->SetItemText(item, str); - + wxListItem info; info.m_mask = wxLIST_MASK_TEXT; @@ -563,7 +1122,7 @@ bool wxListCtrl::SetItemData(long item, long data) { if (m_genericImpl) return m_genericImpl->SetItemData(item, data); - + wxListItem info; info.m_mask = wxLIST_MASK_DATA; @@ -577,10 +1136,10 @@ wxRect wxListCtrl::GetViewRect() const { wxASSERT_MSG( !HasFlag(wxLC_REPORT | wxLC_LIST), _T("wxListCtrl::GetViewRect() only works in icon mode") ); - + if (m_genericImpl) return m_genericImpl->GetViewRect(); - + wxRect rect; return rect; } @@ -591,7 +1150,35 @@ bool wxListCtrl::GetItemRect(long item, wxRect& rect, int code) const if (m_genericImpl) return m_genericImpl->GetItemRect(item, rect, code); - return true; + + if (m_dbImpl) + { + DataBrowserItemID id; + DataBrowserPropertyID col = kMinColumnId; + Rect bounds; + DataBrowserPropertyPart part = kDataBrowserPropertyEnclosingPart; + if ( code == wxLIST_RECT_LABEL ) + part = kDataBrowserPropertyTextPart; + else if ( code == wxLIST_RECT_ICON ) + part = kDataBrowserPropertyIconPart; + + if ( !(GetWindowStyleFlag() & wxLC_VIRTUAL) ) + { + wxMacDataItem* thisItem = m_dbImpl->GetItemFromLine(item); + id = (DataBrowserItemID) thisItem; + } + else + id = item; + + GetDataBrowserItemPartBounds( m_dbImpl->GetControlRef(), id, col, part, &bounds ); + + rect.x = bounds.left; + rect.y = bounds.top; + rect.width = bounds.right - bounds.left; //GetClientSize().x; // we need the width of the whole row, not just the item. + rect.height = bounds.bottom - bounds.top; + //fprintf("id = %d, bounds = %d, %d, %d, %d\n", id, rect.x, rect.y, rect.width, rect.height); + } + return true; } // Gets the item position @@ -602,6 +1189,14 @@ bool wxListCtrl::GetItemPosition(long item, wxPoint& pos) const bool success = false; + if (m_dbImpl) + { + wxRect itemRect; + GetItemRect(item, itemRect); + pos = itemRect.GetPosition(); + success = true; + } + return success; } @@ -622,7 +1217,7 @@ int wxListCtrl::GetItemCount() const if (m_dbImpl) return m_dbImpl->MacGetCount(); - + return m_count; } @@ -642,10 +1237,12 @@ wxSize wxListCtrl::GetItemSpacing() const void wxListCtrl::SetItemTextColour( long item, const wxColour &col ) { - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->SetItemTextColour(item, col); + return; } - + wxListItem info; info.m_itemId = item; info.SetTextColour( col ); @@ -656,8 +1253,9 @@ wxColour wxListCtrl::GetItemTextColour( long item ) const { if (m_genericImpl) return m_genericImpl->GetItemTextColour(item); - - if (m_dbImpl){ + + if (m_dbImpl) + { wxListItem info; if (GetItem(info)) return info.GetTextColour(); @@ -667,11 +1265,12 @@ wxColour wxListCtrl::GetItemTextColour( long item ) const void wxListCtrl::SetItemBackgroundColour( long item, const wxColour &col ) { - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->SetItemBackgroundColour(item, col); return; } - + wxListItem info; info.m_itemId = item; info.SetBackgroundColour( col ); @@ -682,8 +1281,9 @@ wxColour wxListCtrl::GetItemBackgroundColour( long item ) const { if (m_genericImpl) return m_genericImpl->GetItemBackgroundColour(item); - - if (m_dbImpl){ + + if (m_dbImpl) + { wxListItem info; if (GetItem(info)) return info.GetBackgroundColour(); @@ -693,7 +1293,8 @@ wxColour wxListCtrl::GetItemBackgroundColour( long item ) const void wxListCtrl::SetItemFont( long item, const wxFont &f ) { - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->SetItemFont(item, f); return; } @@ -708,8 +1309,9 @@ wxFont wxListCtrl::GetItemFont( long item ) const { if (m_genericImpl) return m_genericImpl->GetItemFont(item); - - if (m_dbImpl){ + + if (m_dbImpl) + { wxListItem info; if (GetItem(info)) return info.GetFont(); @@ -738,19 +1340,22 @@ wxColour wxListCtrl::GetTextColour() const // TODO: we need owner drawn list items to customize text color. if (m_dbImpl) - return *wxBLACK; + return m_textColor; + + return wxNullColour; } // Sets the text colour of the listview void wxListCtrl::SetTextColour(const wxColour& col) { - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->SetTextColour(col); return; } - - // TODO: if we add owner-drawn item support for DataBrowser, - // consider supporting this property + + if (m_dbImpl) + m_textColor = col; } // Gets the index of the topmost visible item when in @@ -759,7 +1364,7 @@ long wxListCtrl::GetTopItem() const { if (m_genericImpl) return m_genericImpl->GetTopItem(); - + return 0; } @@ -776,6 +1381,18 @@ long wxListCtrl::GetNextItem(long item, int geom, int state) const if (m_genericImpl) return m_genericImpl->GetNextItem(item, geom, state); + if (m_dbImpl && geom == wxLIST_NEXT_ALL && state == wxLIST_STATE_SELECTED ) + { + long count = m_dbImpl->MacGetCount() ; + for ( long line = item + 1 ; line < count; line++ ) + { + wxMacDataItem* id = m_dbImpl->GetItemFromLine(line); + if ( m_dbImpl->IsItemSelected(id ) ) + return line; + } + return -1; + } + return 0; } @@ -802,12 +1419,12 @@ wxImageList *wxListCtrl::GetImageList(int which) const void wxListCtrl::SetImageList(wxImageList *imageList, int which) { - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->SetImageList(imageList, which); return; } - int flags = 0; if ( which == wxIMAGE_LIST_NORMAL ) { if (m_ownsImageListNormal) delete m_imageListNormal; @@ -830,7 +1447,8 @@ void wxListCtrl::SetImageList(wxImageList *imageList, int which) void wxListCtrl::AssignImageList(wxImageList *imageList, int which) { - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->AssignImageList(imageList, which); return; } @@ -861,14 +1479,15 @@ bool wxListCtrl::DeleteItem(long item) { if (m_genericImpl) return m_genericImpl->DeleteItem(item); - - if (m_dbImpl){ + + if (m_dbImpl) + { m_dbImpl->MacDelete(item); wxListEvent event( wxEVT_COMMAND_LIST_DELETE_ITEM, GetId() ); event.SetEventObject( this ); event.m_itemIndex = item; - GetEventHandler()->ProcessEvent( event ); - + GetEventHandler()->ProcessEvent( event ); + } return true; } @@ -878,12 +1497,13 @@ bool wxListCtrl::DeleteAllItems() { if (m_genericImpl) return m_genericImpl->DeleteAllItems(); - - if (m_dbImpl){ + + if (m_dbImpl) + { m_dbImpl->MacClear(); wxListEvent event( wxEVT_COMMAND_LIST_DELETE_ALL_ITEMS, GetId() ); event.SetEventObject( this ); - GetEventHandler()->ProcessEvent( event ); + GetEventHandler()->ProcessEvent( event ); } return true; } @@ -893,15 +1513,17 @@ bool wxListCtrl::DeleteAllColumns() { if (m_genericImpl) return m_genericImpl->DeleteAllColumns(); - - if (m_dbImpl){ + + if (m_dbImpl) + { UInt32 cols; m_dbImpl->GetColumnCount(&cols); - for (UInt32 col = 0; col < cols; col++){ + for (UInt32 col = 0; col < cols; col++) + { DeleteColumn(col); - } + } } - + return true; } @@ -910,33 +1532,61 @@ bool wxListCtrl::DeleteColumn(int col) { if (m_genericImpl) return m_genericImpl->DeleteColumn(col); - - if (m_dbImpl){ + + if (m_dbImpl) + { OSStatus err = m_dbImpl->RemoveColumn(col); return err == noErr; } - + return true; } // Clears items, and columns if there are any. void wxListCtrl::ClearAll() { - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->ClearAll(); return; } - - DeleteAllItems(); - if ( m_colCount > 0 ) + + if (m_dbImpl) + { + DeleteAllItems(); DeleteAllColumns(); + } } wxTextCtrl* wxListCtrl::EditLabel(long item, wxClassInfo* textControlClass) { if (m_genericImpl) return m_genericImpl->EditLabel(item, textControlClass); - + + if (m_dbImpl) + { + wxCHECK_MSG( (item >= 0) && ((long)item < GetItemCount()), NULL, + wxT("wrong index in wxListCtrl::EditLabel()") ); + + wxASSERT_MSG( textControlClass->IsKindOf(CLASSINFO(wxTextCtrl)), + wxT("EditLabel() needs a text control") ); + + wxListEvent le( wxEVT_COMMAND_LIST_BEGIN_LABEL_EDIT, GetParent()->GetId() ); + le.SetEventObject( this ); + le.m_itemIndex = item; + le.m_col = 0; + GetItem( le.m_item ); + + if ( GetParent()->GetEventHandler()->ProcessEvent( le ) && !le.IsAllowed() ) + { + // vetoed by user code + return NULL; + } + + wxTextCtrl * const text = (wxTextCtrl *)textControlClass->CreateObject(); + m_textctrlWrapper = new wxListCtrlTextCtrlWrapper(this, text, item); + return m_textctrlWrapper->GetText(); + } return NULL; } @@ -945,10 +1595,11 @@ bool wxListCtrl::EndEditLabel(bool cancel) { // TODO: generic impl. doesn't have this method - is it needed for us? if (m_genericImpl) - return false; // m_genericImpl->EndEditLabel(cancel); - - bool b = true; - return b; + return true; // m_genericImpl->EndEditLabel(cancel); + + if (m_dbImpl) + verify_noerr( SetDataBrowserEditItem(m_dbImpl->GetControlRef(), kDataBrowserNoItem, kMinColumnId) ); + return true; } // Ensures this item is visible @@ -956,11 +1607,14 @@ bool wxListCtrl::EnsureVisible(long item) { if (m_genericImpl) return m_genericImpl->EnsureVisible(item); - - if (m_dbImpl){ + + if (m_dbImpl) + { wxMacDataItem* dataItem = m_dbImpl->GetItemFromLine(item); m_dbImpl->RevealItem(dataItem, kDataBrowserRevealWithoutSelecting); } + + return true; } // Find an item whose label matches this string, starting from the item after 'start' @@ -979,7 +1633,7 @@ long wxListCtrl::FindItem(long start, long data) { if (m_genericImpl) return m_genericImpl->FindItem(start, data); - + long idx = start + 1; long count = GetItemCount(); @@ -1010,6 +1664,47 @@ wxListCtrl::HitTest(const wxPoint& point, int& flags, long *ptrSubItem) const if (m_genericImpl) return m_genericImpl->HitTest(point, flags, ptrSubItem); + flags = wxLIST_HITTEST_NOWHERE; + if (m_dbImpl) + { + int colHeaderHeight = 22; // TODO: Find a way to get this value from the db control? + UInt16 rowHeight = 0; + m_dbImpl->GetDefaultRowHeight(&rowHeight); + + int y = point.y; + // get the actual row by taking scroll position into account + UInt32 offsetX, offsetY; + m_dbImpl->GetScrollPosition( &offsetY, &offsetX ); + y += offsetY; + + if ( !(GetWindowStyleFlag() & wxLC_NO_HEADER) ) + y -= colHeaderHeight; + + int row = y / rowHeight; + DataBrowserItemID id; + m_dbImpl->GetItemID( (DataBrowserTableViewRowIndex) row, &id ); + + // TODO: Use GetDataBrowserItemPartBounds to return if we are in icon or label + if ( !(GetWindowStyleFlag() & wxLC_VIRTUAL ) ) + { + wxMacListCtrlItem* lcItem; + lcItem = (wxMacListCtrlItem*) id; + if (lcItem) + { + flags = wxLIST_HITTEST_ONITEM; + return row; + } + } + else + { + if (row < GetItemCount() ) + { + flags = wxLIST_HITTEST_ONITEM; + return row; + } + } + + } return -1; } @@ -1022,18 +1717,19 @@ long wxListCtrl::InsertItem(wxListItem& info) if (m_genericImpl) return m_genericImpl->InsertItem(info); - - if (m_dbImpl){ + + if (m_dbImpl) + { int count = GetItemCount(); if (info.m_itemId > count) info.m_itemId = count; - + m_dbImpl->MacInsertItem(info.m_itemId, &info ); wxListEvent event( wxEVT_COMMAND_LIST_INSERT_ITEM, GetId() ); event.SetEventObject( this ); event.m_itemIndex = info.m_itemId; - GetEventHandler()->ProcessEvent( event ); + GetEventHandler()->ProcessEvent( event ); } return info.m_itemId; @@ -1056,7 +1752,7 @@ long wxListCtrl::InsertItem(long index, int imageIndex) { if (m_genericImpl) return m_genericImpl->InsertItem(index, imageIndex); - + wxListItem info; info.m_image = imageIndex; info.m_mask = wxLIST_MASK_IMAGE; @@ -1069,7 +1765,7 @@ long wxListCtrl::InsertItem(long index, const wxString& label, int imageIndex) { if (m_genericImpl) return m_genericImpl->InsertItem(index, label, imageIndex); - + wxListItem info; info.m_image = imageIndex; info.m_text = label; @@ -1083,22 +1779,25 @@ long wxListCtrl::InsertColumn(long col, wxListItem& item) { if (m_genericImpl) return m_genericImpl->InsertColumn(col, item); - - if (m_dbImpl){ - if ( !(item.GetMask() & wxLIST_MASK_WIDTH) ) - item.SetWidth(150); - - DataBrowserPropertyType type = kDataBrowserTextType; + + if (m_dbImpl) + { + int width = item.GetWidth(); + if ( !(item.GetMask() & wxLIST_MASK_WIDTH) ) + width = 150; + + DataBrowserPropertyType type = kDataBrowserCustomType; //kDataBrowserTextType; wxImageList* imageList = GetImageList(wxIMAGE_LIST_SMALL); - if (imageList && imageList->GetImageCount() > 0){ + if (imageList && imageList->GetImageCount() > 0) + { wxBitmap bmp = imageList->GetBitmap(0); - if (bmp.Ok()) - type = kDataBrowserIconAndTextType; + //if (bmp.Ok()) + // type = kDataBrowserIconAndTextType; } - - fprintf(stderr, "Flush is %d\n", item.GetAlign()); + SInt16 just = teFlushDefault; - if (item.GetMask() & wxLIST_MASK_FORMAT){ + if (item.GetMask() & wxLIST_MASK_FORMAT) + { if (item.GetAlign() == wxLIST_FORMAT_LEFT) just = teFlushLeft; else if (item.GetAlign() == wxLIST_FORMAT_CENTER) @@ -1106,19 +1805,23 @@ long wxListCtrl::InsertColumn(long col, wxListItem& item) else if (item.GetAlign() == wxLIST_FORMAT_RIGHT) just = teFlushRight; } - m_dbImpl->InsertColumn(col, type, item.GetText(), just, item.GetWidth()); - + m_dbImpl->InsertColumn(col, type, item.GetText(), just, width); + SetColumn(col, item); + + // set/remove options based on the wxListCtrl type. + DataBrowserTableViewColumnID id; + m_dbImpl->GetColumnIDFromIndex(col, &id); + DataBrowserPropertyFlags flags; + verify_noerr(m_dbImpl->GetPropertyFlags(id, &flags)); if (GetWindowStyleFlag() & wxLC_EDIT_LABELS) - { - DataBrowserTableViewColumnID id; - m_dbImpl->GetColumnIDFromIndex(col, &id); - DataBrowserPropertyFlags flags; - verify_noerr(m_dbImpl->GetPropertyFlags(id, &flags)); flags |= kDataBrowserPropertyIsEditable; - verify_noerr(m_dbImpl->SetPropertyFlags(id, flags)); + + if (GetWindowStyleFlag() & wxLC_VIRTUAL){ + flags &= ~kDataBrowserListViewSortableColumn; } + verify_noerr(m_dbImpl->SetPropertyFlags(id, flags)); } - + return col; } @@ -1149,8 +1852,9 @@ bool wxListCtrl::ScrollList(int dx, int dy) { if (m_genericImpl) return m_genericImpl->ScrollList(dx, dy); - - if (m_dbImpl){ + + if (m_dbImpl) + { m_dbImpl->SetScrollPosition(dx, dy); } return true; @@ -1161,10 +1865,49 @@ bool wxListCtrl::SortItems(wxListCtrlCompare fn, long data) { if (m_genericImpl) return m_genericImpl->SortItems(fn, data); - + + if (m_dbImpl) + { + m_compareFunc = fn; + m_compareFuncData = data; + } + return true; } +void wxListCtrl::OnRenameTimer() +{ + wxCHECK_RET( HasCurrent(), wxT("unexpected rename timer") ); + + EditLabel( m_current ); +} + +bool wxListCtrl::OnRenameAccept(long itemEdit, const wxString& value) +{ + wxListEvent le( wxEVT_COMMAND_LIST_END_LABEL_EDIT, GetId() ); + le.SetEventObject( this ); + le.m_itemIndex = itemEdit; + + GetItem( le.m_item ); + le.m_item.m_text = value; + return !GetEventHandler()->ProcessEvent( le ) || + le.IsAllowed(); +} + +void wxListCtrl::OnRenameCancelled(long itemEdit) +{ + // let owner know that the edit was cancelled + wxListEvent le( wxEVT_COMMAND_LIST_END_LABEL_EDIT, GetParent()->GetId() ); + + le.SetEditCanceled(true); + + le.SetEventObject( this ); + le.m_itemIndex = itemEdit; + + GetItem( le.m_item ); + GetEventHandler()->ProcessEvent( le ); +} + // ---------------------------------------------------------------------------- // virtual list controls // ---------------------------------------------------------------------------- @@ -1207,21 +1950,39 @@ void wxListCtrl::SetItemCount(long count) { wxASSERT_MSG( IsVirtual(), _T("this is for virtual controls only") ); - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->SetItemCount(count); return; } - + + if (m_dbImpl) + { + // we need to temporarily disable the new item creation notification + // procedure to speed things up + // FIXME: Even this doesn't seem to help much... + DataBrowserCallbacks callbacks; + DataBrowserItemNotificationUPP itemUPP; + GetDataBrowserCallbacks(m_dbImpl->GetControlRef(), &callbacks); + itemUPP = callbacks.u.v1.itemNotificationCallback; + callbacks.u.v1.itemNotificationCallback = 0; + m_dbImpl->SetCallbacks(&callbacks); + ::AddDataBrowserItems(m_dbImpl->GetControlRef(), kDataBrowserNoItem, + count, NULL, kDataBrowserItemNoProperty); + callbacks.u.v1.itemNotificationCallback = itemUPP; + m_dbImpl->SetCallbacks(&callbacks); + } m_count = count; } void wxListCtrl::RefreshItem(long item) { - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->RefreshItem(item); return; } - + wxRect rect; GetItemRect(item, rect); RefreshRect(rect); @@ -1229,7 +1990,8 @@ void wxListCtrl::RefreshItem(long item) void wxListCtrl::RefreshItems(long itemFrom, long itemTo) { - if (m_genericImpl){ + if (m_genericImpl) + { m_genericImpl->RefreshItems(itemFrom, itemTo); return; } @@ -1245,7 +2007,7 @@ void wxListCtrl::RefreshItems(long itemFrom, long itemTo) } -// wxMac internal data structures +// wxMac internal data structures wxMacListCtrlItem::~wxMacListCtrlItem() { @@ -1269,20 +2031,28 @@ void wxMacListCtrlItem::Notification(wxMacDataItemBrowserControl *owner , delete this; return; } + else if ( message == kDataBrowserItemAdded ) + { + // we don't issue events on adding, the item is not really stored in the list yet, so we + // avoid asserts by gettting out now + return ; + } wxListCtrl *list = wxDynamicCast( owner->GetPeer() , wxListCtrl ); - if ( list ){ + if ( list ) + { bool trigger = false; - + wxListEvent event( wxEVT_COMMAND_LIST_ITEM_SELECTED, list->GetId() ); - bool isSingle = list->GetWindowStyle() | wxLC_SINGLE_SEL; + bool isSingle = (list->GetWindowStyle() & wxLC_SINGLE_SEL) != 0; + event.SetEventObject( list ); event.m_itemIndex = owner->GetLineFromItem( this ) ; if ( !list->IsVirtual() ) { lb->MacGetColumnInfo(event.m_itemIndex,0,event.m_item); } - + switch (message) { case kDataBrowserItemDeselected: @@ -1296,7 +2066,7 @@ void wxMacListCtrlItem::Notification(wxMacDataItemBrowserControl *owner , break; case kDataBrowserItemDoubleClicked: - event.SetEventType( wxEVT_LEFT_DCLICK ); + event.SetEventType( wxEVT_COMMAND_LIST_ITEM_ACTIVATED ); trigger = true; break; @@ -1305,7 +2075,7 @@ void wxMacListCtrlItem::Notification(wxMacDataItemBrowserControl *owner , event.SetEventType( wxEVT_COMMAND_LIST_BEGIN_LABEL_EDIT ) ; trigger = true ; break ; - + case kDataBrowserEditStopped : // TODO probably trigger only upon the value store callback, because // here IIRC we cannot veto @@ -1331,6 +2101,10 @@ wxMacDataBrowserListCtrlControl::wxMacDataBrowserListCtrlControl( wxWindow *peer { OSStatus err = noErr; m_clientDataItemsType = wxClientData_None; + m_isVirtual = false; + + if ( style & wxLC_VIRTUAL ) + m_isVirtual = true; DataBrowserSelectionFlags options = kDataBrowserDragSelect; if ( style & wxLC_SINGLE_SEL ) @@ -1339,84 +2113,646 @@ wxMacDataBrowserListCtrlControl::wxMacDataBrowserListCtrlControl( wxWindow *peer } else { - options |= kDataBrowserAlwaysExtendSelection | kDataBrowserCmdTogglesSelection; + options |= kDataBrowserCmdTogglesSelection; } err = SetSelectionFlags( options ); verify_noerr( err ); - if ( style & wxLC_LIST ){ + DataBrowserCustomCallbacks callbacks; + InitializeDataBrowserCustomCallbacks( &callbacks, kDataBrowserLatestCustomCallbacks ); + + if ( gDataBrowserDrawItemUPP == NULL ) + gDataBrowserDrawItemUPP = NewDataBrowserDrawItemUPP(DataBrowserDrawItemProc); + +// if ( gDataBrowserEditItemUPP == NULL ) +// gDataBrowserEditItemUPP = NewDataBrowserEditItemUPP(DataBrowserEditTextProc); + + if ( gDataBrowserHitTestUPP == NULL ) + gDataBrowserHitTestUPP = NewDataBrowserHitTestUPP(DataBrowserHitTestProc); + + callbacks.u.v1.drawItemCallback = gDataBrowserDrawItemUPP; +// callbacks.u.v1.editTextCallback = gDataBrowserEditItemUPP; + callbacks.u.v1.hitTestCallback = gDataBrowserHitTestUPP; + + SetDataBrowserCustomCallbacks( GetControlRef(), &callbacks ); + + if ( style & wxLC_LIST ) + { InsertColumn(0, kDataBrowserIconAndTextType, wxEmptyString, -1, -1); verify_noerr( AutoSizeColumns() ); - verify_noerr( SetHeaderButtonHeight( 0 ) ); } - SetDataBrowserSortProperty( m_controlRef , kMinColumnId ); + if ( style & wxLC_LIST || style & wxLC_NO_HEADER ) + verify_noerr( SetHeaderButtonHeight( 0 ) ); + + if ( m_isVirtual ) + SetSortProperty( kMinColumnId - 1 ); + else + SetSortProperty( kMinColumnId ); if ( style & wxLC_SORT_ASCENDING ) { m_sortOrder = SortOrder_Text_Ascending; - SetDataBrowserSortProperty( m_controlRef , kMinColumnId ); - SetDataBrowserSortOrder( m_controlRef , kDataBrowserOrderIncreasing); + SetSortOrder( kDataBrowserOrderIncreasing ); } else if ( style & wxLC_SORT_DESCENDING ) { m_sortOrder = SortOrder_Text_Descending; - SetDataBrowserSortProperty( m_controlRef , kMinColumnId ); - SetDataBrowserSortOrder( m_controlRef , kDataBrowserOrderDecreasing); + SetSortOrder( kDataBrowserOrderDecreasing ); } else { m_sortOrder = SortOrder_None; - SetDataBrowserSortProperty( m_controlRef , kMinColumnId); - SetDataBrowserSortOrder( m_controlRef , kDataBrowserOrderIncreasing); } - verify_noerr( AutoSizeColumns() ); + if ( style & wxLC_VRULES ) + { +#if MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_4 + verify_noerr( DataBrowserChangeAttributes(m_controlRef, kDataBrowserAttributeListViewDrawColumnDividers, kDataBrowserAttributeNone) ); +#endif + } + verify_noerr( SetHiliteStyle(kDataBrowserTableViewFillHilite ) ); err = SetHasScrollBars( (style & wxHSCROLL) != 0 , true ); } +pascal Boolean wxMacDataBrowserListCtrlControl::DataBrowserEditTextProc( + ControlRef browser, + DataBrowserItemID itemID, + DataBrowserPropertyID property, + CFStringRef theString, + Rect *maxEditTextRect, + Boolean *shrinkToFit) +{ + Boolean result = false; + wxMacDataBrowserListCtrlControl* ctl = dynamic_cast( wxMacControl::GetReferenceFromNativeControl( browser ) ); + if ( ctl != 0 ) + { + result = ctl->ConfirmEditText(itemID, property, theString, maxEditTextRect, shrinkToFit); + theString = CFSTR("Hello!"); + } + return result; +} + +bool wxMacDataBrowserListCtrlControl::ConfirmEditText( + DataBrowserItemID itemID, + DataBrowserPropertyID property, + CFStringRef theString, + Rect *maxEditTextRect, + Boolean *shrinkToFit) +{ + //wxListCtrl* list = wxDynamicCast( GetPeer() , wxListCtrl ); + return false; +} + +pascal void wxMacDataBrowserListCtrlControl::DataBrowserDrawItemProc( + ControlRef browser, + DataBrowserItemID itemID, + DataBrowserPropertyID property, + DataBrowserItemState itemState, + const Rect *itemRect, + SInt16 gdDepth, + Boolean colorDevice) +{ + wxMacDataBrowserListCtrlControl* ctl = dynamic_cast( wxMacControl::GetReferenceFromNativeControl( browser ) ); + if ( ctl != 0 ) + { + ctl->DrawItem(itemID, property, itemState, itemRect, gdDepth, colorDevice); + } +} + +// routines needed for DrawItem +enum +{ + kIconWidth = 16, + kIconHeight = 16, + kTextBoxHeight = 14, + kIconTextSpacingV = 2, + kItemPadding = 4, + kContentHeight = kIconHeight + kTextBoxHeight + kIconTextSpacingV +}; + +static void calculateCGDrawingBounds(CGRect inItemRect, CGRect *outIconRect, CGRect *outTextRect, bool hasIcon = false) +{ + float textBottom; + float iconH, iconW = 0; + float padding = kItemPadding; + if (hasIcon) + { + iconH = kIconHeight; + iconW = kIconWidth; + padding = padding*2; + } + + textBottom = inItemRect.origin.y; + + *outIconRect = CGRectMake(inItemRect.origin.x + kItemPadding, + textBottom + kIconTextSpacingV, kIconWidth, + kIconHeight); + + *outTextRect = CGRectMake(inItemRect.origin.x + padding + iconW, + textBottom + kIconTextSpacingV, inItemRect.size.width - padding - iconW, + inItemRect.size.height - kIconTextSpacingV); +} + +void wxMacDataBrowserListCtrlControl::DrawItem( + DataBrowserItemID itemID, + DataBrowserPropertyID property, + DataBrowserItemState itemState, + const Rect *itemRect, + SInt16 gdDepth, + Boolean colorDevice) +{ + wxString text; + wxFont font = wxNullFont; + int imgIndex = -1; + short listColumn = property - kMinColumnId; + + wxListCtrl* list = wxDynamicCast( GetPeer() , wxListCtrl ); + wxMacListCtrlItem* lcItem; + wxColour color = *wxBLACK; + wxColour bgColor = wxNullColour; + + if (listColumn >= 0) + { + if (!m_isVirtual) + { + lcItem = (wxMacListCtrlItem*) itemID; + if (lcItem->HasColumnInfo(listColumn)){ + wxListItem* item = lcItem->GetColumnInfo(listColumn); + + // we always use the 0 column to get font and text/background colors. + if (lcItem->HasColumnInfo(0)) + { + wxListItem* firstItem = lcItem->GetColumnInfo(0); + color = firstItem->GetTextColour(); + bgColor = firstItem->GetBackgroundColour(); + font = firstItem->GetFont(); + } + + if (item->GetMask() & wxLIST_MASK_TEXT) + text = item->GetText(); + if (item->GetMask() & wxLIST_MASK_IMAGE) + imgIndex = item->GetImage(); + } + + } + else + { + text = list->OnGetItemText( (long)itemID-1, listColumn ); + imgIndex = list->OnGetItemColumnImage( (long)itemID-1, listColumn ); + wxListItemAttr* attrs = list->OnGetItemAttr( (long)itemID-1 ); + if (attrs) + { + if (attrs->HasBackgroundColour()) + bgColor = attrs->GetBackgroundColour(); + if (attrs->HasTextColour()) + color = attrs->GetTextColour(); + if (attrs->HasFont()) + font = attrs->GetFont(); + } + } + } + + wxColour listBgColor = list->GetBackgroundColour(); + if (bgColor == wxNullColour) + bgColor = listBgColor; + + wxFont listFont = list->GetFont(); + if (font == wxNullFont) + font = listFont; + + wxMacCFStringHolder cfString; + cfString.Assign( text, wxLocale::GetSystemEncoding() ); + + Rect enclosingRect; + CGRect enclosingCGRect, iconCGRect, textCGRect; + Boolean active; + ThemeDrawingState savedState = NULL; + CGContextRef context = (CGContextRef)list->MacGetDrawingContext(); + RGBColor labelColor; + + GetDataBrowserItemPartBounds(GetControlRef(), itemID, property, kDataBrowserPropertyEnclosingPart, + &enclosingRect); + + enclosingCGRect = CGRectMake(enclosingRect.left, + enclosingRect.top, + enclosingRect.right - enclosingRect.left, + enclosingRect.bottom - enclosingRect.top); + + active = IsControlActive(GetControlRef()); + + if (itemState == kDataBrowserItemIsSelected) + { + RGBColor foregroundColor; + + GetThemeDrawingState(&savedState); + + GetThemeBrushAsColor(kThemeBrushAlternatePrimaryHighlightColor, 32, true, &foregroundColor); + GetThemeTextColor(kThemeTextColorWhite, gdDepth, colorDevice, &labelColor); + + CGContextSaveGState(context); + + CGContextSetRGBFillColor(context, (float)foregroundColor.red / (float)USHRT_MAX, + (float)foregroundColor.green / (float)USHRT_MAX, + (float)foregroundColor.blue / (float)USHRT_MAX, 1.0); + CGContextFillRect(context, enclosingCGRect); + + CGContextRestoreGState(context); + } + else + { + + if (color.Ok()) + labelColor = MAC_WXCOLORREF( color.GetPixel() ); + else if (list->GetTextColour().Ok()) + labelColor = MAC_WXCOLORREF( list->GetTextColour().GetPixel() ); + else + { + labelColor.red = 0; + labelColor.green = 0; + labelColor.blue = 0; + } + + if (bgColor.Ok()) + { + RGBColor foregroundColor = MAC_WXCOLORREF( bgColor.GetPixel() ); + CGContextSaveGState(context); + + CGContextSetRGBFillColor(context, (float)foregroundColor.red / (float)USHRT_MAX, + (float)foregroundColor.green / (float)USHRT_MAX, + (float)foregroundColor.blue / (float)USHRT_MAX, 1.0); + CGContextFillRect(context, enclosingCGRect); + + CGContextRestoreGState(context); + } + } + + calculateCGDrawingBounds(enclosingCGRect, &iconCGRect, &textCGRect, (imgIndex != -1) ); + + if (imgIndex != -1) + { + wxImageList* imageList = list->GetImageList(wxIMAGE_LIST_SMALL); + if (imageList && imageList->GetImageCount() > 0){ + wxBitmap bmp = imageList->GetBitmap(imgIndex); + IconRef icon = bmp.GetBitmapData()->GetIconRef(); + + RGBColor iconLabel; + iconLabel.red = 0; + iconLabel.green = 0; + iconLabel.blue = 0; + + CGContextSaveGState(context); + CGContextTranslateCTM(context, 0,iconCGRect.origin.y + CGRectGetMaxY(iconCGRect)); + CGContextScaleCTM(context,1.0f,-1.0f); + PlotIconRefInContext(context, &iconCGRect, kAlignNone, + active ? kTransformNone : kTransformDisabled, &iconLabel, + kPlotIconRefNormalFlags, icon); + + CGContextRestoreGState(context); + } + } + + HIThemeTextHorizontalFlush hFlush = kHIThemeTextHorizontalFlushLeft; + UInt16 fontID = kThemeViewsFont; + + if (font.Ok()) + { + if (font.GetFamily() != wxFONTFAMILY_DEFAULT) + fontID = font.MacGetThemeFontID(); + +// FIXME: replace these with CG or ATSUI calls so we can remove this #ifndef. +#ifndef __LP64__ + ::TextSize( (short)(font.MacGetFontSize()) ) ; + ::TextFace( font.MacGetFontStyle() ) ; +#endif + } + + wxListItem item; + list->GetColumn(listColumn, item); + if (item.GetMask() & wxLIST_MASK_FORMAT) + { + if (item.GetAlign() == wxLIST_FORMAT_LEFT) + hFlush = kHIThemeTextHorizontalFlushLeft; + else if (item.GetAlign() == wxLIST_FORMAT_CENTER) + hFlush = kHIThemeTextHorizontalFlushCenter; + else if (item.GetAlign() == wxLIST_FORMAT_RIGHT) + { + hFlush = kHIThemeTextHorizontalFlushRight; + textCGRect.origin.x -= kItemPadding; // give a little extra paddding + } + } + + HIThemeTextInfo info; + info.version = kHIThemeTextInfoVersionZero; + info.state = active ? kThemeStateActive : kThemeStateInactive; + info.fontID = fontID; + info.horizontalFlushness = hFlush; + info.verticalFlushness = kHIThemeTextVerticalFlushCenter; + info.options = kHIThemeTextBoxOptionNone; + info.truncationPosition = kHIThemeTextTruncationEnd; + info.truncationMaxLines = 1; + + CGContextSaveGState(context); + CGContextSetRGBFillColor (context, (float)labelColor.red / (float)USHRT_MAX, + (float)labelColor.green / (float)USHRT_MAX, + (float)labelColor.blue / (float)USHRT_MAX, 1.0); + + HIThemeDrawTextBox(cfString, &textCGRect, &info, context, kHIThemeOrientationNormal); + + CGContextRestoreGState(context); + + if (savedState != NULL) + SetThemeDrawingState(savedState, true); +} + +OSStatus wxMacDataBrowserListCtrlControl::GetSetItemData(DataBrowserItemID itemID, + DataBrowserPropertyID property, + DataBrowserItemDataRef itemData, + Boolean changeValue ) +{ + wxString text; + int imgIndex = -1; + short listColumn = property - kMinColumnId; + + OSStatus err = errDataBrowserPropertyNotSupported; + wxListCtrl* list = wxDynamicCast( GetPeer() , wxListCtrl ); + wxMacListCtrlItem* lcItem; + + if (listColumn >= 0) + { + if (!m_isVirtual) + { + lcItem = (wxMacListCtrlItem*) itemID; + if (lcItem->HasColumnInfo(listColumn)){ + wxListItem* item = lcItem->GetColumnInfo(listColumn); + if (item->GetMask() & wxLIST_MASK_TEXT) + text = item->GetText(); + if (item->GetMask() & wxLIST_MASK_IMAGE) + imgIndex = item->GetImage(); + } + } + else + { + text = list->OnGetItemText( (long)itemID-1, listColumn ); + imgIndex = list->OnGetItemColumnImage( (long)itemID-1, listColumn ); + } + } + + if ( !changeValue ) + { + switch (property) + { + case kDataBrowserItemIsEditableProperty : + if ( list && list->HasFlag( wxLC_EDIT_LABELS ) ) + { + verify_noerr(SetDataBrowserItemDataBooleanValue( itemData, true )); + err = noErr ; + } + break ; + default : + if ( property >= kMinColumnId ) + { + wxMacCFStringHolder cfStr; + + if (text){ + cfStr.Assign( text, wxLocale::GetSystemEncoding() ); + err = ::SetDataBrowserItemDataText( itemData, cfStr ); + err = noErr; + } + + + + if ( imgIndex != -1 ) + { + wxImageList* imageList = list->GetImageList(wxIMAGE_LIST_SMALL); + if (imageList && imageList->GetImageCount() > 0){ + wxBitmap bmp = imageList->GetBitmap(imgIndex); + IconRef icon = bmp.GetBitmapData()->GetIconRef(); + ::SetDataBrowserItemDataIcon(itemData, icon); + } + } + + } + break ; + } + + } + else + { + switch (property) + { + default: + if ( property >= kMinColumnId ) + { + short listColumn = property - kMinColumnId; + + // TODO probably send the 'end edit' from here, as we + // can then deal with the veto + CFStringRef sr ; + verify_noerr( GetDataBrowserItemDataText( itemData , &sr ) ) ; + wxMacCFStringHolder cfStr(sr) ;; + if (m_isVirtual) + list->SetItem( (long)itemData-1 , listColumn, cfStr.AsString() ) ; + else + { + if (lcItem) + lcItem->SetColumnTextValue( listColumn, cfStr.AsString() ); + } + err = noErr ; + } + break; + } + } + return err; +} + +void wxMacDataBrowserListCtrlControl::ItemNotification(DataBrowserItemID itemID, + DataBrowserItemNotification message, + DataBrowserItemDataRef itemData ) +{ + // we want to depend on as little as possible to make sure tear-down of controls is safe + if ( message == kDataBrowserItemRemoved) + { + // make sure MacDelete does the proper teardown. + return; + } + else if ( message == kDataBrowserItemAdded ) + { + // we don't issue events on adding, the item is not really stored in the list yet, so we + // avoid asserts by getting out now + return ; + } + + wxListCtrl *list = wxDynamicCast( GetPeer() , wxListCtrl ); + if ( list ) + { + bool trigger = false; + + wxListEvent event( wxEVT_COMMAND_LIST_ITEM_SELECTED, list->GetId() ); + bool isSingle = (list->GetWindowStyle() & wxLC_SINGLE_SEL) != 0; + + event.SetEventObject( list ); + if ( !list->IsVirtual() ) + { + DataBrowserTableViewRowIndex result = 0; + verify_noerr( GetItemRow( itemID, &result ) ) ; + event.m_itemIndex = result; + + if (event.m_itemIndex >= 0) + MacGetColumnInfo(event.m_itemIndex,0,event.m_item); + } + else + { + event.m_itemIndex = (long)itemID; + } + + switch (message) + { + case kDataBrowserItemDeselected: + event.SetEventType(wxEVT_COMMAND_LIST_ITEM_DESELECTED); + if ( !isSingle ) + trigger = !IsSelectionSuppressed(); + break; + + case kDataBrowserItemSelected: + trigger = !IsSelectionSuppressed(); + + break; + + case kDataBrowserItemDoubleClicked: + event.SetEventType( wxEVT_COMMAND_LIST_ITEM_ACTIVATED ); + trigger = true; + break; + + case kDataBrowserEditStarted : + // TODO : how to veto ? + event.SetEventType( wxEVT_COMMAND_LIST_BEGIN_LABEL_EDIT ) ; + trigger = true ; + break ; + + case kDataBrowserEditStopped : + // TODO probably trigger only upon the value store callback, because + // here IIRC we cannot veto + event.SetEventType( wxEVT_COMMAND_LIST_END_LABEL_EDIT ) ; + trigger = true ; + break ; + + default: + break; + } + + if ( trigger ) + { + // direct notification is not always having the listbox GetSelection() having in synch with event + wxPostEvent( list->GetEventHandler(), event ); + } + } +} + +Boolean wxMacDataBrowserListCtrlControl::CompareItems(DataBrowserItemID itemOneID, + DataBrowserItemID itemTwoID, + DataBrowserPropertyID sortProperty) +{ + + bool retval = false; + wxString itemText; + wxString otherItemText; + int colId = sortProperty - kMinColumnId; + long itemNum = 0; + long otherItemNum = 0; + + wxListCtrl* list = wxDynamicCast( GetPeer() , wxListCtrl ); + + // means we need to + if (colId >= 0) + { + if (!m_isVirtual) + { + wxMacListCtrlItem* item = (wxMacListCtrlItem*)itemOneID; + wxMacListCtrlItem* otherItem = (wxMacListCtrlItem*)itemTwoID; + wxListCtrlCompare func = list->GetCompareFunc(); + if (func != NULL && item->HasColumnInfo(colId) && otherItem->HasColumnInfo(colId)) + return func(item->GetColumnInfo(colId)->GetData(), otherItem->GetColumnInfo(colId)->GetData(), list->GetCompareFuncData()) >= 0; + + itemNum = item->GetOrder(); + otherItemNum = otherItem->GetOrder(); + if (item->HasColumnInfo(colId)) + itemText = item->GetColumnInfo(colId)->GetText(); + if (otherItem->HasColumnInfo(colId)) + otherItemText = otherItem->GetColumnInfo(colId)->GetText(); + } + else + { + itemNum = (long)itemOneID; + otherItemNum = (long)itemTwoID; + itemText = list->OnGetItemText( itemNum-1, colId ); + otherItemText = list->OnGetItemText( otherItemNum-1, colId ); + + } + + DataBrowserSortOrder sort; + verify_noerr(GetSortOrder(&sort)); + + if ( sort == kDataBrowserOrderIncreasing ) + { + retval = itemText.CmpNoCase( otherItemText ) > 0; + } + else if ( sort == kDataBrowserOrderDecreasing ) + { + retval = itemText.CmpNoCase( otherItemText ) < 0; + } + } + else{ + // fallback for undefined cases + retval = itemOneID < itemTwoID; + } + + return retval; +} + wxMacDataBrowserListCtrlControl::~wxMacDataBrowserListCtrlControl() { } void wxMacDataBrowserListCtrlControl::MacSetColumnInfo( unsigned int row, unsigned int column, wxListItem* item ) -{ +{ wxMacDataItem* dataItem = GetItemFromLine(row); - if (item){ + if (item) + { wxMacListCtrlItem* listItem = dynamic_cast(dataItem); listItem->SetColumnInfo( column, item ); UpdateState(dataItem, item); } } -// apply changes that need to happen immediately, rather than when the +// apply changes that need to happen immediately, rather than when the // databrowser control fires a callback. void wxMacDataBrowserListCtrlControl::UpdateState(wxMacDataItem* dataItem, wxListItem* listItem) { bool isSelected = IsItemSelected( dataItem ); bool isSelectedState = (listItem->GetState() == wxLIST_STATE_SELECTED); - // toggle the selection state if wxListInfo state and actual state don't match. - if ( isSelected != isSelectedState ){ + // toggle the selection state if wxListInfo state and actual state don't match. + if ( isSelected != isSelectedState ) + { DataBrowserSetOption options = kDataBrowserItemsAdd; if (!isSelectedState) options = kDataBrowserItemsRemove; - fprintf(stderr, "state = %d, db options = %d\n", isSelectedState, options); SetSelectedItem(dataItem, options); } // TODO: Set column width if item width > than current column width } void wxMacDataBrowserListCtrlControl::MacGetColumnInfo( unsigned int row, unsigned int column, wxListItem& item ) -{ +{ wxMacDataItem* dataItem = GetItemFromLine(row); // CS should this guard against dataItem = 0 ? , as item is not a pointer if (item) is not appropriate - //if (item) + //if (item) { wxMacListCtrlItem* listItem = dynamic_cast(dataItem); wxListItem* oldItem = listItem->GetColumnInfo( column ); - + long mask = item.GetMask(); if ( !mask ) // by default, get everything for backwards compatibility @@ -1437,7 +2773,7 @@ void wxMacDataBrowserListCtrlControl::MacGetColumnInfo( unsigned int row, unsign item.SetTextColour(oldItem->GetTextColour()); item.SetBackgroundColour(oldItem->GetBackgroundColour()); - item.SetFont(oldItem->GetFont()); + item.SetFont(oldItem->GetFont()); } } @@ -1447,7 +2783,7 @@ void wxMacDataBrowserListCtrlControl::MacInsertItem( unsigned int n, wxListItem* MacSetColumnInfo(n, 0, item); } -wxMacListCtrlItem* wxMacDataBrowserListCtrlControl::CreateItem() +wxMacDataItem* wxMacDataBrowserListCtrlControl::CreateItem() { return new wxMacListCtrlItem(); } @@ -1478,7 +2814,7 @@ const wxString& wxMacListCtrlItem::GetColumnTextValue( unsigned int column ) void wxMacListCtrlItem::SetColumnTextValue( unsigned int column, const wxString& text ) { GetColumnInfo(column)->SetText(text); - + // for compatibility with superclass APIs if ( column == 0 ) SetLabel(text); @@ -1500,21 +2836,13 @@ bool wxMacListCtrlItem::HasColumnInfo( unsigned int column ) void wxMacListCtrlItem::SetColumnInfo( unsigned int column, wxListItem* item ) { - if ( column >= m_rowItems.GetCount() ){ + if ( column >= m_rowItems.GetCount() ) + { wxListItem* listItem = new wxListItem(*item); - //listItem->SetAlign(item->GetAlign()); - //listItem->SetBackgroundColour(item->GetBackgroundColour()); - //listItem->SetColumn(item->GetColumn()); - //listItem->SetData(item->GetData()); - //listItem->SetFont(item->GetFont()); - //listItem->SetId(GetOrder()); - //listItem->SetImage(item->GetImage()); - //listItem->SetMask(item->GetMask()); - - //listItem->SetText(item->GetText()); m_rowItems.Append( listItem ); } - else{ + else + { wxListItem* listItem = GetColumnInfo( column ); long mask = item->GetMask(); if (mask & wxLIST_MASK_TEXT) @@ -1529,84 +2857,11 @@ void wxMacListCtrlItem::SetColumnInfo( unsigned int column, wxListItem* item ) listItem->SetAlign(item->GetAlign()); if (mask & wxLIST_MASK_WIDTH) listItem->SetWidth(item->GetWidth()); - } -} -OSStatus wxMacListCtrlItem::GetSetData( wxMacDataItemBrowserControl *owner , - DataBrowserPropertyID property, - DataBrowserItemDataRef itemData, - bool changeValue ) -{ - - OSStatus err = errDataBrowserPropertyNotSupported; - wxListCtrl* list = wxDynamicCast( owner->GetPeer() , wxListCtrl ); - if ( !changeValue ) - { - switch (property) - { - case kDataBrowserItemIsEditableProperty : - if ( list && list->HasFlag( wxLC_EDIT_LABELS ) ) - { - verify_noerr(SetDataBrowserItemDataBooleanValue( itemData, true )); - err = noErr ; - } - break ; - default : - if ( property >= kMinColumnId ){ - short listColumn = property - kMinColumnId; - - if (HasColumnInfo(listColumn)){ - wxListItem* item = GetColumnInfo(listColumn); - wxMacCFStringHolder cfStr; - - if (item->GetText()){ - cfStr.Assign( item->GetText(), wxLocale::GetSystemEncoding() ); - err = ::SetDataBrowserItemDataText( itemData, cfStr ); - err = noErr; - } - - int imgIndex = item->GetImage(); - if ( (item->GetMask() & wxLIST_MASK_IMAGE) ){ - wxImageList* imageList = list->GetImageList(wxIMAGE_LIST_SMALL); - if (imageList && imageList->GetImageCount() > 0){ - wxBitmap bmp = imageList->GetBitmap(imgIndex); - IconRef icon = bmp.GetBitmapData()->GetIconRef(); - ::SetDataBrowserItemDataIcon(itemData, icon); - } - } - - } - } - break ; - } + listItem->SetTextColour(item->GetTextColour()); + listItem->SetBackgroundColour(item->GetBackgroundColour()); + listItem->SetFont(item->GetFont()); } - else - { - switch (property) - { - default: - if ( property >= kMinColumnId ){ - short listColumn = property - kMinColumnId; - - if (HasColumnInfo(listColumn)){ - // TODO probably send the 'end edit' from here, as we - // can then deal with the veto - CFStringRef sr ; - verify_noerr( GetDataBrowserItemDataText( itemData , &sr ) ) ; - wxMacCFStringHolder cfStr(sr) ;; - list->SetItem( owner->GetLineFromItem(this) , listColumn, cfStr.AsString() ) ; - err = noErr ; - } - } - break; - } - } - - // don't duplicate the numeric order column handling - if (err == errDataBrowserPropertyNotSupported) - err = wxMacDataItem::GetSetData(owner, property, itemData, changeValue); - - return err; } #endif // wxUSE_LISTCTRL