From 66e24e48b234499a2d46ad6b7f81bdf0c84814b0 Mon Sep 17 00:00:00 2001
From: Pablo Moreno <pablo@kasfactory.net>
Date: Sat, 23 Nov 2024 20:20:14 +0100
Subject: [PATCH] fix: improve navbar

---
 .../components/cart/cart_dropdown.html        |  35 +++
 shop/templates/components/generic/navbar.html | 223 ++----------------
 .../components/users/user_dropdown.html       |  46 ++++
 shop/templates/shop/product_detail.html       |   9 +-
 theme/static/theme/js/theme.js                |   4 +-
 5 files changed, 106 insertions(+), 211 deletions(-)
 create mode 100644 shop/templates/components/cart/cart_dropdown.html
 create mode 100644 shop/templates/components/users/user_dropdown.html

diff --git a/shop/templates/components/cart/cart_dropdown.html b/shop/templates/components/cart/cart_dropdown.html
new file mode 100644
index 0000000..017f940
--- /dev/null
+++ b/shop/templates/components/cart/cart_dropdown.html
@@ -0,0 +1,35 @@
+{% load i18n %}
+
+<div class="grid grid-cols-2">
+  <div>
+    <a href="#" class="truncate text-sm font-semibold leading-none text-gray-900 dark:text-white hover:underline">Apple iPhone 15</a>
+    <p class="mt-0.5 truncate text-sm font-normal text-gray-500 dark:text-gray-400">$599</p>
+  </div>
+
+  <div class="flex items-center justify-end gap-6">
+    <p class="text-sm font-normal leading-none text-gray-500 dark:text-gray-400">Qty: 1</p>
+
+    <button data-tooltip-target="tooltipRemoveItem1a" type="button"
+            class="text-red-600 hover:text-red-700 dark:text-red-500 dark:hover:text-red-600">
+      <span class="sr-only"> Remove </span>
+      <svg class="h-4 w-4" aria-hidden="true" xmlns="http://www.w3.org/2000/svg" fill="currentColor"
+           viewBox="0 0 24 24">
+        <path fill-rule="evenodd"
+              d="M2 12a10 10 0 1 1 20 0 10 10 0 0 1-20 0Zm7.7-3.7a1 1 0 0 0-1.4 1.4l2.3 2.3-2.3 2.3a1 1 0 1 0 1.4 1.4l2.3-2.3 2.3 2.3a1 1 0 0 0 1.4-1.4L13.4 12l2.3-2.3a1 1 0 0 0-1.4-1.4L12 10.6 9.7 8.3Z"
+              clip-rule="evenodd"/>
+      </svg>
+    </button>
+    <div id="tooltipRemoveItem1a" role="tooltip"
+         class="tooltip invisible absolute z-10 inline-block rounded-lg bg-gray-900 px-3 py-2 text-sm font-medium text-white opacity-0 shadow-sm transition-opacity duration-300 dark:bg-gray-700">
+      Remove item
+      <div class="tooltip-arrow" data-popper-arrow></div>
+    </div>
+  </div>
+</div>
+
+<a href="#" title=""
+   class="mb-2 me-2 inline-flex w-full items-center justify-center rounded-lg bg-primary-700 px-5 py-2.5 text-sm
+     font-medium text-white hover:bg-primary-800 focus:outline-none focus:ring-4 focus:ring-primary-300
+     dark:bg-primary-600 dark:hover:bg-primary-700 dark:focus:ring-primary-800"
+   role="button"> {% translate 'Ir a pagar' %}
+</a>
diff --git a/shop/templates/components/generic/navbar.html b/shop/templates/components/generic/navbar.html
index dc5aa8b..e00ebc0 100644
--- a/shop/templates/components/generic/navbar.html
+++ b/shop/templates/components/generic/navbar.html
@@ -15,25 +15,25 @@
         <!-- Categories -->
         <ul class="hidden lg:flex items-center justify-start gap-6 md:gap-8 py-3 sm:justify-center">
           <li>
