2009-03-12 Gustavo Noronha Silva <gns@gnome.org>
authorkov@webkit.org <kov@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Mar 2009 15:00:25 +0000 (15:00 +0000)
committerkov@webkit.org <kov@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Mar 2009 15:00:25 +0000 (15:00 +0000)
        Reviewed by Alp Toker.

        https://bugs.webkit.org/show_bug.cgi?id=24541
        Scrolling with home and end keys not always works

        Also make page up and page down keys be handled by the webview key
        event code, so that they also work in cases where the GTK+
        scrollbars don't handle them directly, like in the bugzill's patch
        review page.

        * webkit/webkitwebview.cpp:

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

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

index 959a673..6c1827e 100644 (file)
@@ -5,6 +5,20 @@
         https://bugs.webkit.org/show_bug.cgi?id=24541
         Scrolling with home and end keys not always works
 
+        Also make page up and page down keys be handled by the webview key
+        event code, so that they also work in cases where the GTK+
+        scrollbars don't handle them directly, like in the bugzill's patch
+        review page.
+
+        * webkit/webkitwebview.cpp:
+
+2009-03-12  Gustavo Noronha Silva  <gns@gnome.org>
+
+        Reviewed by Alp Toker.
+
+        https://bugs.webkit.org/show_bug.cgi?id=24541
+        Scrolling with home and end keys not always works
+
         Make home and end keys behave more consistently for scrolling the
         view.
 
index 4cc608b..bed45d5 100644 (file)
@@ -456,6 +456,12 @@ static gboolean webkit_web_view_key_press_event(GtkWidget* widget, GdkEventKey*
         else
             view->scrollBy(IntSize(0, view->visibleHeight()));
         return TRUE;
+    case GDK_Page_Up:
+        view->scrollBy(IntSize(0, -view->visibleHeight()));
+        return TRUE;
+    case GDK_Page_Down:
+        view->scrollBy(IntSize(0, view->visibleHeight()));
+        return TRUE;
     case GDK_Home:
         view->scrollBy(IntSize(0, -view->contentsHeight()));
         return TRUE;