[GTK] Pinch Zooming has no maximum
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 May 2019 15:46:36 +0000 (15:46 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 May 2019 15:46:36 +0000 (15:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=194865

Patch by Alexander Mikhaylenko <exalm7659@gmail.com> on 2019-05-08
Reviewed by Michael Catanzaro.

Set maximum zoom to 3.0, reflecting the value on macOS and iOS.

* UIProcess/gtk/GestureController.cpp:
(WebKit::GestureController::ZoomGesture::scaleChanged):

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/gtk/GestureController.cpp

index b6d3be8..4a41917 100644 (file)
@@ -1,3 +1,15 @@
+2019-05-08  Alexander Mikhaylenko  <exalm7659@gmail.com>
+
+        [GTK] Pinch Zooming has no maximum
+        https://bugs.webkit.org/show_bug.cgi?id=194865
+
+        Reviewed by Michael Catanzaro.
+
+        Set maximum zoom to 3.0, reflecting the value on macOS and iOS.
+
+        * UIProcess/gtk/GestureController.cpp:
+        (WebKit::GestureController::ZoomGesture::scaleChanged):
+
 2019-05-08  Rob Buis  <rbuis@igalia.com>
 
         Link prefetch not useful for top-level navigation
index 3d27d6b..5f58449 100644 (file)
@@ -34,6 +34,8 @@
 namespace WebKit {
 using namespace WebCore;
 
+static const double maximumZoom = 3.0;
+
 GestureController::GestureController(GtkWidget* widget, std::unique_ptr<GestureControllerClient>&& client)
     : m_client(WTFMove(client))
     , m_dragGesture(widget, *m_client)
@@ -214,6 +216,8 @@ void GestureController::ZoomGesture::scaleChanged(ZoomGesture* zoomGesture, doub
     zoomGesture->m_scale = zoomGesture->m_initialScale * scale;
     if (zoomGesture->m_scale < 1.0)
         zoomGesture->m_scale = 1.0;
+    if (zoomGesture->m_scale > maximumZoom)
+        zoomGesture->m_scale = maximumZoom;
 
     zoomGesture->m_viewPoint = zoomGesture->center();