]> git.saurik.com Git - wxWidgets.git/commitdiff
Generalized wxScrolledWindow into wxScrolled<T> that can derive from
authorVáclav Slavík <vslavik@fastmail.fm>
Tue, 1 Apr 2008 15:07:03 +0000 (15:07 +0000)
committerVáclav Slavík <vslavik@fastmail.fm>
Tue, 1 Apr 2008 15:07:03 +0000 (15:07 +0000)
anything (typically wxPanel or wxWindow). Fixed generic wxListCtrl to
use wxScrolled<wxWindow> - this fixes focus problems with label-editing
text control. Fixes to wxScrolledWindow doxygen docs.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@52961 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

docs/changes.txt
include/wx/dlimpexp.h
include/wx/scrolwin.h
interface/scrolwin.h
src/generic/listctrl.cpp
src/generic/scrlwing.cpp

index 4c8ff6c2b244d5f33fcc001c3d0441bffff5d494..18522e2f5db5572963a505a5a32f39a5bd89d670 100644 (file)
@@ -308,6 +308,8 @@ All (GUI):
 - Added wxApp::SetNativeTheme() (Stefan H.).
 - Made wxSpinCtrl::Reparent() in MSW and generic versions (Angelo Mottola).
 - Freeze() and Thaw() now recursively freeze/thaw the children too.
+- Generalized wxScrolledWindow into wxScrolled<T> template that can derive
+  from any window class, not just wxPanel.
 
 wxGTK:
 
index e87c749ab71431107a0f3495cca7d4b153d40f60..8b88602f04d2f049e864dad4542ab3056d4fd483 100644 (file)
        template <typename T> class Foo { ... };
        WXDLLIMPEXP_TEMPLATE_INSTANCE_BASE( Foo<int> )
 
-   (notice that currently we only need this for the wxBase library)
+   (notice that currently we only need this for wxBase and wxCore libraries)
  */
 #if defined(__VISUALC__) && (__VISUALC__ <= 1200)
     #ifdef WXMAKINGDLL_BASE
         #define WXDLLIMPEXP_TEMPLATE_INSTANCE_BASE(decl) \
             template class WXDLLIMPEXP_BASE decl;
+        #define WXDLLIMPEXP_TEMPLATE_INSTANCE_CORE(decl) \
+            template class WXDLLIMPEXP_CORE decl;
     #else
         /*
            We need to disable this warning when using this macro, as
 
         #define WXDLLIMPEXP_TEMPLATE_INSTANCE_BASE(decl) \
             extern template class WXDLLIMPEXP_BASE decl;
+        #define WXDLLIMPEXP_TEMPLATE_INSTANCE_CORE(decl) \
+            extern template class WXDLLIMPEXP_CORE decl;
     #endif
 #else /* not VC <= 6 */
     #define WXDLLIMPEXP_TEMPLATE_INSTANCE_BASE(decl)
+    #define WXDLLIMPEXP_TEMPLATE_INSTANCE_CORE(decl)
 #endif /* VC6/others */
 
 #endif /* _WX_DLIMPEXP_H_ */
index 0215586e952b3bba7439ac74820fc6c2c5225ac9..62ccdb04b13fb30b245a987d1d8f98fb9753ff42 100644 (file)
@@ -204,6 +204,10 @@ protected:
     // delete the event handler we installed
     void DeleteEvtHandler();
 
+    // calls wxScrollHelperEvtHandler::ResetDrawnFlag(), see explanation
+    // in wxScrollHelperEvtHandler::ProcessEvent()
+    void ResetDrawnFlag();
+
 
     double                m_scaleX;
     double                m_scaleY;
@@ -257,53 +261,121 @@ public:                                                                       \
 #endif
 
 // ----------------------------------------------------------------------------
-// wxScrolledWindow: a wxWindow which knows how to scroll
+// wxScrolled<T>: a wxWindow which knows how to scroll
 // ----------------------------------------------------------------------------
 
