#if wxUSE_SCROLLBAR
+#include "wx/scrolbar.h"
+
#ifndef WX_PRECOMP
#include "wx/utils.h"
+ #include "wx/settings.h"
#endif
-#include "wx/scrolbar.h"
#include "wx/msw/private.h"
-#include "wx/settings.h"
-
-#if wxUSE_EXTENDED_RTTI
-WX_DEFINE_FLAGS( wxScrollBarStyle )
-
-wxBEGIN_FLAGS( wxScrollBarStyle )
- // new style border flags, we put them first to
- // use them for streaming out
- wxFLAGS_MEMBER(wxBORDER_SIMPLE)
- wxFLAGS_MEMBER(wxBORDER_SUNKEN)
- wxFLAGS_MEMBER(wxBORDER_DOUBLE)
- wxFLAGS_MEMBER(wxBORDER_RAISED)
- wxFLAGS_MEMBER(wxBORDER_STATIC)
- wxFLAGS_MEMBER(wxBORDER_NONE)
-
- // old style border flags
- wxFLAGS_MEMBER(wxSIMPLE_BORDER)
- wxFLAGS_MEMBER(wxSUNKEN_BORDER)
- wxFLAGS_MEMBER(wxDOUBLE_BORDER)
- wxFLAGS_MEMBER(wxRAISED_BORDER)
- wxFLAGS_MEMBER(wxSTATIC_BORDER)
- wxFLAGS_MEMBER(wxBORDER)
-
- // standard window styles
- wxFLAGS_MEMBER(wxTAB_TRAVERSAL)
- wxFLAGS_MEMBER(wxCLIP_CHILDREN)
- wxFLAGS_MEMBER(wxTRANSPARENT_WINDOW)
- wxFLAGS_MEMBER(wxWANTS_CHARS)
- wxFLAGS_MEMBER(wxFULL_REPAINT_ON_RESIZE)
- wxFLAGS_MEMBER(wxALWAYS_SHOW_SB )
- wxFLAGS_MEMBER(wxVSCROLL)
- wxFLAGS_MEMBER(wxHSCROLL)
-
- wxFLAGS_MEMBER(wxSB_HORIZONTAL)
- wxFLAGS_MEMBER(wxSB_VERTICAL)
-
-wxEND_FLAGS( wxScrollBarStyle )
-
-IMPLEMENT_DYNAMIC_CLASS_XTI(wxScrollBar, wxControl,"wx/scrolbar.h")
-
-wxBEGIN_PROPERTIES_TABLE(wxScrollBar)
- wxEVENT_RANGE_PROPERTY( Scroll , wxEVT_SCROLL_TOP , wxEVT_SCROLL_CHANGED , wxScrollEvent )
-
- wxPROPERTY( ThumbPosition , int , SetThumbPosition, GetThumbPosition, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
- wxPROPERTY( Range , int , SetRange, GetRange, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
- wxPROPERTY( ThumbSize , int , SetThumbSize, GetThumbSize, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
- wxPROPERTY( PageSize , int , SetPageSize, GetPageSize, 0 , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
- wxPROPERTY_FLAGS( WindowStyle , wxScrollBarStyle , long , SetWindowStyleFlag , GetWindowStyleFlag , EMPTY_MACROVALUE , 0 /*flags*/ , wxT("Helpstring") , wxT("group")) // style
-wxEND_PROPERTIES_TABLE()
-
-wxBEGIN_HANDLERS_TABLE(wxScrollBar)
-wxEND_HANDLERS_TABLE()
-
-wxCONSTRUCTOR_5( wxScrollBar , wxWindow* , Parent , wxWindowID , Id , wxPoint , Position , wxSize , Size , long , WindowStyle )
-#else
-IMPLEMENT_DYNAMIC_CLASS(wxScrollBar, wxControl)
-#endif
// Scrollbar
bool wxScrollBar::Create(wxWindow *parent, wxWindowID id,
}
bool wxScrollBar::MSWOnScroll(int WXUNUSED(orientation), WXWORD wParam,
- WXWORD pos, WXHWND WXUNUSED(control))
+ WXWORD WXUNUSED(pos), WXHWND WXUNUSED(control))
{
- // current and max positions
- int position,
- maxPos, trackPos = pos;
-
- wxUnusedVar(trackPos);
+ // don't use pos parameter because it is limited to 16 bits, get the full
+ // 32 bit position from the control itself instead
+ WinStruct<SCROLLINFO> scrollInfo;
+ scrollInfo.fMask = SIF_RANGE | SIF_POS | SIF_TRACKPOS;
- // when we're dragging the scrollbar we can't use pos parameter because it
- // is limited to 16 bits
- // JACS: now always using GetScrollInfo, since there's no reason
- // not to
-// if ( wParam == SB_THUMBPOSITION || wParam == SB_THUMBTRACK )
- {
- SCROLLINFO scrollInfo;
- wxZeroMemory(scrollInfo);
- scrollInfo.cbSize = sizeof(SCROLLINFO);
-
- // also get the range if we call GetScrollInfo() anyhow -- this is less
- // expensive than call it once here and then call GetScrollRange()
- // below
- scrollInfo.fMask = SIF_RANGE | SIF_POS | SIF_TRACKPOS;
-
- if ( !::GetScrollInfo(GetHwnd(), SB_CTL, &scrollInfo) )
- {
- wxLogLastError(_T("GetScrollInfo"));
- }
-
- trackPos = scrollInfo.nTrackPos;
- position = scrollInfo.nPos;
- maxPos = scrollInfo.nMax;
- }
-#if 0
- else
+ if ( !::GetScrollInfo(GetHwnd(), SB_CTL, &scrollInfo) )
{
- position = ::GetScrollPos((HWND) control, SB_CTL);
- int minPos;
- ::GetScrollRange((HWND) control, SB_CTL, &minPos, &maxPos);
+ wxLogLastError(wxT("GetScrollInfo"));
+ return false;
}
-#endif
+
+ int maxPos = scrollInfo.nMax;
// A page size greater than one has the effect of reducing the effective
// range, therefore the range has already been boosted artificially - so
if ( m_pageSize > 1 )
maxPos -= (m_pageSize - 1);
+ int position = scrollInfo.nPos;
wxEventType scrollEvent = wxEVT_NULL;
-
- int nScrollInc;
switch ( wParam )
{
- case SB_BOTTOM:
- nScrollInc = maxPos - position;
+ case SB_TOP:
+ position = 0;
scrollEvent = wxEVT_SCROLL_TOP;
break;
- case SB_TOP:
- nScrollInc = -position;
+ case SB_BOTTOM:
+ position = maxPos;
scrollEvent = wxEVT_SCROLL_BOTTOM;
break;
case SB_LINEUP:
- nScrollInc = -1;
+ position--;
scrollEvent = wxEVT_SCROLL_LINEUP;
break;
case SB_LINEDOWN:
- nScrollInc = 1;
+ position++;
scrollEvent = wxEVT_SCROLL_LINEDOWN;
break;
case SB_PAGEUP:
- nScrollInc = -GetPageSize();
+ position -= GetPageSize();
scrollEvent = wxEVT_SCROLL_PAGEUP;
break;
case SB_PAGEDOWN:
- nScrollInc = GetPageSize();
+ position += GetPageSize();
scrollEvent = wxEVT_SCROLL_PAGEDOWN;
break;
case SB_THUMBPOSITION:
- nScrollInc = trackPos - position;
- scrollEvent = wxEVT_SCROLL_THUMBRELEASE;
- break;
-
case SB_THUMBTRACK:
- nScrollInc = trackPos - position;
- scrollEvent = wxEVT_SCROLL_THUMBTRACK;
+ position = scrollInfo.nTrackPos;
+ scrollEvent = wParam == SB_THUMBPOSITION ? wxEVT_SCROLL_THUMBRELEASE
+ : wxEVT_SCROLL_THUMBTRACK;
break;
case SB_ENDSCROLL:
- nScrollInc = 0;
scrollEvent = wxEVT_SCROLL_CHANGED;
break;
-
- default:
- nScrollInc = 0;
}
- if ( nScrollInc )
+ if ( position != scrollInfo.nPos )
{
- position += nScrollInc;
-
if ( position < 0 )
position = 0;
if ( position > maxPos )
event.SetPosition(position);
event.SetEventObject( this );
- return GetEventHandler()->ProcessEvent(event);
+ return HandleWindowEvent(event);
}
void wxScrollBar::SetThumbPosition(int viewStart)
if ( !::GetScrollInfo(GetHwnd(), SB_CTL, &scrollInfo) )
{
- wxLogLastError(_T("GetScrollInfo"));
+ wxLogLastError(wxT("GetScrollInfo"));
}
return scrollInfo.nPos;
-// return ::GetScrollPos((HWND)m_hWnd, SB_CTL);
}
void wxScrollBar::SetScrollbar(int position, int thumbSize, int range, int pageSize,