-            <a href="#" title=""
+            <a href="{% url 'shop:index' %}" title=""
                class="flex text-sm font-medium text-gray-900 hover:text-primary-700 dark:text-white dark:hover:text-primary-500">
               {% translate 'Inicio' %}
             </a>
           </li>
           <li class="shrink-0">
-            <a href="#" title=""
+            <a href="{% url 'shop:index' %}" title=""
                class="flex text-sm font-medium text-gray-900 hover:text-primary-700 dark:text-white dark:hover:text-primary-500">
               {% translate 'Más vendidos' %}
             </a>
           </li>
           <li class="shrink-0">
-            <a href="#" title=""
+            <a href="{% url 'shop:index' %}" title=""
                class="flex text-sm font-medium text-gray-900 hover:text-primary-700 dark:text-white dark:hover:text-primary-500">
               {% translate 'Ideas de regalo' %}
             </a>
           </li>
           <li class="shrink-0">
-            <a href="#" title=""
+            <a href="{% url 'shop:index' %}" title=""
                class="text-sm font-medium text-gray-900 hover:text-primary-700 dark:text-white dark:hover:text-primary-500">
               {% translate 'Ofertas del día' %}
             </a>
@@ -69,156 +69,10 @@
         <!-- Cart -->
         <div id="myCartDropdown1"
              class="absolute top-10 hidden z-10 mx-auto max-w-sm space-y-4 overflow-hidden rounded-lg bg-white p-4 antialiased shadow-lg dark:bg-gray-800">