-class WXDLLIMPEXP_CORE wxScrolledWindow : public wxPanel,
-                                     public wxScrollHelperNative
+// helper class for wxScrolled<T> below
+struct WXDLLIMPEXP_CORE wxScrolledT_Helper
+{
+    static wxSize FilterBestSize(const wxWindow *win,
+                                 const wxScrollHelperNative *helper,
+                                 const wxSize& origBest);
+#ifdef __WXMSW__
+    static WXLRESULT FilterMSWWindowProc(WXLRESULT origResult);
+#endif
+};
+
+// Scrollable window base on window type T. This used to be wxScrolledWindow,
+// but wxScrolledWindow includes wxControlContainer functionality and that's
+// not always desirable.
+template<class T>
+class WXDLLIMPEXP_CORE wxScrolled : public T,
+                                    public wxScrollHelperNative,
+                                    private wxScrolledT_Helper
 {
 public:
-    wxScrolledWindow() : wxScrollHelperNative(this) { }
-    wxScrolledWindow(wxWindow *parent,
-                     wxWindowID winid = wxID_ANY,
-                     const wxPoint& pos = wxDefaultPosition,
-                     const wxSize& size = wxDefaultSize,
-                     long style = wxScrolledWindowStyle,
-                     const wxString& name = wxPanelNameStr)
+    wxScrolled() : wxScrollHelperNative(this) { }
+    wxScrolled(wxWindow *parent,
+               wxWindowID winid = wxID_ANY,
+               const wxPoint& pos = wxDefaultPosition,
+               const wxSize& size = wxDefaultSize,
+               long style = wxScrolledWindowStyle,
+               const wxString& name = wxPanelNameStr)
         : wxScrollHelperNative(this)
     {
         Create(parent, winid, pos, size, style, name);
     }
 
-    virtual ~wxScrolledWindow();
-
     bool Create(wxWindow *parent,
                 wxWindowID winid,
                 const wxPoint& pos = wxDefaultPosition,
                 const wxSize& size = wxDefaultSize,
                 long style = wxScrolledWindowStyle,
-                const wxString& name = wxPanelNameStr);
+                const wxString& name = wxPanelNameStr)
+    {
+        m_targetWindow = this;
+
+#ifdef __WXMAC__
+        MacSetClipChildren(true);
+#endif
+
+        this->Connect(wxEVT_PAINT, wxPaintEventHandler(wxScrolled::OnPaint));
+
+        // by default, we're scrollable in both directions (but if one of the
+        // styles is specified explicitly, we shouldn't add the other one
+        // automatically)
+        if ( !(style & (wxHSCROLL | wxVSCROLL)) )
+            style |= wxHSCROLL | wxVSCROLL;
+
+        return T::Create(parent, winid, pos, size, style, name);
+    }
 
     // we need to return a special WM_GETDLGCODE value to process just the
     // arrows but let the other navigation characters through
 #ifdef __WXMSW__
-    virtual WXLRESULT MSWWindowProc(WXUINT nMsg, WXWPARAM wParam, WXLPARAM lParam);
+    virtual WXLRESULT MSWWindowProc(WXUINT nMsg, WXWPARAM wParam, WXLPARAM lParam)
+    {
+        return FilterMSWWindowProc(T::MSWWindowProc(nMsg, wParam, lParam));
+    }
 #endif // __WXMSW__
 
     WX_FORWARD_TO_SCROLL_HELPER()
 
 protected:
-    virtual wxSize DoGetBestSize() const;
+    virtual wxSize DoGetBestSize() const
+    {
+        return FilterBestSize(this, this, T::DoGetBestSize());
+    }
 
+private:
     // this is needed for wxEVT_PAINT processing hack described in
     // wxScrollHelperEvtHandler::ProcessEvent()
-    void OnPaint(wxPaintEvent& event);
+    void OnPaint(wxPaintEvent& event)
+    {
+        // the user code didn't really draw the window if we got here, so set
+        // this flag to try to call OnDraw() later
+        ResetDrawnFlag();
+        event.Skip();
+    }
+
+    DECLARE_NO_COPY_CLASS(wxScrolled)
+};
+
+// VC++ <= 6 requires this; it's unlikely any other specializations would
+// be needed by user code _and_ they were using VC6, so we list only wxWindow
+// (typical use) and wxPanel (wxScrolledWindow use) specializations here
+WXDLLIMPEXP_TEMPLATE_INSTANCE_CORE( wxScrolled<wxPanel> )
+WXDLLIMPEXP_TEMPLATE_INSTANCE_CORE( wxScrolled<wxWindow> )
+
+// for compatibility with existing code, we provide wxScrolledWindow
+// "typedef" for wxScrolled<wxPanel>. It's not a real typedef because we
+// want wxScrolledWindow to show in wxRTTI information (the class is widely
+// used and likelihood of its wxRTTI information being used too is high):
+class WXDLLIMPEXP_CORE wxScrolledWindow : public wxScrolled<wxPanel>
+{
+public:
+    wxScrolledWindow() : wxScrolled<wxPanel>() {}
+    wxScrolledWindow(wxWindow *parent,
+                     wxWindowID winid = wxID_ANY,
+                     const wxPoint& pos = wxDefaultPosition,
+                     const wxSize& size = wxDefaultSize,
+                     long style = wxScrolledWindowStyle,
+                     const wxString& name = wxPanelNameStr)
+        : wxScrolled<wxPanel>(parent, winid, pos, size, style, name) {}
 
-private:
     DECLARE_DYNAMIC_CLASS_NO_COPY(wxScrolledWindow)
-    DECLARE_EVENT_TABLE()
 };
 
-#endif // _WX_SCROLWIN_H_BASE_
+typedef wxScrolled<wxWindow> wxScrolledCanvas;
 
