From b422d079fb7bef6e63730dc64cb0ea59b89c98e2 Mon Sep 17 00:00:00 2001
From: Emil Gulamov <125820963+mearashadowfax@users.noreply.github.com>
Date: Sat, 24 Feb 2024 23:59:08 +0400
Subject: [PATCH] Refactor event handlers to remove unused event parameter

In several areas of the code, event handlers had an unused event parameter. This change removes these unused parameters to improve the code's readability and maintainability. Specifically, the changes were made in the "Navbar.astro", "[...slug].astro", and "services.astro" components.
---
 src/components/sections/Navbar.astro          |  2 +-
 .../ui/banners/AnnouncementBanner.astro       | 87 ++++++++++---------
 src/pages/products/[...slug].astro            |  2 +-
 src/pages/services.astro                      |  2 +-
 4 files changed, 47 insertions(+), 46 deletions(-)

diff --git a/src/components/sections/Navbar.astro b/src/components/sections/Navbar.astro
index 44e6297..027269f 100644
--- a/src/components/sections/Navbar.astro
+++ b/src/components/sections/Navbar.astro
@@ -210,7 +210,7 @@ import { navBarLinks } from "../../utils/navigation.ts";
 
   window
     .matchMedia("(prefers-color-scheme: dark)")
-    .addEventListener("change", (e) => {
+    .addEventListener("change", () => {
       if (HSThemeAppearance.getOriginalAppearance() === "auto") {
         HSThemeAppearance.setAppearance("auto", false);
       }
diff --git a/src/components/ui/banners/AnnouncementBanner.astro b/src/components/ui/banners/AnnouncementBanner.astro
index b53420f..a63393a 100644
--- a/src/components/ui/banners/AnnouncementBanner.astro
+++ b/src/components/ui/banners/AnnouncementBanner.astro
@@ -10,28 +10,46 @@ interface Props {
 ---
 
 <astro-banner btnId={btnId}>
-<div
-  class="fixed bottom-0 start-1/2 z-50 mx-auto w-full -translate-x-1/2 transform p-6 sm:max-w-4xl"
->
   <div
-    class="rounded-xl bg-neutral-800 bg-[url('src/images/banner-pattern.svg')] bg-cover bg-center bg-no-repeat p-4 text-center shadow-sm dark:bg-neutral-200"
+    class="fixed bottom-0 start-1/2 z-50 mx-auto w-full -translate-x-1/2 transform p-6 sm:max-w-4xl"
   >
-    <div class="flex items-center justify-center">
-      <div class="ml-auto">
-        <p
-          class="me-2 inline-block font-medium text-neutral-200 dark:text-neutral-800"
+    <div
+      class="rounded-xl bg-neutral-800 bg-[url('src/images/banner-pattern.svg')] bg-cover bg-center bg-no-repeat p-4 text-center shadow-sm dark:bg-neutral-200"
+    >
+      <div class="flex items-center justify-center">
+        <div class="ml-auto">
+          <p
+            class="me-2 inline-block font-medium text-neutral-200 dark:text-neutral-800"
+          >
+            {title}
+          </p>
+          <a
+            class="group inline-flex items-center gap-x-2 rounded-full border-2 border-neutral-50 px-3 py-2 text-sm font-semibold text-neutral-50 transition duration-300 hover:border-neutral-100/70 hover:text-neutral-100/70 disabled:pointer-events-none disabled:opacity-50 dark:border-neutral-700 dark:text-neutral-800 dark:hover:border-neutral-700/70 dark:hover:text-neutral-800/70 dark:focus:outline-none"
+            href={url}
+            target="_blank"
+          >
+            {btnTitle}
+            <svg
+              class="size-4 flex-shrink-0 transition duration-300 group-hover:translate-x-1"
+              width="24"
+              height="24"
+              viewBox="0 0 24 24"
+              fill="none"
+              stroke="currentColor"
+              stroke-width="2"
+              stroke-linecap="round"
+              stroke-linejoin="round"><path d="m9 18 6-6-6-6"></path></svg
+            >
+          </a>
+        </div>
+        <button
+          type="button"
+          class="ml-auto inline-flex items-center gap-x-2 rounded-full border border-transparent bg-gray-100 p-2 text-sm font-semibold text-gray-800 hover:bg-gray-200 disabled:pointer-events-none disabled:opacity-50 dark:bg-neutral-700 dark:text-neutral-50 dark:hover:bg-neutral-700/80 dark:hover:text-neutral-50 dark:focus:outline-none"
+          id={btnId}
         >
-          {title}
-        </p>
-        <a
-          class="group inline-flex items-center gap-x-2 rounded-full border-2 border-neutral-50 px-3 py-2 text-sm font-semibold text-neutral-50 transition duration-300 hover:border-neutral-100/70 hover:text-neutral-100/70 disabled:pointer-events-none disabled:opacity-50 dark:border-neutral-700 dark:text-neutral-800 dark:hover:border-neutral-700/70 dark:hover:text-neutral-800/70 dark:focus:outline-none"
-          href={url}
-          target="_blank"
-
-        >
-          {btnTitle}
+          <span class="sr-only">Dismiss</span>
           <svg
-            class="size-4 flex-shrink-0 transition duration-300 group-hover:translate-x-1"
+            class="size-5 flex-shrink-0"
             width="24"
             height="24"
             viewBox="0 0 24 24"
@@ -39,41 +57,24 @@ interface Props {
             stroke="currentColor"
             stroke-width="2"
             stroke-linecap="round"
-            stroke-linejoin="round"><path d="m9 18 6-6-6-6"></path></svg
+            stroke-linejoin="round"
+            ><path d="M18 6 6 18"></path><path d="m6 6 12 12"></path></svg
           >
-        </a>
+        </button>
       </div>
-      <button
-        type="button"
-        class="ml-auto inline-flex items-center gap-x-2 rounded-full border border-transparent bg-gray-100 p-2 text-sm font-semibold text-gray-800 hover:bg-gray-200 disabled:pointer-events-none disabled:opacity-50 dark:bg-neutral-700 dark:text-neutral-50 dark:hover:bg-neutral-700/80 dark:hover:text-neutral-50 dark:focus:outline-none"
-        id={btnId}
-      >
-        <span class="sr-only">Dismiss</span>
-        <svg
-          class="size-5 flex-shrink-0"
-          width="24"
-          height="24"
-          viewBox="0 0 24 24"
-          fill="none"
-          stroke="currentColor"
-          stroke-width="2"
-          stroke-linecap="round"
-          stroke-linejoin="round"
-          ><path d="M18 6 6 18"></path><path d="m6 6 12 12"></path></svg
-        >
-      </button>
     </div>
   </div>
-</div>
 </astro-banner>
 <script>
   class AstroBanner extends HTMLElement {
     connectedCallback() {
-      const btnId = this.getAttribute('btnId');
+      const btnId = this.getAttribute("btnId");
       const button = this.querySelector(`#${btnId}`);
-      button.addEventListener('click', () => this.remove());
+      if (button != null) {
+        button.addEventListener("click", () => this.remove());
+      }
     }
   }
 
-  customElements.define('astro-banner', AstroBanner);
+  customElements.define("astro-banner", AstroBanner);
 </script>
diff --git a/src/pages/products/[...slug].astro b/src/pages/products/[...slug].astro
index 362be84..3ead72a 100644
--- a/src/pages/products/[...slug].astro
+++ b/src/pages/products/[...slug].astro
@@ -197,7 +197,7 @@ const { product } = Astro.props;
 
   <script is:inline src="/scripts/vendor/gsap/gsap.min.js"></script>
   <script>
-    window.addEventListener("load", (event) => {
+    window.addEventListener("load", () => {
       if (window.gsap) {
         const gsap = window.gsap;
         gsap.set("#fadeText", {
diff --git a/src/pages/services.astro b/src/pages/services.astro
index 7b714f0..acea630 100644
--- a/src/pages/services.astro
+++ b/src/pages/services.astro
@@ -110,7 +110,7 @@ const articles: Article[] = [
   If 'single' is true, it displays one image, otherwise it displays two.
   -->
   {
-    articles.map((article, index) => {
+    articles.map(article => {
       return article.isRightSection ? (
         <RightSection
           title={article.title}