nodebb-plugin-pdf-secure2 1.4.7 → 1.4.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-pdf-secure2",
-  "version": "1.4.7",
+  "version": "1.4.8",
   "description": "Secure PDF viewer plugin for NodeBB - prevents downloading, enables canvas-only rendering with Premium group support",
   "main": "library.js",
   "repository": {

package/static/viewer-app.js

@@ -2787,18 +2787,23 @@
 
 
     // Mouse wheel / touchpad pinch zoom with Ctrl (clamped 0.5x-5x)
+    // Touchpad pinch fires 60+ small events/sec → must heavily dampen.
+    // Mouse wheel fires fewer large events → needs separate handling.
     let zoomAccumulator = 0;
     let zoomRafId = null;
     container.addEventListener('wheel', (e) => {
         if (!e.ctrlKey) return;
         e.preventDefault();
 
-        // Touchpad pinch sends small deltaY values; mouse wheel sends large ones (100+)
-        // Normalize: clamp large mouse wheel jumps, keep touchpad's fine granularity
         const raw = e.deltaY;
-        const clamped = Math.max(-10, Math.min(10, raw));
-        const sensitivity = 0.008;
-        zoomAccumulator -= clamped * sensitivity;
+        const isMouseWheel = Math.abs(raw) >= 50;
+        if (isMouseWheel) {
+            zoomAccumulator += (raw < 0 ? 0.08 : -0.08);
+        } else {
+            const clamped = Math.max(-5, Math.min(5, raw));
+            zoomAccumulator -= clamped * 0.002;
+        }
+        zoomAccumulator = Math.max(-0.04, Math.min(0.04, zoomAccumulator));
 
         if (!zoomRafId) {
             zoomRafId = requestAnimationFrame(() => {

package/static/viewer.html

@@ -7581,8 +7581,8 @@
 
 
             // Mouse wheel / touchpad pinch zoom with Ctrl (clamped 0.5x–5x)
-            // Touchpad pinch sends many small deltaY events; mouse wheel sends large ones (100+).
-            // We accumulate deltas and apply once per frame to keep zoom smooth on both.
+            // Touchpad pinch fires 60+ small events/sec → must heavily dampen.
+            // Mouse wheel fires fewer large events → needs separate handling.
             let zoomAccumulator = 0;
             let zoomRafId = null;
             container.addEventListener('wheel', (e) => {
@@ -7590,9 +7590,18 @@
                 e.preventDefault();
 
                 const raw = e.deltaY;
-                const clamped = Math.max(-10, Math.min(10, raw));
-                const sensitivity = 0.008;
-                zoomAccumulator -= clamped * sensitivity;
+                // Detect input type: mouse wheel sends |deltaY| >= 50, touchpad sends < 50
+                const isMouseWheel = Math.abs(raw) >= 50;
+                if (isMouseWheel) {
+                    // Mouse wheel: fixed step per click (ignore magnitude)
+                    zoomAccumulator += (raw < 0 ? 0.08 : -0.08);
+                } else {
+                    // Touchpad: very low sensitivity, clamp individual delta
+                    const clamped = Math.max(-5, Math.min(5, raw));
+                    zoomAccumulator -= clamped * 0.002;
+                }
+                // Cap total accumulator per frame (max ~4% zoom per frame)
+                zoomAccumulator = Math.max(-0.04, Math.min(0.04, zoomAccumulator));
 
                 if (!zoomRafId) {
                     zoomRafId = requestAnimationFrame(() => {