+#endif // _WX_SCROLWIN_H_BASE_
index eb01f8d60abb4133bf92732896d42d559eb0ee97..7bd80daedc5f2d87232ffbd68e8ea08e2e27a146 100644 (file)
 /////////////////////////////////////////////////////////////////////////////
 
 /**
-    @class wxScrolledWindow
     @wxheader{scrolwin.h}
 
-    The wxScrolledWindow class manages scrolling for its client area, transforming
+    The wxScrolled<T> class manages scrolling for its client area, transforming
     the coordinates according to the scrollbar positions, and setting the
     scroll positions, thumb sizes and ranges according to the area in view.
 
+    There are two commonly used (but not the only possible!) specializations of
+    this class:
+
+    - ::wxScrolledWindow, aka wxScrolled<wxPanel>, is equivalent to
+      ::wxScrolledWindow from earlier versions. Derived from wxPanel, it shares
+      wxPanel's behaviour with regard to TAB traversal and focus handling. Use
+      this if the scrolled window will have children controls.
+
+    - ::wxScrolledCanvas, aka wxScrolled<wxWindow>, derives from wxWindow and
+      so doesn't handle children specially. This is suitable e.g. for
+      implementating scrollable controls such as tree or list controls.
+
     Starting from version 2.4 of wxWidgets, there are several ways to use a
-    wxScrolledWindow. In particular, there are now three ways to set the
+    wxScrolled<T>. In particular, there are now three ways to set the
     size of the scrolling area:
 
-    One way is to set the scrollbars directly using a call to
-    wxScrolledWindow::SetScrollbars.
-    This is the way it used to be in any previous version of wxWidgets
-    and it will be kept for backwards compatibility.
+    One way is to set the scrollbars directly using a call to SetScrollbars().
+    This is the way it used to be in any previous version of wxWidgets and it
+    will be kept for backwards compatibility.
 
     An additional method of manual control, which requires a little less
     computation of your own, is to set the total size of the scrolling area by
-    calling either wxWindow::SetVirtualSize,
-    or wxWindow::FitInside, and setting the
-    scrolling increments for it by calling
-    wxScrolledWindow::SetScrollRate.
+    calling either wxWindow::SetVirtualSize(), or wxWindow::FitInside(), and
+    setting the scrolling increments for it by calling SetScrollRate().
     Scrolling in some orientation is enabled by setting a non-zero increment
     for it.
 
     The most automatic and newest way is to simply let sizers determine the
-    scrolling area. This is now the default when you set an interior sizer
-    into a wxScrolledWindow with wxWindow::SetSizer.
-    The scrolling area will be set to the size requested by the sizer and
-    the scrollbars will be assigned for each orientation according to the need
-    for them and the scrolling increment set by
-    wxScrolledWindow::SetScrollRate.
-    As above, scrolling is only enabled in orientations with a non-zero
-    increment.  You can influence the minimum size of the scrolled area
-    controlled by a sizer by calling
-    wxWindow::SetVirtualSizeHints.
-    (calling wxScrolledWindow::SetScrollbars
-     has analogous effects in wxWidgets 2.4 -- in later versions it may not continue
-     to override the sizer)
-
-    Note:  if Maximum size hints are still supported by SetVirtualSizeHints, use
-    them at your own dire risk.  They may or may not have been removed for 2.4,
-    but it really only makes sense to set minimum size hints here.  We should
-    probably replace SetVirtualSizeHints with SetMinVirtualSize or similar
-    and remove it entirely in future.
-
-    As with all windows, an application can draw onto a wxScrolledWindow using
+    scrolling area. This is now the default when you set an interior sizer into
+    a wxScrolled<T> with wxWindow::SetSizer().  The scrolling area will be
+    set to the size requested by the sizer and the scrollbars will be assigned
+    for each orientation according to the need for them and the scrolling
+    increment set by SetScrollRate().  As above, scrolling is only enabled in
+    orientations with a non-zero increment.  You can influence the minimum size
+    of the scrolled area controlled by a sizer by calling
+    wxWindow::SetVirtualSizeHints(). (Calling SetScrollbars() has analogous
+    effects in wxWidgets 2.4 -- in later versions it may not continue to
+    override the sizer.)
+
+    Note that if maximum size hints are still supported by
+    wxWindow::SetVirtualSizeHints(), use them at your own dire risk. They may
+    or may not have been removed for 2.4, but it really only makes sense to set
+    minimum size hints here.  We should probably replace
+    wxWindow::SetVirtualSizeHints() with wxWindow::SetMinVirtualSize() or
+    similar and remove it entirely in future.
+
+    As with all windows, an application can draw onto a wxScrolled<T> using
     a @ref overview_dcoverview "device context".
 
-    You have the option of handling the OnPaint handler
-    or overriding the wxScrolledWindow::OnDraw function, which is
-    passed a pre-scrolled device context (prepared by
-    wxScrolledWindow::DoPrepareDC).
-
-    If you don't wish to calculate your own scrolling, you must call DoPrepareDC
-    when not drawing from
-    within OnDraw, to set the device origin for the device context according to the
-    current
-    scroll position.
-
-    A wxScrolledWindow will normally scroll itself and therefore its child windows
-    as well. It
-    might however be desired to scroll a different window than itself: e.g. when
-    designing a
-    spreadsheet, you will normally only have to scroll the (usually white) cell
-    area, whereas the
-    (usually grey) label area will scroll very differently. For this special
-    purpose, you can
-    call wxScrolledWindow::SetTargetWindow which means that pressing
-    the scrollbars will scroll a different window.
-
-    Note that the underlying system knows nothing about scrolling coordinates, so
-    that all system
-    functions (mouse events, expose events, refresh calls etc) as well as the
-    position of subwindows
-    are relative to the "physical" origin of the scrolled window. If the user
-    insert a child window at
+    You have the option of handling the OnPaint handler or overriding the
+    wxScrolled<T>::OnDraw() function, which is passed a pre-scrolled device
+    context (prepared by wxScrolled<T>::DoPrepareDC()).
+
+    If you don't wish to calculate your own scrolling, you must call
+    DoPrepareDC() when not drawing from within OnDraw(), to set the device
+    origin for the device context according to the current scroll position.
+
+    A wxScrolled<T> will normally scroll itself and therefore its child windows
+    as well. It might however be desired to scroll a different window than
+    itself: e.g. when designing a spreadsheet, you will normally only have to
+    scroll the (usually white) cell area, whereas the (usually grey) label area
+    will scroll very differently. For this special purpose, you can call
+    SetTargetWindow() which means that pressing the scrollbars will scroll a
+    different window.
+
+    Note that the underlying system knows nothing about scrolling coordinates,
+    so that all system functions (mouse events, expose events, refresh calls
+    etc) as well as the position of subwindows are relative to the "physical"
+    origin of the scrolled window. If the user insert a child window at
     position (10,10) and scrolls the window down 100 pixels (moving the child
-    window out of the visible
-    area), the child window will report a position of (10,-90).
+    window out of the visible area), the child window will report a position
+    of (10,-90).
 
     @beginStyleTable
     @style{wxRETAINED}:
            Uses a backing pixmap to speed refreshes. Motif only.
     @endStyleTable
 
+    @remarks
+    Use wxScrolled<T> for applications where the user scrolls by a fixed
+    amount, and where a 'page' can be interpreted to be the current visible
+    portion of the window. For more sophisticated applications, use the
+    wxScrolled<T> implementation as a guide to build your own scroll
+    behaviour or use wxVScrolledWindow or its variants.
+
+    @since wxScrolled<T> template exists since version 2.9.0. In older
+           versions, only ::wxScrolledWindow (equivalent of wxScrolled<wxPanel>)
+           was available.
+
     @library{wxcore}
     @category{miscwnd}
 
-    @see wxScrollBar, wxClientDC, wxPaintDC, wxVScrolledWindow
+    @see wxScrollBar, wxClientDC, wxPaintDC,
+         wxVScrolledWindow, wxHScrolledWindow, wxHVScrolledWindow,
 */