-          <div class="grid grid-cols-2">
-            <div>
-              <a href="#"
-                 class="truncate text-sm font-semibold leading-none text-gray-900 dark:text-white hover:underline">Apple
-                iPhone 15</a>
-              <p class="mt-0.5 truncate text-sm font-normal text-gray-500 dark:text-gray-400">$599</p>
-            </div>
-
-            <div class="flex items-center justify-end gap-6">
-              <p class="text-sm font-normal leading-none text-gray-500 dark:text-gray-400">Qty: 1</p>
-
-              <button data-tooltip-target="tooltipRemoveItem1a" type="button"
-                      class="text-red-600 hover:text-red-700 dark:text-red-500 dark:hover:text-red-600">
-                <span class="sr-only"> Remove </span>
-                <svg class="h-4 w-4" aria-hidden="true" xmlns="http://www.w3.org/2000/svg" fill="currentColor"
-                     viewBox="0 0 24 24">
-                  <path fill-rule="evenodd"
-                        d="M2 12a10 10 0 1 1 20 0 10 10 0 0 1-20 0Zm7.7-3.7a1 1 0 0 0-1.4 1.4l2.3 2.3-2.3 2.3a1 1 0 1 0 1.4 1.4l2.3-2.3 2.3 2.3a1 1 0 0 0 1.4-1.4L13.4 12l2.3-2.3a1 1 0 0 0-1.4-1.4L12 10.6 9.7 8.3Z"
-                        clip-rule="evenodd"/>
-                </svg>
-              </button>
-              <div id="tooltipRemoveItem1a" role="tooltip"
-                   class="tooltip invisible absolute z-10 inline-block rounded-lg bg-gray-900 px-3 py-2 text-sm font-medium text-white opacity-0 shadow-sm transition-opacity duration-300 dark:bg-gray-700">
-                Remove item
-                <div class="tooltip-arrow" data-popper-arrow></div>
-              </div>
-            </div>
-          </div>
-
-          <div class="grid grid-cols-2">
-            <div>
-              <a href="#"
-                 class="truncate text-sm font-semibold leading-none text-gray-900 dark:text-white hover:underline">Apple
-                iPad Air</a>
-              <p class="mt-0.5 truncate text-sm font-normal text-gray-500 dark:text-gray-400">$499</p>
-            </div>
-
-            <div class="flex items-center justify-end gap-6">
-              <p class="text-sm font-normal leading-none text-gray-500 dark:text-gray-400">Qty: 1</p>
-
-              <button data-tooltip-target="tooltipRemoveItem2a" type="button"
-                      class="text-red-600 hover:text-red-700 dark:text-red-500 dark:hover:text-red-600">
-                <span class="sr-only"> Remove </span>
-                <svg class="h-4 w-4" aria-hidden="true" xmlns="http://www.w3.org/2000/svg" fill="currentColor"
-                     viewBox="0 0 24 24">
-                  <path fill-rule="evenodd"
-                        d="M2 12a10 10 0 1 1 20 0 10 10 0 0 1-20 0Zm7.7-3.7a1 1 0 0 0-1.4 1.4l2.3 2.3-2.3 2.3a1 1 0 1 0 1.4 1.4l2.3-2.3 2.3 2.3a1 1 0 0 0 1.4-1.4L13.4 12l2.3-2.3a1 1 0 0 0-1.4-1.4L12 10.6 9.7 8.3Z"
-                        clip-rule="evenodd"/>
-                </svg>
-              </button>
-              <div id="tooltipRemoveItem2a" role="tooltip"
-                   class="tooltip invisible absolute z-10 inline-block rounded-lg bg-gray-900 px-3 py-2 text-sm font-medium text-white opacity-0 shadow-sm transition-opacity duration-300 dark:bg-gray-700">
-                Remove item
-                <div class="tooltip-arrow" data-popper-arrow></div>
-              </div>
-            </div>
-          </div>
-
-          <div class="grid grid-cols-2">
-            <div>
-              <a href="#"
-                 class="truncate text-sm font-semibold leading-none text-gray-900 dark:text-white hover:underline">Apple
-                Watch SE</a>
-              <p class="mt-0.5 truncate text-sm font-normal text-gray-500 dark:text-gray-400">$598</p>
-            </div>
-
-            <div class="flex items-center justify-end gap-6">
-              <p class="text-sm font-normal leading-none text-gray-500 dark:text-gray-400">Qty: 2</p>
-
-              <button data-tooltip-target="tooltipRemoveItem3b" type="button"
-                      class="text-red-600 hover:text-red-700 dark:text-red-500 dark:hover:text-red-600">
-                <span class="sr-only"> Remove </span>
-                <svg class="h-4 w-4" aria-hidden="true" xmlns="http://www.w3.org/2000/svg" fill="currentColor"
-                     viewBox="0 0 24 24">
-                  <path fill-rule="evenodd"
-                        d="M2 12a10 10 0 1 1 20 0 10 10 0 0 1-20 0Zm7.7-3.7a1 1 0 0 0-1.4 1.4l2.3 2.3-2.3 2.3a1 1 0 1 0 1.4 1.4l2.3-2.3 2.3 2.3a1 1 0 0 0 1.4-1.4L13.4 12l2.3-2.3a1 1 0 0 0-1.4-1.4L12 10.6 9.7 8.3Z"
-                        clip-rule="evenodd"/>
-                </svg>
-              </button>
-              <div id="tooltipRemoveItem3b" role="tooltip"
-                   class="tooltip invisible absolute z-10 inline-block rounded-lg bg-gray-900 px-3 py-2 text-sm font-medium text-white opacity-0 shadow-sm transition-opacity duration-300 dark:bg-gray-700">
-                Remove item
-                <div class="tooltip-arrow" data-popper-arrow></div>
-              </div>
-            </div>
-          </div>
-
-          <div class="grid grid-cols-2">
-            <div>
-              <a href="#"
-                 class="truncate text-sm font-semibold leading-none text-gray-900 dark:text-white hover:underline">Sony
-                Playstation 5</a>
-              <p class="mt-0.5 truncate text-sm font-normal text-gray-500 dark:text-gray-400">$799</p>
-            </div>
-
-            <div class="flex items-center justify-end gap-6">
-              <p class="text-sm font-normal leading-none text-gray-500 dark:text-gray-400">Qty: 1</p>
-
-              <button data-tooltip-target="tooltipRemoveItem4b" type="button"
-                      class="text-red-600 hover:text-red-700 dark:text-red-500 dark:hover:text-red-600">
-                <span class="sr-only"> Remove </span>
-                <svg class="h-4 w-4" aria-hidden="true" xmlns="http://www.w3.org/2000/svg" fill="currentColor"
-                     viewBox="0 0 24 24">
-                  <path fill-rule="evenodd"
-                        d="M2 12a10 10 0 1 1 20 0 10 10 0 0 1-20 0Zm7.7-3.7a1 1 0 0 0-1.4 1.4l2.3 2.3-2.3 2.3a1 1 0 1 0 1.4 1.4l2.3-2.3 2.3 2.3a1 1 0 0 0 1.4-1.4L13.4 12l2.3-2.3a1 1 0 0 0-1.4-1.4L12 10.6 9.7 8.3Z"
-                        clip-rule="evenodd"/>
-                </svg>
-              </button>
-              <div id="tooltipRemoveItem4b" role="tooltip"
-                   class="tooltip invisible absolute z-10 inline-block rounded-lg bg-gray-900 px-3 py-2 text-sm font-medium text-white opacity-0 shadow-sm transition-opacity duration-300 dark:bg-gray-700">
-                Remove item
-                <div class="tooltip-arrow" data-popper-arrow></div>
-              </div>
-            </div>
-          </div>
-
-          <div class="grid grid-cols-2">
-            <div>
-              <a href="#"
-                 class="truncate text-sm font-semibold leading-none text-gray-900 dark:text-white hover:underline">Apple
-                iMac 20"</a>
-              <p class="mt-0.5 truncate text-sm font-normal text-gray-500 dark:text-gray-400">$8,997</p>
-            </div>
-
-            <div class="flex items-center justify-end gap-6">
-              <p class="text-sm font-normal leading-none text-gray-500 dark:text-gray-400">Qty: 3</p>
-
-              <button data-tooltip-target="tooltipRemoveItem5b" type="button"
-                      class="text-red-600 hover:text-red-700 dark:text-red-500 dark:hover:text-red-600">
-                <span class="sr-only"> Remove </span>
-                <svg class="h-4 w-4" aria-hidden="true" xmlns="http://www.w3.org/2000/svg" fill="currentColor"
-                     viewBox="0 0 24 24">
-                  <path fill-rule="evenodd"
-                        d="M2 12a10 10 0 1 1 20 0 10 10 0 0 1-20 0Zm7.7-3.7a1 1 0 0 0-1.4 1.4l2.3 2.3-2.3 2.3a1 1 0 1 0 1.4 1.4l2.3-2.3 2.3 2.3a1 1 0 0 0 1.4-1.4L13.4 12l2.3-2.3a1 1 0 0 0-1.4-1.4L12 10.6 9.7 8.3Z"
-                        clip-rule="evenodd"/>
-                </svg>
-              </button>
-              <div id="tooltipRemoveItem5b" role="tooltip"
-                   class="tooltip invisible absolute z-10 inline-block rounded-lg bg-gray-900 px-3 py-2 text-sm font-medium text-white opacity-0 shadow-sm transition-opacity duration-300 dark:bg-gray-700">
-                Remove item
-                <div class="tooltip-arrow" data-popper-arrow></div>
-              </div>
-            </div>
-          </div>
-
-          <a href="#" title=""
-             class="mb-2 me-2 inline-flex w-full items-center justify-center rounded-lg bg-primary-700 px-5 py-2.5 text-sm font-medium text-white hover:bg-primary-800 focus:outline-none focus:ring-4 focus:ring-primary-300 dark:bg-primary-600 dark:hover:bg-primary-700 dark:focus:ring-primary-800"
-             role="button"> Proceed to Checkout </a>
+        {% include 'components/cart/cart_dropdown.html' %}
         </div>
 
