@koine/dom 1.0.101 → 1.0.103

package/node/scrollTo.js

@@ -10,22 +10,34 @@ var off_1 = require("./off");
  *
  * @param offset - offset to scroll to
  * @param callback - callback function
+ * @param [fallbackTimeout] - this appears to be needed in some hard to reproduce scenario on safari, where the callback seem to be never called
+ * @param [behavior="smooth"]
  */
-function scrollTo(offset, callback) {
-    var fixedOffset = offset.toFixed();
+function scrollTo(destination, callback, fallbackTimeout, behavior) {
+    var fixedDestination = destination.toFixed();
     if (callback) {
+        var callbackFired_1 = false;
         var onScroll_1 = function () {
-            if (window.pageYOffset.toFixed() === fixedOffset) {
+            if (window.pageYOffset.toFixed() === fixedDestination) {
                 (0, off_1.off)(window, "scroll", onScroll_1);
+                callbackFired_1 = true;
                 callback();
             }
         };
         (0, on_1.on)(window, "scroll", onScroll_1);
         onScroll_1();
+        if (fallbackTimeout) {
+            setTimeout(function () {
+                if (!callbackFired_1) {
+                    (0, off_1.off)(window, "scroll", onScroll_1);
+                    callback();
+                }
+            }, fallbackTimeout);
+        }
     }
     window.scrollTo({
-        top: offset,
-        behavior: "smooth",
+        top: destination,
+        behavior: behavior || "smooth",
     });
 }
 exports.scrollTo = scrollTo;

package/package.json

@@ -4,12 +4,12 @@
   "main": "./node/index.js",
   "typings": "./index.d.ts",
   "dependencies": {
-    "@koine/utils": "1.0.101",
+    "@koine/utils": "1.0.103",
     "ts-debounce": "^4.0.0",
     "tslib": "^2.4.0"
   },
   "peerDependencies": {},
-  "version": "1.0.101",
+  "version": "1.0.103",
   "module": "./index.js",
   "types": "./index.d.ts"
 }

package/scrollTo.d.ts

@@ -5,6 +5,8 @@
  *
  * @param offset - offset to scroll to
  * @param callback - callback function
+ * @param [fallbackTimeout] - this appears to be needed in some hard to reproduce scenario on safari, where the callback seem to be never called
+ * @param [behavior="smooth"]
  */
-export declare function scrollTo(offset: number, callback?: () => void): void;
+export declare function scrollTo(destination: number, callback?: () => void, fallbackTimeout?: number, behavior?: ScrollBehavior): void;
 export default scrollTo;

package/scrollTo.js

@@ -7,22 +7,34 @@ import { off } from "./off";
  *
  * @param offset - offset to scroll to
  * @param callback - callback function
+ * @param [fallbackTimeout] - this appears to be needed in some hard to reproduce scenario on safari, where the callback seem to be never called
+ * @param [behavior="smooth"]
  */
-export function scrollTo(offset, callback) {
-    var fixedOffset = offset.toFixed();
+export function scrollTo(destination, callback, fallbackTimeout, behavior) {
+    var fixedDestination = destination.toFixed();
     if (callback) {
+        var callbackFired_1 = false;
         var onScroll_1 = function () {
-            if (window.pageYOffset.toFixed() === fixedOffset) {
+            if (window.pageYOffset.toFixed() === fixedDestination) {
                 off(window, "scroll", onScroll_1);
+                callbackFired_1 = true;
                 callback();
             }
         };
         on(window, "scroll", onScroll_1);
         onScroll_1();
+        if (fallbackTimeout) {
+            setTimeout(function () {
+                if (!callbackFired_1) {
+                    off(window, "scroll", onScroll_1);
+                    callback();
+                }
+            }, fallbackTimeout);
+        }
     }
     window.scrollTo({
-        top: offset,
-        behavior: "smooth",
+        top: destination,
+        behavior: behavior || "smooth",
     });
 }
 export default scrollTo;