-class wxScrolledWindow : public wxPanel
+template<class T>
+class wxScrolled : public T
 {
 public:
-    //@{
+    /// Default constructor.
+    wxScrolled();
+
     /**
         Constructor.
 
         @param parent
             Parent window.
         @param id
-            Window identifier. The value wxID_ANY indicates a default value.
+            Window identifier. The value @c wxID_ANY indicates a default value.
         @param pos
-            Window position. If a position of (-1, -1) is specified then a default
-        position
-            is chosen.
+            Window position. If a position of @c wxDefaultPosition is specified
+            then a default position is chosen.
         @param size
-            Window size. If a size of (-1, -1) is specified then the window is sized
-            appropriately.
+            Window size. If a size of @c wxDefaultSize is specified then the
+            window is sized appropriately.
         @param style
-            Window style. See wxScrolledWindow.
+            Window style. See wxScrolled<T>.
         @param name
             Window name.
 
-        @remarks The window is initially created without visible scrollbars. Call
-                 SetScrollbars() to specify how big the
-                 virtual window size should be.
+        @remarks The window is initially created without visible scrollbars.
+                 Call SetScrollbars() to specify how big the virtual window
+                 size should be.
     */
-    wxScrolledWindow();
-    wxScrolledWindow(wxWindow* parent, wxWindowID id = -1,
-                     const wxPoint& pos = wxDefaultPosition,
-                     const wxSize& size = wxDefaultSize,
-                     long style = wxHSCROLL | wxVSCROLL,
-                     const wxString& name = "scrolledWindow");
-    //@}
+    wxScrolled(wxWindow* parent, wxWindowID id = -1,
+               const wxPoint& pos = wxDefaultPosition,
+               const wxSize& size = wxDefaultSize,
+               long style = wxHSCROLL | wxVSCROLL,
+               const wxString& name = "scrolledWindow");
 
-    /**
-        Destructor.
-    */
-    ~wxScrolledWindow();
 
     /**
-        Translates the logical coordinates to the device ones. For example, if a window
-        is
-        scrolled 10 pixels to the bottom, the device coordinates of the origin are (0,
-        0)
-        (as always), but the logical coordinates are (0, 10) and so the call to
-        CalcScrolledPosition(0, 10, xx, yy) will return 0 in yy.
+        Translates the logical coordinates to the device ones. For example, if
+        a window is scrolled 10 pixels to the bottom, the device coordinates of
+        the origin are (0, 0) (as always), but the logical coordinates are (0,
+        10) and so the call to CalcScrolledPosition(0, 10, xx, yy) will return
+        0 in yy.
 
         @see CalcUnscrolledPosition()
     */
     void CalcScrolledPosition(int x, int y, int* xx, int* yy) const;
 
     /**
-        Translates the device coordinates to the logical ones. For example, if a window
-        is
-        scrolled 10 pixels to the bottom, the device coordinates of the origin are (0,
-        0)
-        (as always), but the logical coordinates are (0, 10) and so the call to
-        CalcUnscrolledPosition(0, 0, xx, yy) will return 10 in yy.
+        Translates the device coordinates to the logical ones. For example, if
+        a window is scrolled 10 pixels to the bottom, the device coordinates of
+        the origin are (0, 0) (as always), but the logical coordinates are (0,
+        10) and so the call to CalcUnscrolledPosition(0, 0, xx, yy) will return
+        10 in yy.
 
         @see CalcScrolledPosition()
     */
@@ -164,7 +168,7 @@ public:
 
     /**
         Creates the window for two-step construction. Derived classes
-        should call or replace this function. See wxScrolledWindow()
+        should call or replace this function. See wxScrolled<T> constructor
         for details.
     */
     bool Create(wxWindow* parent, wxWindowID id = -1,
@@ -174,16 +178,35 @@ public:
                 const wxString& name = "scrolledWindow");
 
     /**
-        Call this function to prepare the device context for drawing a scrolled image.
-        It
-        sets the device origin according to the current scroll position.
-        DoPrepareDC is called automatically within the default OnPaint() event
-        handler, so your OnDraw() override
-        will be passed a 'pre-scrolled' device context. However, if you wish to draw
-        from
-        outside of OnDraw (via OnPaint), or you wish to implement OnPaint yourself, you
-        must
-        call this function yourself. For example:
+        Call this function to prepare the device context for drawing a scrolled
+        image.
+
+        It sets the device origin according to the current scroll position.
+        DoPrepareDC() is called automatically within the default OnPaint()
+        event handler, so your OnDraw() override will be passed a
+        'pre-scrolled' device context. However, if you wish to draw from
+        outside of OnDraw() (via OnPaint()), or you wish to implement OnPaint()
+        yourself, you must call this function yourself.
+
+        For example:
+        @code
+        void MyWindow::OnEvent(wxMouseEvent& event)
+        {
+          wxClientDC dc(this);
+          DoPrepareDC(dc);
+
+          dc.SetPen(*wxBLACK_PEN);
+          float x, y;
+          event.Position(&x, &y);
+          if (xpos > -1 && ypos > -1 && event.Dragging())
+          {
+            dc.DrawLine(xpos, ypos, x, y);
+          }
+          xpos = x;
+          ypos = y;
+        }
+        @endcode
+
     */
     void DoPrepareDC(wxDC& dc);
 
@@ -207,9 +230,9 @@ public:
     void EnableScrolling(bool xScrolling, bool yScrolling);
 
     /**
-        Get the number of pixels per scroll unit (line), in each direction, as set
-        by SetScrollbars(). A value of zero indicates no
-        scrolling in that direction.
+        Get the number of pixels per scroll unit (line), in each direction, as
+        set by SetScrollbars(). A value of zero indicates no scrolling in that
+        direction.
 
         @param xUnit
             Receives the number of pixels per horizontal unit.
@@ -230,7 +253,7 @@ public:
 
         @remarks If either of the scrollbars is not at the home position, x
                  and/or y will be greater than zero.  Combined with
-                 wxWindow::GetClientSize, the application can use this
+                 wxWindow::GetClientSize(), the application can use this
                  function to efficiently redraw only the visible portion
                  of the window.  The positions are in logical scroll
                  units, not pixels, so to convert to pixels you will
@@ -251,7 +274,7 @@ public:
         @param y
             Receives the height of the scrollable window, in pixels.
 
-        @remarks Use wxDC::DeviceToLogicalX and wxDC::DeviceToLogicalY to
+        @remarks Use wxDC::DeviceToLogicalX() and wxDC::DeviceToLogicalY() to
                  translate these units to logical units.
 
         @see SetScrollbars(), GetScrollPixelsPerUnit()
@@ -264,20 +287,21 @@ public:
     bool IsRetained() const;
 
     /**
-        Called by the default paint event handler to allow the application to define
-        painting behaviour without having to worry about calling
+        Called by the default paint event handler to allow the application to
+        define painting behaviour without having to worry about calling
         DoPrepareDC().
-        Instead of overriding this function you may also just process the paint event
-        in the derived class as usual, but then you will have to call DoPrepareDC()
-        yourself.
+
+        Instead of overriding this function you may also just process the paint
+        event in the derived class as usual, but then you will have to call
+        DoPrepareDC() yourself.
     */
     virtual void OnDraw(wxDC& dc);
 
     /**
         This function is for backwards compatibility only and simply calls
-        DoPrepareDC() now. Notice that it is
-        not called by the default paint event handle (DoPrepareDC() is), so
-        overriding this method in your derived class is useless.
+        DoPrepareDC() now. Notice that it is not called by the default paint
+        event handle (DoPrepareDC() is), so overriding this method in your
+        derived class is useless.
     */
     void PrepareDC(wxDC& dc);
 
@@ -300,14 +324,37 @@ public:
     void Scroll(int x, int y);
 
     /**
-        Set the horizontal and vertical scrolling increment only. See the pixelsPerUnit
-        parameter in SetScrollbars.
+        Set the horizontal and vertical scrolling increment only. See the
+        pixelsPerUnit parameter in SetScrollbars().
     */
     void SetScrollRate(int xstep, int ystep);
 
     /**
         Sets up vertical and/or horizontal scrollbars.
 
+        The first pair of parameters give the number of pixels per 'scroll
+        step', i.e. amount moved when the up or down scroll arrows are pressed.
+        The second pair gives the length of scrollbar in scroll steps, which
+        sets the size of the virtual window.
+
+        @a xPos and @a yPos optionally specify a position to scroll to
+        immediately.
+
+        For example, the following gives a window horizontal and vertical
+        scrollbars with 20 pixels per scroll step, and a size of 50 steps (1000
+        pixels) in each direction:
+        @code
+        window->SetScrollbars(20, 20, 50, 50);
+        @endcode
+
+        wxScrolled<T> manages the page size itself, using the current client
+        window size as the page size.
+
+        Note that for more sophisticated scrolling applications, for example
+        where scroll steps may be variable according to the position in the
+        document, it will be necessary to derive a new class from wxWindow,
+        overriding OnSize() and adjusting the scrollbars appropriately.
+
         @param pixelsPerUnitX
             Pixels per scroll unit in the horizontal direction.
         @param pixelsPerUnitY
@@ -317,21 +364,15 @@ public:
         @param noUnitsY
             Number of units in the vertical direction.
         @param xPos
-            Position to initialize the scrollbars in the horizontal direction, in
-        scroll units.
+            Position to initialize the scrollbars in the horizontal direction,
+            in scroll units.
         @param yPos
-            Position to initialize the scrollbars in the vertical direction, in scroll
-        units.
+            Position to initialize the scrollbars in the vertical direction, in
+            scroll units.
         @param noRefresh
             Will not refresh window if @true.
 
-        @remarks The first pair of parameters give the number of pixels per
-                 'scroll step', i.e. amount moved when the up or down
-                 scroll arrows are pressed. The second pair gives the
-                 length of scrollbar in scroll steps, which sets the
-                 size of the virtual window.
-
-        @see wxWindow::SetVirtualSize
+        @see wxWindow::SetVirtualSize()
     */
     void SetScrollbars(int pixelsPerUnitX, int pixelsPerUnitY,
                        int noUnitsX,
@@ -341,9 +382,42 @@ public:
                        bool noRefresh = false);
 
     /**
-        Call this function to tell wxScrolledWindow to perform the actual scrolling on
-        a different window (and not on itself).
+        Call this function to tell wxScrolled to perform the actual
+        scrolling on a different window (and not on itself).
     */
     void SetTargetWindow(wxWindow* window);
 };
 
