X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/621860064662aa80ba4fdc21ca55b6d33c1bea11..43ff861df48f0837dac98005dd397c5f127ca136:/include/wx/generic/dvrenderer.h

diff --git a/include/wx/generic/dvrenderer.h b/include/wx/generic/dvrenderer.h
index 3abf273181..3914e83ccb 100644
--- a/include/wx/generic/dvrenderer.h
+++ b/include/wx/generic/dvrenderer.h
@@ -41,29 +41,24 @@ public:
 
     // implementation
 
-    // this is a replacement for dynamic_cast<wxDataViewCustomRenderer> in the
-    // code checking whether the renderer is interested in mouse events, it's
-    // overridden in wxDataViewCustomRenderer to return the object itself but
-    // intentionally returns NULL for all the other renderer classes as the
-    // user should _not_ be able to override Activate/LeftClick() when deriving
-    // from them for consistency with the other ports and while we can't
-    // prevent this from working at compile-time because all renderers are
-    // custom renderers in the generic implementation, we at least make sure
-    // that it doesn't work at run-time because Activate/LeftClick() would
-    // never be called
-    virtual wxDataViewCustomRenderer *WXGetAsCustom() { return NULL; }
-
-    // The generic implementation of some standard renderers reacts to item
-    // activation, so provide this internal function which is called by
-    // wxDataViewCtrl for them. It is called with the old value of the cell and
-    // is passed the model and cell coordinates to be able to change the model
-    // value for this cell.
-    virtual void WXOnActivate(wxDataViewModel * WXUNUSED(model),
-                              const wxVariant& WXUNUSED(valueOld),
-                              const wxDataViewItem& WXUNUSED(item),
+    // These callbacks are used by generic implementation of wxDVC itself.
+    // They're different from the corresponding Activate/LeftClick() methods
+    // which should only be overridable for the custom renderers while the
+    // generic implementation uses these ones for all of them, including the
+    // standard ones.
+
+    virtual bool WXOnActivate(wxRect WXUNUSED(cell),
+                              wxDataViewModel *WXUNUSED(model),
+                              const wxDataViewItem & WXUNUSED(item),
                               unsigned int WXUNUSED(col))
-    {
-    }
+        { return false; }
+
+    virtual bool WXOnLeftClick(wxPoint WXUNUSED(cursor),
+                               wxRect WXUNUSED(cell),
+                               wxDataViewModel *WXUNUSED(model),
+                               const wxDataViewItem & WXUNUSED(item),
+                               unsigned int WXUNUSED(col) )
+        { return false; }
 
 private:
     int                          m_align;