nodebb-plugin-pdf-secure 1.2.21 → 1.2.22

package/package.json

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

package/static/viewer.html

@@ -1582,7 +1582,7 @@
             align-items: center;
             justify-content: center;
             background: linear-gradient(180deg, rgba(31,31,31,0.95) 0%, rgba(20,20,20,0.98) 100%);
-            z-index: 5;
+            z-index: 15;
             text-align: center;
             padding: 20px;
         }
@@ -2249,8 +2249,16 @@
                 pageEl.appendChild(overlay);
             }
 
+            function isOverlayTampered(overlay) {
+                var s = overlay.style;
+                return s.position !== 'absolute' || s.display !== 'flex' ||
+                    s.zIndex !== '15' || s.opacity !== '1' ||
+                    s.visibility !== 'visible' || s.pointerEvents !== 'auto';
+            }
+
             function resetOverlayCSS(overlay) {
-                overlay.style.cssText = 'position:absolute;top:0;left:0;right:0;bottom:0;display:flex;z-index:5;opacity:1;visibility:visible;pointer-events:auto;';
+                if (!isOverlayTampered(overlay)) return; // Skip if already correct (prevents observer loop)
+                overlay.style.cssText = 'position:absolute;top:0;left:0;right:0;bottom:0;display:flex;z-index:15;opacity:1;visibility:visible;pointer-events:auto;';
             }
 
             function applyPageLocks() {
@@ -2313,9 +2321,11 @@
                     for (var i = 0; i < mutations.length; i++) {
                         var target = mutations[i].target;
                         if (!target || !target.classList) continue;
-                        // Overlay CSS tampered - reset it
+                        // Overlay CSS tampered - reset only if values actually differ
                         if (target.classList.contains('page-lock-overlay')) {
-                            resetOverlayCSS(target);
+                            if (isOverlayTampered(target)) {
+                                resetOverlayCSS(target);
+                            }
                         }
                         // Blur class removed from locked page - re-add it
                         if (target.classList.contains('page') && !target.classList.contains('page-locked-blur')) {
@@ -2907,6 +2917,9 @@
 
             // Annotation Layer with Persistence
             async function injectAnnotationLayer(pageNum) {
+                // Skip annotation injection on locked pages
+                if (premiumInfo && !premiumInfo.isPremium && pageNum > 1) return;
+
                 const pageView = pdfViewer.getPageView(pageNum - 1);
                 if (!pageView?.div) return;