Fix for 4077106, make sure that mouse wheeling in overflow sections uses 40 rather...
authorhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Apr 2005 23:41:18 +0000 (23:41 +0000)
committerhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Apr 2005 23:41:18 +0000 (23:41 +0000)
step.

        Reviewed by darin

        * khtml/rendering/render_layer.cpp:
        (RenderLayer::positionScrollbars):
        * kwq/KWQScrollBar.mm:
        (QScrollBar::scroll):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@8994 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/khtml/rendering/render_layer.cpp
WebCore/kwq/KWQScrollBar.mm

index 6c94c6a9713c1ea2abcdfa3fea34a648728e91ce..2729973a118a2962f0b3a58750e54e5507017acb 100644 (file)
@@ -1,3 +1,15 @@
+2005-04-05  David Hyatt  <hyatt@apple.com>
+
+       Fix for 4077106, make sure that mouse wheeling in overflow sections uses 40 rather than 10 as the base line
+       step.
+       
+        Reviewed by darin
+
+        * khtml/rendering/render_layer.cpp:
+        (RenderLayer::positionScrollbars):
+        * kwq/KWQScrollBar.mm:
+        (QScrollBar::scroll):
+
 2005-04-04  Vicki Murley  <vicki@apple.com>
 
         Reviewed by Maciej.
index c2e80ba512ae54d64b3bb86d7f78ad2ba2393620..f3b7b8f972a28f103f9f16b4e0b7929832f89766 100644 (file)
 #include "KWQKHTMLPart.h" // For Dashboard.
 #endif
 
+// These match the numbers we use over in WebKit (WebFrameView.m).
+#define LINE_STEP   40
+#define PAGE_KEEP   40
+
 using namespace DOM;
 using namespace khtml;
 
@@ -655,9 +659,6 @@ RenderLayer::positionScrollbars(const QRect& absBounds)
     }
 }
 
-#define LINE_STEP   10
-#define PAGE_KEEP   40
-
 int RenderLayer::scrollWidth()
 {
     if (m_scrollDimensionsDirty)
index 49bc194e1cf85a91d523503de9c68e4350df469e..196214048ca36095773e8dc83d483a375d27b9bc 100644 (file)
@@ -207,7 +207,7 @@ bool QScrollBar::scroll(KWQScrollDirection direction, KWQScrollGranularity granu
     float delta = 0.0;
     if ((direction == KWQScrollUp && m_orientation == Vertical) || (direction == KWQScrollLeft && m_orientation == Horizontal)) {
         if (granularity == KWQScrollLine) {
-            delta = -(m_lineStep * 4);
+            delta = -m_lineStep;
         } else if (granularity == KWQScrollPage) {
             delta = -m_pageStep;
         } else if (granularity == KWQScrollDocument) {
@@ -217,7 +217,7 @@ bool QScrollBar::scroll(KWQScrollDirection direction, KWQScrollGranularity granu
         }
     } else if ((direction == KWQScrollDown && m_orientation == Vertical) || (direction == KWQScrollRight && m_orientation == Horizontal)) {
         if (granularity == KWQScrollLine) {
-            delta = (m_lineStep * 4);
+            delta = m_lineStep;
         } else if (granularity == KWQScrollPage) {
             delta = m_pageStep;
         } else if (granularity == KWQScrollDocument) {