+
+/**
+    Scrolled window derived from wxPanel.
+
+    See wxScrolled<T> for detailed description.
+
+    @note Note that because this class derives from wxPanel, it shares its
+          behavior with regard to TAB traversal and focus handling (in
+          particular, it forwards focus to its children). If you don't want
+          this behaviour, use ::wxScrolledCanvas instead.
+
+    @note wxScrolledWindow is an alias for wxScrolled<wxPanel> since version
+          2.9.0. In older versions, it was a standalone class.
+
+    @library{wxcore}
+    @category{miscwnd}
+
+    @see wxScrolled, wxScrolledCanvas
+*/
+typedef wxScrolled<wxPanel> wxScrolledWindow;
+
+/**
+    Alias for wxScrolled<wxWindow>. Scrolled window that doesn't have children
+    and so doesn't need or want special handling of TAB traversal.
+
+    @since 2.9.0
+
+    @library{wxcore}
+    @category{miscwnd}
+
+    @see wxScrolled, ::wxScrolledWindow
+*/
+typedef wxScrolled<wxWindow> wxScrolledCanvas;
index a3419d15c7d37337ddf6bc6dcf2afbf71760a4ea..eb553e13284246cc89ac8fee169697e7ff3453a5 100644 (file)
@@ -432,7 +432,6 @@ private:
     // it wasn't vetoed, i.e. if we should proceed
     bool SendListEvent(wxEventType type, const wxPoint& pos);
 
