@zaplier/sdk 1.7.4 → 1.7.5

package/dist/index.cjs

@@ -19574,27 +19574,52 @@ class AutoTracker {
         if (this.config.trackScrolls) {
             const scrollHandler = this.createEnhancedScrollHandler();
             this.delegationHandlers.set('scroll', scrollHandler);
-            // Always use document for scroll events (window doesn't work with removeEventListener)
-            document.addEventListener('scroll', scrollHandler, this.getEventOptions('scroll'));
+            // MAJOR FIX: Use window instead of document for scroll events
+            // Document doesn't reliably receive scroll events in all browsers
+            const scrollTarget = window;
+            const scrollOptions = {
+                passive: false, // Changed from passive: true to ensure handler executes
+                capture: false
+            };
+            scrollTarget.addEventListener('scroll', scrollHandler, scrollOptions);
+            // Also add to document as backup (some scenarios need both)
+            document.addEventListener('scroll', scrollHandler, scrollOptions);
             if (this.config.debug) {
-                console.log("[📜 AutoTracker] ✅ Scroll delegation handler added to document");
+                console.log("[📜 AutoTracker] ✅ Scroll delegation handler added to BOTH window AND document");
                 console.log("[📜 AutoTracker] 📋 Event listener details:", {
-                    target: 'document',
+                    primaryTarget: 'window',
+                    backupTarget: 'document',
                     eventType: 'scroll',
                     handlerType: typeof scrollHandler,
-                    options: this.getEventOptions('scroll'),
+                    options: scrollOptions,
                     handlerStored: this.delegationHandlers.has('scroll')
                 });
-                // Test if handler is actually working
+                // Enhanced test function with real scroll simulation
                 const testHandler = () => {
                     console.log("[📜 AutoTracker] 🧪 MANUAL SCROLL TEST - Handler should trigger");
                     scrollHandler(new Event('scroll'));
                 };
-                // Expose test function globally for manual testing
+                const testRealScroll = () => {
+                    console.log("[📜 AutoTracker] 🧪 TESTING REAL SCROLL EVENT");
+                    window.scrollBy(0, 1); // Trigger real scroll
+                    setTimeout(() => {
+                        window.scrollBy(0, -1); // Scroll back
+                    }, 100);
+                };
+                // Expose test functions globally for manual testing
                 if (typeof window !== 'undefined') {
                     window.testScrollHandler = testHandler;
-                    console.log("[📜 AutoTracker] 🧪 Manual test available: window.testScrollHandler()");
+                    window.testRealScroll = testRealScroll;
+                    console.log("[📜 AutoTracker] 🧪 Manual tests available:");
+                    console.log("[📜 AutoTracker] 🧪 - window.testScrollHandler() (direct handler test)");
+                    console.log("[📜 AutoTracker] 🧪 - window.testRealScroll() (real scroll test)");
                 }
+                // Add debug listener to verify scroll events are firing
+                const debugScrollListener = () => {
+                    console.log("[📜 AutoTracker] 🔍 DEBUG: Real scroll event detected!");
+                };
+                window.addEventListener('scroll', debugScrollListener, { once: true });
+                console.log("[📜 AutoTracker] 🔍 Debug scroll listener added (will fire once on next scroll)");
                 // Immediate test of element detection
                 setTimeout(() => {
                     const scrollElements = this.getCachedScrollElements();
@@ -19611,16 +19636,11 @@ class AutoTracker {
                             });
                         });
                     }
-                    // Also test manually trigger one element
-                    if (scrollElements.length > 0) {
-                        const firstElement = scrollElements[0];
-                        if (firstElement) {
-                            console.log(`[📜 AutoTracker] 🧪 Testing manual trigger on first element:`, firstElement.getAttribute('data-track-scroll'));
-                            setTimeout(() => {
-                                console.log("[📜 AutoTracker] 🧪 Manual trigger test executed");
-                            }, 500);
-                        }
-                    }
+                    // Test auto-scroll detection
+                    setTimeout(() => {
+                        console.log("[📜 AutoTracker] 🧪 Auto-testing scroll detection in 1 second...");
+                        testRealScroll();
+                    }, 1000);
                 }, 100);
             }
         }
@@ -20034,11 +20054,25 @@ class AutoTracker {
      */
     stop() {
         if (this.config.debug) {
-            console.log('[AutoTracker] Stopping auto tracking and cleaning up...');
+            console.log('[📜 AutoTracker] Stopping enhanced auto tracking and cleaning up...');
         }
-        // Remove delegation handlers
+        // Remove delegation handlers from correct targets
         this.delegationHandlers.forEach((handler, eventType) => {
-            document.removeEventListener(eventType, handler);
+            if (eventType === 'click') {
+                const delegationTarget = this.getOptimalDelegationTarget();
+                delegationTarget.removeEventListener('click', handler);
+            }
+            else if (eventType === 'scroll') {
+                // Remove from both window and document (as we added to both)
+                window.removeEventListener('scroll', handler);
+                document.removeEventListener('scroll', handler);
+                if (this.config.debug) {
+                    console.log('[📜 AutoTracker] Removed scroll listeners from window and document');
+                }
+            }
+            else {
+                document.removeEventListener(eventType, handler);
+            }
         });
         this.delegationHandlers.clear();
         // Disconnect observers
@@ -20053,6 +20087,9 @@ class AutoTracker {
         // Clear caches
         this.observedElements.clear();
         this.isInitialized = false;
+        if (this.config.debug) {
+            console.log('[📜 AutoTracker] Enhanced cleanup completed');
+        }
     }
     /**
      * Legacy method - now handled by intelligent mutation observer