2009-04-28 Xan Lopez <xlopez@igalia.com>
authorxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Apr 2009 16:06:21 +0000 (16:06 +0000)
committerxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Apr 2009 16:06:21 +0000 (16:06 +0000)
        Reviewed by Gustavo Noronha.

        Leave a bit of space from the previous page when advacing a whole
        page. This matches the behavior of our scrollbar scrolling code
        and is just nicer in general.

        * webkit/webkitwebview.cpp:

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

WebKit/gtk/ChangeLog
WebKit/gtk/webkit/webkitwebview.cpp

index a3cd618..0b73afe 100644 (file)
@@ -1,5 +1,15 @@
 2009-04-28  Xan Lopez  <xlopez@igalia.com>
 
+        Reviewed by Gustavo Noronha.
+
+        Leave a bit of space from the previous page when advacing a whole
+        page. This matches the behavior of our scrollbar scrolling code
+        and is just nicer in general.
+
+        * webkit/webkitwebview.cpp:
+
+2009-04-28  Xan Lopez  <xlopez@igalia.com>
+
         Unreviewed: committed a few bits by mistake, remove them.
 
         * WebCoreSupport/EditorClientGtk.cpp:
index 6a369d6..453604d 100644 (file)
@@ -827,9 +827,9 @@ static gboolean webkit_web_view_real_move_cursor (WebKitWebView* webView, GtkMov
         break;
     case GTK_MOVEMENT_PAGES:
         if (count == 1)
-            view->scrollBy(IntSize(0, view->visibleHeight()));
+            view->scrollBy(IntSize(0, view->visibleHeight() * 0.9));
         else if (count == -1)
-            view->scrollBy(IntSize(0, -view->visibleHeight()));
+            view->scrollBy(IntSize(0, -view->visibleHeight() * 0.9));
         break;
     case GTK_MOVEMENT_BUFFER_ENDS:
         if (count == 1)