From 5d703baed11f8f1b44df5ed7e03142cbdce839c5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?William=20Herg=C3=A8s?= <william@herges.fr>
Date: Sun, 10 Aug 2025 13:27:49 +0200
Subject: refactor(background): use event listener instead of onload

---
 src/component/MosaicBackground.astro | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

(limited to 'src/component/MosaicBackground.astro')

diff --git a/src/component/MosaicBackground.astro b/src/component/MosaicBackground.astro
index c8e01f5..8c38bcf 100644
--- a/src/component/MosaicBackground.astro
+++ b/src/component/MosaicBackground.astro
@@ -7,7 +7,7 @@ const { height = "400" } = Astro.props;
 </div>
 
 <script>
-  window.onload = () => {
+  window.addEventListener("load", () => {
     // because Astro doesn't support defer
     document.querySelectorAll(".bg").forEach((e) => {
       const children = e.children;
@@ -24,9 +24,15 @@ const { height = "400" } = Astro.props;
         // adding elements
         while (i < children.length && currentWidth <= max) {
           let current = children[i++];
-          let ratio = current.clientWidth / current.clientHeight;
-          line.appendChild(current.cloneNode(true));
-          currentWidth += ratio * height;
+          if (!(current instanceof HTMLScriptElement)) {
+            let ratio = current.clientWidth / current.clientHeight;
+            if (!isNaN(ratio)) {
+              line.appendChild(current.cloneNode(true));
+              currentWidth += ratio * height;
+            } else {
+              console.error("is NaN", current)
+            }
+          }
         }
         // setting height
         let h = (max * height) / currentWidth;
@@ -39,5 +45,5 @@ const { height = "400" } = Astro.props;
       }
       e.replaceChildren(...newChildren);
     });
-  };
+  })
 </script>
-- 
cgit v1.2.3