+        {% if request.user.is_authenticated %}
         <!-- Show/Hide user account dropdown -->
         <button hx-on:click="htmx.toggleClass(htmx.find('#userDropdown1'), 'hidden'); htmx.addClass(htmx.find('#myCartDropdown1'), 'hidden')"
                 id="userDropdownButton1"
@@ -229,7 +83,7 @@
             <path stroke="currentColor" stroke-width="2"
                   d="M7 17v1a1 1 0 0 0 1 1h8a1 1 0 0 0 1-1v-1a3 3 0 0 0-3-3h-4a3 3 0 0 0-3 3Zm8-9a3 3 0 1 1-6 0 3 3 0 0 1 6 0Z"/>
           </svg>
-          {% translate 'Mi cuenta' %}
+            {% translate 'Mi cuenta' %}
           <svg class="w-4 h-4 text-gray-900 dark:text-white ms-1" aria-hidden="true" xmlns="http://www.w3.org/2000/svg"
                width="24" height="24" fill="none" viewBox="0 0 24 24">
             <path stroke="currentColor" stroke-linecap="round" stroke-linejoin="round" stroke-width="2"
@@ -239,53 +93,12 @@
 
         <!-- User account dropdown -->
         <div id="userDropdown1"
-             class="absolute top-10 hidden z-10 w-56 divide-y divide-gray-100 overflow-hidden overflow-y-auto rounded-lg bg-white antialiased shadow dark:divide-gray-600 dark:bg-gray-700">
-          <ul class="p-2 text-start text-sm font-medium text-gray-900 dark:text-white">
-            <li>
-              <a href="#" title=""
-                 class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
-                My Account
-              </a>
-            </li>
-            <li>
-              <a href="#" title=""
-                 class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
-                My Orders
-              </a>
-            </li>
-            <li>
-              <a href="#" title=""
-                 class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
-                Settings
-              </a>
-            </li>
-            <li>
-              <a href="#" title=""
-                 class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
-                Favourites
-              </a>
-            </li>
-            <li>
-              <a href="#" title=""
-                 class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
-                Delivery Addresses
-              </a>
-            </li>
-            <li>
-              <a href="#" title=""
-                 class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
-                Billing Data
-              </a>
-            </li>
-          </ul>
-
-          <div class="p-2 text-sm font-medium text-gray-900 dark:text-white">
-            <a href="#" title=""
-               class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
-              Sign Out </a>
-          </div>
+             class="absolute top-10 hidden z-10 w-56 divide-y divide-gray-100 overflow-hidden overflow-y-auto rounded-lg bg-white antialiased shadow dark:divide-gray-600 dark:bg-gray-700 right-0">
+          {% include 'components/users/user_dropdown.html' %}
         </div>
