Merge pull request #1618 from Grandi0z/cypht-fix-mobil-logo-and-them-path

fix(frontend): dynamically set logo based on theme in navigation bar

Author: marclaporte

modules/core/navigation/navbar.js

@@ -21,10 +21,18 @@ $(() => {
         </div>
         `
 
+        // Get the appropriate logo based on current theme
+        const getLogoPath = () => {
+            const isDarkTheme = getComputedStyle(document.documentElement)
+                .getPropertyValue('--bs-secondary-bg') === '#333';
+            const logoFile = isDarkTheme ? 'logo.svg' : 'logo_dark.svg';
+            return `${hm_web_root_path()}modules/core/assets/images/${logoFile}`;
+        };
+
         const navHeader = `
         <div class="nav-header">
             <a href="?page=home" class="menu_home">
-                <img class="app-logo" src="modules/core/assets/images/logo_dark.svg">
+                <img class="app-logo" src="${getLogoPath()}">
             </a>
             <div class="menu-toggle fw-bold cursor-pointer close-toggle" style="display: none;">
                 <i class="bi bi-x-lg fs-5 fw-bold"></i>

modules/core/site.css

@@ -1454,6 +1454,7 @@ div.unseen,
   justify-content: flex-end;
   padding: 0;
   border-bottom: 1px solid var(--bs-secondary-bg);
+  background-color: var(--bs-body-bg);
 }
 
 .mobile .nav-header a {