fix: chance RangeError

marmelab · Apr 22, 2020 · 3856336 · 3856336
1 parent 358baf3
commit 3856336
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/src/species/clicker.js b/src/species/clicker.js
@@ -20,10 +20,10 @@ const getDefaultConfig = (randomizer) => {
     const defaultPositionSelector = () => {
         return [
             randomizer.natural({
-                max: document.documentElement.clientWidth - 1,
+                max: Math.max(0, document.documentElement.clientWidth - 1),
             }),
             randomizer.natural({
-                max: document.documentElement.clientHeight - 1,
+                max: Math.max(0, document.documentElement.clientHeight - 1),
             }),
         ];
     };

diff --git a/src/species/toucher.js b/src/species/toucher.js
@@ -17,10 +17,10 @@ const getDefaultConfig = (randomizer) => {
     const defaultPositionSelector = () => {
         return [
             randomizer.natural({
-                max: document.documentElement.clientWidth - 1,
+                max: Math.max(0, document.documentElement.clientWidth - 1),
             }),
             randomizer.natural({
-                max: document.documentElement.clientHeight - 1,
+                max: Math.max(0, document.documentElement.clientHeight - 1),
             }),
         ];
     };

diff --git a/src/species/typer.js b/src/species/typer.js
@@ -61,10 +61,10 @@ export default (userConfig) => (logger, randomizer) => {
         const eventType = randomizer.pick(config.eventTypes);
         const key = config.keyGenerator();
         const posX = randomizer.natural({
-            max: documentElement.clientWidth - 1,
+            max: Math.max(0, documentElement.clientWidth - 1),
         });
         const posY = randomizer.natural({
-            max: documentElement.clientHeight - 1,
+            max: Math.max(0, documentElement.clientHeight - 1),
         });
         const targetElement = config.targetElement(posX, posY);