-    DECLARE_DYNAMIC_CLASS(wxListHeaderWindow)
     DECLARE_EVENT_TABLE()
 };
 
@@ -492,7 +491,7 @@ WX_DECLARE_LIST(wxListHeaderData, wxListHeaderDataList);
 #include "wx/listimpl.cpp"
 WX_DEFINE_LIST(wxListHeaderDataList)
 
-class wxListMainWindow : public wxScrolledWindow
+class wxListMainWindow : public wxScrolledCanvas
 {
 public:
     wxListMainWindow();
@@ -701,7 +700,7 @@ public:
     // override base class virtual to reset m_lineHeight when the font changes
     virtual bool SetFont(const wxFont& font)
     {
-        if ( !wxScrolledWindow::SetFont(font) )
+        if ( !wxScrolledCanvas::SetFont(font) )
             return false;
 
         m_lineHeight = 0;
@@ -849,7 +848,6 @@ private:
     wxListTextCtrlWrapper *m_textctrlWrapper;
 
 
-    DECLARE_DYNAMIC_CLASS(wxListMainWindow)
     DECLARE_EVENT_TABLE()
 
     friend class wxGenericListCtrl;
@@ -1686,8 +1684,6 @@ void wxListLineData::ReverseHighlight( void )
 //  wxListHeaderWindow
 //-----------------------------------------------------------------------------
 
-IMPLEMENT_DYNAMIC_CLASS(wxListHeaderWindow,wxWindow)
-
 BEGIN_EVENT_TABLE(wxListHeaderWindow,wxWindow)
     EVT_PAINT         (wxListHeaderWindow::OnPaint)
     EVT_MOUSE_EVENTS  (wxListHeaderWindow::OnMouse)
@@ -2160,7 +2156,7 @@ void wxListTextCtrlWrapper::Finish( bool setfocus )
     wxPendingDelete.Append( this );
 
     if (setfocus)
-        m_owner->SetFocusIgnoringChildren();
+        m_owner->SetFocus();
 }
 
 bool wxListTextCtrlWrapper::AcceptChanges()
@@ -2237,9 +2233,7 @@ void wxListTextCtrlWrapper::OnKillFocus( wxFocusEvent &event )
 //  wxListMainWindow
 //-----------------------------------------------------------------------------
 
-IMPLEMENT_DYNAMIC_CLASS(wxListMainWindow,wxScrolledWindow)
-
-BEGIN_EVENT_TABLE(wxListMainWindow,wxScrolledWindow)
+BEGIN_EVENT_TABLE(wxListMainWindow,wxScrolledCanvas)
   EVT_PAINT          (wxListMainWindow::OnPaint)
   EVT_MOUSE_EVENTS   (wxListMainWindow::OnMouse)
   EVT_CHAR           (wxListMainWindow::OnChar)
@@ -2295,7 +2289,7 @@ wxListMainWindow::wxListMainWindow( wxWindow *parent,
                                     const wxSize& size,
                                     long style,
                                     const wxString &name )
-                : wxScrolledWindow( parent, id, pos, size,
+                : wxScrolledCanvas( parent, id, pos, size,
                                     style | wxHSCROLL | wxVSCROLL, name )
 {
     Init();
@@ -2994,7 +2988,7 @@ void wxListMainWindow::OnMouse( wxMouseEvent &event )
 #endif // __WXMAC__
 
     if ( event.LeftDown() )
-        SetFocusIgnoringChildren();
+        SetFocus();
 
     event.SetEventObject( GetParent() );
     if ( GetParent()->GetEventHandler()->ProcessEvent( event) )
@@ -4860,7 +4854,7 @@ void wxListMainWindow::OnScroll(wxScrollWinEvent& event)
 {
     // FIXME
 #if ( defined(__WXGTK__) || defined(__WXMAC__) ) && !defined(__WXUNIVERSAL__)
-    wxScrolledWindow::OnScroll(event);
+    wxScrolledCanvas::OnScroll(event);
 #else
     HandleOnScroll( event );
 #endif
index 00dac29274966b9d9402830f990904d086da48a2..7d14e6a55682024b26b429dd725166c00291a3ab 100644 (file)
@@ -438,6 +438,12 @@ void wxScrollHelper::DeleteEvtHandler()
     }
 }
 
+void wxScrollHelper::ResetDrawnFlag()
+{
+    wxCHECK_RET( m_handler, "invalid use of ResetDrawnFlag - no handler?" );
+    m_handler->ResetDrawnFlag();
+}
+
 void wxScrollHelper::DoSetTargetWindow(wxWindow *target)
 {
     m_targetWindow = target;
@@ -1398,52 +1404,12 @@ void wxScrollHelper::HandleOnChildFocus(wxChildFocusEvent& event)
 }
 
 // ----------------------------------------------------------------------------
-// wxScrolledWindow implementation
+// wxScrolled<T> and wxScrolledWindow implementation
 // ----------------------------------------------------------------------------
 
-IMPLEMENT_DYNAMIC_CLASS(wxScrolledWindow, wxPanel)
-
-BEGIN_EVENT_TABLE(wxScrolledWindow, wxPanel)
-    EVT_PAINT(wxScrolledWindow::OnPaint)
-END_EVENT_TABLE()
-
-bool wxScrolledWindow::Create(wxWindow *parent,
-                              wxWindowID id,
-                              const wxPoint& pos,
-                              const wxSize& size,
-                              long style,
-                              const wxString& name)
-{
-    m_targetWindow = this;
-#ifdef __WXMAC__
-    MacSetClipChildren( true ) ;
-#endif
-
-    // by default, we're scrollable in both directions (but if one of the
-    // styles is specified explicitly, we shouldn't add the other one
-    // automatically)
-    if ( !(style & (wxHSCROLL | wxVSCROLL)) )
-        style |= wxHSCROLL | wxVSCROLL;
-
-    bool ok = wxPanel::Create(parent, id, pos, size, style, name);
-
-    return ok;
-}
-
-wxScrolledWindow::~wxScrolledWindow()
-{
-}
-
-void wxScrolledWindow::OnPaint(wxPaintEvent& event)
-{
-    // the user code didn't really draw the window if we got here, so set this
-    // flag to try to call OnDraw() later
-    m_handler->ResetDrawnFlag();
-
-    event.Skip();
-}
-
-wxSize wxScrolledWindow::DoGetBestSize() const
+wxSize wxScrolledT_Helper::FilterBestSize(const wxWindow *win,
+                                          const wxScrollHelperNative *helper,
+                                          const wxSize& origBest)
 {
     // NB: We don't do this in WX_FORWARD_TO_SCROLL_HELPER, because not
     //     all scrollable windows should behave like this, only those that
@@ -1451,15 +1417,15 @@ wxSize wxScrolledWindow::DoGetBestSize() const
     //     (i.e., wxScrolledWindow) and other some scrollable windows may
     //     have different DoGetBestSize() implementation (e.g. wxTreeCtrl).
 
-    wxSize best = wxPanel::DoGetBestSize();
+    wxSize best = origBest;
 
-    if ( GetAutoLayout() )
+    if ( win->GetAutoLayout() )
     {
         // Only use the content to set the window size in the direction
         // where there's no scrolling; otherwise we're going to get a huge
         // window in the direction in which scrolling is enabled
         int ppuX, ppuY;
-        GetScrollPixelsPerUnit(&ppuX, &ppuY);
+        helper->GetScrollPixelsPerUnit(&ppuX, &ppuY);
 
         // NB: This code used to use *current* size if min size wasn't
         //     specified, presumably to get some reasonable (i.e., larger than
@@ -1471,7 +1437,7 @@ wxSize wxScrolledWindow::DoGetBestSize() const
         //
         //     See also http://svn.wxwidgets.org/viewvc/wx?view=rev&revision=45864
 
-        wxSize minSize = GetMinSize();
+        wxSize minSize = win->GetMinSize();
 
         if ( ppuX > 0 )
             best.x = minSize.x + wxSystemSettings::GetMetric(wxSYS_VSCROLL_X);
@@ -1484,12 +1450,8 @@ wxSize wxScrolledWindow::DoGetBestSize() const
 }
 
 #ifdef __WXMSW__
-WXLRESULT wxScrolledWindow::MSWWindowProc(WXUINT nMsg,
-                                       WXWPARAM wParam,
-                                       WXLPARAM lParam)
+WXLRESULT wxScrolledT_Helper::FilterMSWWindowProc(WXLRESULT rc)
 {
-    WXLRESULT rc = wxPanel::MSWWindowProc(nMsg, wParam, lParam);
-
 #ifndef __WXWINCE__
     // we need to process arrows ourselves for scrolling
     if ( nMsg == WM_GETDLGCODE )
@@ -1497,8 +1459,10 @@ WXLRESULT wxScrolledWindow::MSWWindowProc(WXUINT nMsg,
         rc |= DLGC_WANTARROWS;
     }
 #endif
-
     return rc;
 }
-
 #endif // __WXMSW__
+
+// NB: skipping wxScrolled<T> in wxRTTI information because being a templte,
+//     it doesn't and can't implement wxRTTI support
+IMPLEMENT_DYNAMIC_CLASS(wxScrolledWindow, wxPanel)