From 4b3feaa75de76963492c4202be04be9bf20b321f Mon Sep 17 00:00:00 2001 From: Robert Roebling Date: Tue, 21 Mar 2006 12:30:06 +0000 Subject: [PATCH] Added windowing and scrolling logic to generic implementation of wxDataViewCtrl (it runs now). git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@38238 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- include/wx/dataview.h | 2 +- include/wx/generic/dataview.h | 24 ++++-- src/generic/datavgen.cpp | 133 +++++++++++++++++++++++++++++++--- 3 files changed, 138 insertions(+), 21 deletions(-) diff --git a/include/wx/dataview.h b/include/wx/dataview.h index a7b4ea8e77..211edc7577 100644 --- a/include/wx/dataview.h +++ b/include/wx/dataview.h @@ -24,7 +24,7 @@ #if defined(__WXGTK20__) // for testing - // #define wxUSE_GENERICDATAVIEWCTRL 1 + // #define wxUSE_GENERICDATAVIEWCTRL 1 #elif defined(__WXMAC__) #define wxUSE_GENERICDATAVIEWCTRL 1 #else diff --git a/include/wx/generic/dataview.h b/include/wx/generic/dataview.h index c7802da9e7..9673737c95 100644 --- a/include/wx/generic/dataview.h +++ b/include/wx/generic/dataview.h @@ -21,6 +21,8 @@ // --------------------------------------------------------- class WXDLLIMPEXP_CORE wxDataViewCtrl; +class WXDLLIMPEXP_CORE wxDataViewMainWindow; +class WXDLLIMPEXP_CORE wxDataViewHeaderWindow; // --------------------------------------------------------- // wxDataViewCell @@ -169,7 +171,11 @@ public: virtual void SetTitle( const wxString &title ); + void SetWidth( int width ) { m_width = width; } + int GetWidth() { return m_width; } + private: + int m_width; protected: DECLARE_DYNAMIC_CLASS_NO_COPY(wxDataViewColumn) @@ -179,7 +185,7 @@ protected: // wxDataViewCtrl // --------------------------------------------------------- -class WXDLLIMPEXP_CORE wxDataViewCtrl: public wxDataViewCtrlBase, +class WXDLLIMPEXP_CORE wxDataViewCtrl: public wxDataViewCtrlBase, public wxScrollHelperNative { public: @@ -191,8 +197,8 @@ public: wxDataViewCtrl( wxWindow *parent, wxWindowID id, const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxDefaultSize, long style = 0, - const wxValidator& validator = wxDefaultValidator ) : - wxScrollHelperNative(this) + const wxValidator& validator = wxDefaultValidator ) + : wxScrollHelperNative(this) { Create(parent, id, pos, size, style, validator ); } @@ -210,13 +216,14 @@ public: virtual bool AppendColumn( wxDataViewColumn *col ); private: + friend class wxDataViewMainWindow; wxDataViewListModelNotifier *m_notifier; - wxWindow *m_clientArea; - wxWindow *m_headerArea; + wxDataViewMainWindow *m_clientArea; + wxDataViewHeaderWindow *m_headerArea; private: - virtual void ScrollWindow( int dx, int dy, const wxRect *rect ); - + void OnSize( wxSizeEvent &event ); + // we need to return a special WM_GETDLGCODE value to process just the // arrows but let the other navigation characters through #ifdef __WXMSW__ @@ -224,10 +231,11 @@ private: #endif // __WXMSW__ WX_FORWARD_TO_SCROLL_HELPER() - + private: DECLARE_DYNAMIC_CLASS(wxDataViewCtrl) DECLARE_NO_COPY_CLASS(wxDataViewCtrl) + DECLARE_EVENT_TABLE() }; diff --git a/src/generic/datavgen.cpp b/src/generic/datavgen.cpp index 41af9707f1..63e64dbdd4 100644 --- a/src/generic/datavgen.cpp +++ b/src/generic/datavgen.cpp @@ -381,6 +381,7 @@ wxDataViewColumn::wxDataViewColumn( const wxString &title, wxDataViewCell *cell, size_t model_column, int flags ) : wxDataViewColumnBase( title, cell, model_column, flags ) { + m_width = 80; } wxDataViewColumn::~wxDataViewColumn() @@ -436,7 +437,7 @@ wxDataViewHeaderWindow::wxDataViewHeaderWindow( wxDataViewCtrl *parent, wxWindow wxWindow( parent, id, pos, size, 0, name ) { SetOwner( parent ); - + m_resizeCursor = new wxCursor( wxCURSOR_SIZEWE ); wxVisualAttributes attr = wxPanel::GetClassDefaultAttributes(); @@ -453,8 +454,10 @@ wxDataViewHeaderWindow::~wxDataViewHeaderWindow() void wxDataViewHeaderWindow::OnPaint( wxPaintEvent &event ) { - wxPaintDC dc; - PrepareDC( dc ); + int w, h; + GetClientSize( &w, &h ); + + wxPaintDC dc( this ); int xpix; m_owner->GetScrollPixelsPerUnit( &xpix, NULL ); @@ -467,7 +470,37 @@ void wxDataViewHeaderWindow::OnPaint( wxPaintEvent &event ) dc.SetFont( GetFont() ); - dc.DrawText( wxT("This is the header.."), 5, 5 ); + size_t cols = GetOwner()->GetNumberOfColumns(); + size_t i; + int xpos = 0; + for (i = 0; i < cols; i++) + { + wxDataViewColumn *col = GetOwner()->GetColumn( i ); + int width = col->GetWidth(); + + // the width of the rect to draw: make it smaller to fit entirely + // inside the column rect +#ifdef __WXMAC__ + int cw = width; + int ch = h; +#else + int cw = width - 2; + int ch = h - 2; +#endif + + wxRendererNative::Get().DrawHeaderButton + ( + this, + dc, + wxRect(xpos, 0, cw, ch), + m_parent->IsEnabled() ? 0 + : (int)wxCONTROL_DISABLED + ); + + dc.DrawText( col->GetTitle(), xpos+3, 3 ); + + xpos += width; + } } void wxDataViewHeaderWindow::OnMouse( wxMouseEvent &event ) @@ -500,8 +533,15 @@ public: void OnMouse( wxMouseEvent &event ); void OnSetFocus( wxFocusEvent &event ); + void UpdateDisplay(); + void RecalculateDisplay(); + void OnInternalIdle(); + + void ScrollWindow( int dx, int dy, const wxRect *rect ); private: wxDataViewCtrl *m_owner; + int m_lineHeight; + bool m_dirty; private: DECLARE_DYNAMIC_CLASS(wxDataViewMainWindow) @@ -521,20 +561,70 @@ wxDataViewMainWindow::wxDataViewMainWindow( wxDataViewCtrl *parent, wxWindowID i wxWindow( parent, id, pos, size, 0, name ) { SetOwner( parent ); + + // We need to calculate this smartly.. + m_lineHeight = 20; + + UpdateDisplay(); } wxDataViewMainWindow::~wxDataViewMainWindow() { } +void wxDataViewMainWindow::UpdateDisplay() +{ + m_dirty = true; +} + +void wxDataViewMainWindow::OnInternalIdle() +{ + wxWindow::OnInternalIdle(); + + if (m_dirty) + { + RecalculateDisplay(); + m_dirty = false; + } +} + +void wxDataViewMainWindow::RecalculateDisplay() +{ + wxDataViewListModel *model = GetOwner()->GetModel(); + if (!model) + { + Refresh(); + return; + } + + int width = 0; + size_t cols = GetOwner()->GetNumberOfColumns(); + size_t i; + for (i = 0; i < cols; i++) + { + wxDataViewColumn *col = GetOwner()->GetColumn( i ); + width += col->GetWidth(); + } + + int height = model->GetNumberOfRows() * m_lineHeight; + + SetVirtualSize( width, height ); + GetOwner()->SetScrollRate( 10, m_lineHeight ); + + Refresh(); +} + +void wxDataViewMainWindow::ScrollWindow( int dx, int dy, const wxRect *rect ) +{ + wxWindow::ScrollWindow( dx, dy, rect ); + GetOwner()->m_headerArea->ScrollWindow( dx, 0 ); +} + void wxDataViewMainWindow::OnPaint( wxPaintEvent &event ) { wxPaintDC dc( this ); - PrepareDC( dc ); - - int dev_x, dev_y; - m_owner->CalcScrolledPosition( 0, 0, &dev_x, &dev_y ); + GetOwner()->PrepareDC( dc ); dc.SetFont( GetFont() ); @@ -557,6 +647,10 @@ void wxDataViewMainWindow::OnSetFocus( wxFocusEvent &event ) IMPLEMENT_DYNAMIC_CLASS(wxDataViewCtrl, wxDataViewCtrlBase) +BEGIN_EVENT_TABLE(wxDataViewCtrl, wxDataViewCtrlBase) + EVT_SIZE(wxDataViewCtrl::OnSize) +END_EVENT_TABLE() + wxDataViewCtrl::~wxDataViewCtrl() { if (m_notifier) @@ -572,6 +666,9 @@ bool wxDataViewCtrl::Create(wxWindow *parent, wxWindowID id, const wxPoint& pos, const wxSize& size, long style, const wxValidator& validator ) { + if (!wxControl::Create( parent, id, pos, size, style | wxScrolledWindowStyle|wxSUNKEN_BORDER, validator)) + return false; + Init(); #ifdef __WXMAC__ @@ -610,10 +707,18 @@ WXLRESULT wxDataViewCtrl::MSWWindowProc(WXUINT nMsg, } #endif -void wxDataViewCtrl::ScrollWindow( int dx, int dy, const wxRect *rect ) +void wxDataViewCtrl::OnSize( wxSizeEvent &event ) { - wxDataViewCtrlBase::ScrollWindow( dx, dy, rect ); - m_headerArea->ScrollWindow( dx, 0, rect ); + // We need to override OnSize so that our scrolled + // window a) does call Layout() to use sizers for + // positioning the controls but b) does not query + // the sizer for their size and use that for setting + // the scrollable area as set that ourselves by + // calling SetScrollbar() further down. + + Layout(); + + AdjustScrollbars(); } bool wxDataViewCtrl::AssociateModel( wxDataViewListModel *model ) @@ -625,6 +730,8 @@ bool wxDataViewCtrl::AssociateModel( wxDataViewListModel *model ) model->AddNotifier( m_notifier ); + m_clientArea->UpdateDisplay(); + return true; } @@ -632,7 +739,9 @@ bool wxDataViewCtrl::AppendColumn( wxDataViewColumn *col ) { if (!wxDataViewCtrlBase::AppendColumn(col)) return false; - + + m_clientArea->UpdateDisplay(); + return true; }