-
+        {% else %}
+          <a href="{% url 'users:login' %}" class="inline-flex items-center rounded-lg justify-center p-2 hover:bg-gray-100 dark:hover:bg-gray-700 text-sm font-medium leading-none text-gray-900 dark:text-white">Inicia sesión</a>
+        {% endif %}
         <!-- Show/Hide menu items -->
         <button hx-on:click="htmx.toggleClass(htmx.find('#ecommerce-navbar-menu-1'), 'hidden')" type="button"
                 data-collapse-toggle="ecommerce-navbar-menu-1" aria-controls="ecommerce-navbar-menu-1"
@@ -306,22 +119,22 @@
          class="bg-gray-50 dark:bg-gray-700 dark:border-gray-600 border border-gray-200 rounded-lg py-3 hidden px-4 mt-4">
       <ul class="text-gray-900 dark:text-white text-sm font-medium dark:text-white space-y-3">
         <li>
-          <a href="#" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Inicio' %}</a>
+          <a href="{% url 'shop:index' %}" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Inicio' %}</a>
         </li>
         <li>
-          <a href="#" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Más vendidos' %}</a>
+          <a href="{% url 'shop:index' %}" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Más vendidos' %}</a>
         </li>
         <li>
-          <a href="#" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Ideas de regalo' %}</a>
+          <a href="{% url 'shop:index' %}" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Ideas de regalo' %}</a>
         </li>
         <li>
-          <a href="#" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Juegos' %}</a>
+          <a href="{% url 'shop:index' %}" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Juegos' %}</a>
         </li>
         <li>
-          <a href="#" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Electrónica' %}</a>
+          <a href="{% url 'shop:index' %}" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Electrónica' %}</a>
         </li>
         <li>
-          <a href="#" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Ofertas del día' %}</a>
+          <a href="{% url 'shop:index' %}" class="hover:text-primary-700 dark:hover:text-primary-500">{% translate 'Ofertas del día' %}</a>
         </li>
       </ul>
     </div>
