@zaplier/sdk 1.7.4 → 1.7.5

package/dist/sdk.js

@@ -19576,27 +19576,52 @@
             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();
@@ -19613,16 +19638,11 @@
                                 });
                             });
                         }
-                        // 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);
                 }
             }
@@ -20036,11 +20056,25 @@
          */
         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
@@ -20055,6 +20089,9 @@
             // 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