@glitchr/stickyjs 1.0.43 → 1.0.50

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@glitchr/stickyjs",
-  "version": "1.0.43",
+  "version": "1.0.50",
   "description": "Sticky scroll library",
   "main": "src/index.js",
   "repository": {

package/src/js/sticky.js

@@ -677,7 +677,20 @@ $.fn.serializeObject = function () {
         return parseInt(closest);
     }
 
-    Sticky.userScroll = function(el = undefined) { return $(el === undefined ? document.documentElement : el).closestScrollable().prop("user-scroll") ?? true; }
+    Sticky.userScroll = function(el = undefined) {
+        // Defensive: closestScrollable() can return a value without .prop in
+        // some race conditions (transient DOM during transparent.js page
+        // transitions, sticky-scrollpercent triggers fired against detached
+        // nodes). Default to true so the autoscroll falls back to "user is
+        // scrolling, don't auto-advance."
+        try {
+            var $target = $(el === undefined ? document.documentElement : el);
+            if (!$target || !$target.length) return true;
+            var $scroll = $target.closestScrollable && $target.closestScrollable();
+            if (!$scroll || typeof $scroll.prop !== "function") return true;
+            return $scroll.prop("user-scroll") ?? true;
+        } catch (e) { return true; }
+    }
     Sticky.scrollTo = function(dict, callback = function() {}, el = window)
     {
         el = $(el).length ? $(el)[0] : window;
@@ -707,6 +720,8 @@ $.fn.serializeObject = function () {
 
         if(speed) {
 
+            var distanceX = 0, distanceY = 0;
+
             var currentScrollX = $(el)[0].scrollLeft;
             if(currentScrollX < scrollLeft || scrollLeft == 0) // Going to the right
                 distanceX = Math.abs(scrollLeft - currentScrollX);