diff --git a/shop/templates/components/users/user_dropdown.html b/shop/templates/components/users/user_dropdown.html
new file mode 100644
index 0000000..fff6665
--- /dev/null
+++ b/shop/templates/components/users/user_dropdown.html
@@ -0,0 +1,46 @@
+{% load i18n %}
+<ul class="p-2 text-start text-sm font-medium text-gray-900 dark:text-white">
+  <li>
+    <a href="#" title=""
+       class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
+      {% translate 'Mi cuenta' %}
+    </a>
+  </li>
+  <li>
+    <a href="#" title=""
+       class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
+      {% translate 'Mis pedidos' %}
+    </a>
+  </li>
+  <li>
+    <a href="#" title=""
+       class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
+      {% translate 'Ajustes' %}
+    </a>
+  </li>
+  <li>
+    <a href="#" title=""
+       class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
+      {% translate 'Lista de deseados' %}
+    </a>
+  </li>
+  <li>
+    <a href="#" title=""
+       class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
+      {% translate 'Direcciones de envío' %}
+    </a>
+  </li>
+  <li>
+    <a href="#" title=""
+       class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
+      {% translate 'Métodos de pago' %}
+    </a>
+  </li>
+</ul>
+
+<div class="p-2 text-sm font-medium text-gray-900 dark:text-white">
+  <a href="{% url 'users:logout' %}" title=""
+     class="inline-flex w-full items-center gap-2 rounded-md px-3 py-2 text-sm hover:bg-gray-100 dark:hover:bg-gray-600">
+    {% translate 'Cerrar sesión' %}
+  </a>
+</div>
diff --git a/shop/templates/shop/product_detail.html b/shop/templates/shop/product_detail.html
index ee7768c..baa02ee 100644
--- a/shop/templates/shop/product_detail.html
+++ b/shop/templates/shop/product_detail.html
@@ -4,13 +4,14 @@
 {% load i18n %}
 
 {% block main %}
+  {% include 'components/generic/navbar.html' %}
 
-<section class="min-w-full py-8 bg-white md:py-16 dark:bg-gray-900 antialiased">
+  <section class="min-w-full py-8 bg-white md:py-16 dark:bg-gray-900 antialiased">
     <div class="max-w-screen-xl px-4 mx-auto 2xl:px-0">
       <div class="lg:grid lg:grid-cols-2 lg:gap-8 xl:gap-16">
         <div class="shrink-0 max-w-md lg:max-w-lg mx-auto">
-          <img class="w-full dark:hidden" src="{{ product.images.all|first }}" alt="" />
-          <img class="w-full hidden dark:block" src="{{ product.images.all|first }}" alt="" />
+          <img class="w-full dark:hidden" src="{{ product.images.all|first }}" alt=""/>
+          <img class="w-full hidden dark:block" src="{{ product.images.all|first }}" alt=""/>
         </div>
 
         <div class="mt-6 sm:mt-8 lg:mt-0">
@@ -163,7 +164,7 @@
             </a>
           </div>
 
-          <hr class="my-6 md:my-8 border-gray-200 dark:border-gray-800" />
+          <hr class="my-6 md:my-8 border-gray-200 dark:border-gray-800"/>
 
           <p class="mb-6 text-gray-500 dark:text-gray-400">
             {{ product.description }}
diff --git a/theme/static/theme/js/theme.js b/theme/static/theme/js/theme.js
index ab17e5c..d76bb37 100644
--- a/theme/static/theme/js/theme.js
+++ b/theme/static/theme/js/theme.js
@@ -10,9 +10,9 @@ if (currentTheme === 'dark') {
   document.getElementById('theme-toggle-dark-icon').classList.remove('hidden')
 }
 
+
 function switchTheme() {
   const theme = localStorage.getItem('theme')
-  console.log("Switching theme from ", theme)
 
   if (theme === 'dark') {
     document.documentElement.classList.remove('dark')
@@ -25,4 +25,4 @@ function switchTheme() {
     document.getElementById('theme-toggle-light-icon').classList.remove('hidden')
     document.getElementById('theme-toggle-dark-icon').classList.add('hidden')
   }
-}
\ No newline at end of file
+}