@o2vend/theme-cli 1.0.37 → 1.0.38

package/test-theme/sections/header-fallback.liquid

@@ -216,13 +216,13 @@
 
 <style>
   .site-header__fallback {
-    background: var(--color-background, {{ settings.color_background }});
+    background: var(--color-background, {{ settings.color_background | default: '#ffffff' }});
     border-bottom: 1px solid var(--color-border, rgba(15, 23, 42, 0.08));
     position: relative;
     overflow: visible;
   }
   .site-header__fallback-inner {
-    max-width: var(--container-width, {{ settings.container_width }}px);
+    max-width: var(--container-width, {{ settings.container_width | default: 1200 }}px);
     margin: 0 auto;
     padding: var(--spacing-element, 0.85rem) var(--spacing-component, 1.25rem);
     display: flex;
@@ -241,7 +241,7 @@
   .site-header__actions a,
   .site-header__actions button {
     text-decoration: none;
-    color: var(--color-text, {{ settings.color_text }});
+    color: var(--color-text, {{ settings.color_text | default: '#000000' }});
     font-weight: var(--font-weight-normal, 400);
     background: none;
     border: none;
@@ -276,8 +276,8 @@
     min-width: 18px;
     height: 18px;
     padding: 0 var(--spacing-element, 5px);
-    background: var(--color-text, {{ settings.color_text }});
-    color: var(--color-background, {{ settings.color_background }});
+    background: var(--color-text, {{ settings.color_text | default: '#000000' }});
+    color: var(--color-background, {{ settings.color_background | default: '#ffffff' }});
     border-radius: var(--border-radius-medium, 9px);
     font-size: var(--text-xs, 0.75rem);
     font-weight: var(--font-weight-bold, 600);
@@ -315,7 +315,7 @@
   .site-header__logo-text {
     font-size: var(--text-xl, 1.25rem);
     font-weight: var(--font-weight-bold, 600);
-    color: var(--color-text, {{ settings.color_text }});
+    color: var(--color-text, {{ settings.color_text | default: '#000000' }});
   }
   
   /* Desktop Search Container */
@@ -341,7 +341,7 @@
   .site-header__search-icon {
     position: absolute;
     left: 1rem;
-    color: var(--color-text, {{ settings.color_text }});
+    color: var(--color-text, {{ settings.color_text | default: '#000000' }});
     opacity: 0.5;
     pointer-events: none;
     z-index: 1;
@@ -353,8 +353,8 @@
     border: 1px solid rgba(0, 0, 0, 0.15);
     border-radius: 8px;
     font-size: 1rem;
-    color: var(--color-text, {{ settings.color_text }});
-    background: var(--color-background, {{ settings.color_background }});
+    color: var(--color-text, {{ settings.color_text | default: '#000000' }});
+    background: var(--color-background, {{ settings.color_background | default: '#ffffff' }});
     transition: border-color 0.2s ease, box-shadow 0.2s ease;
     font-family: -apple-system, "system-ui", "Segoe UI", Roboto, Helvetica, Arial, sans-serif;
   }
@@ -379,7 +379,7 @@
     justify-content: center;
     border: none;
     background: transparent;
-    color: var(--color-text, {{ settings.color_text }});
+    color: var(--color-text, {{ settings.color_text | default: '#000000' }});
     opacity: 0.5;
     cursor: pointer;
     padding: 0;
@@ -430,7 +430,7 @@
     justify-content: center;
     gap: 0.75rem;
     padding: 2rem;
-    color: var(--color-text, {{ settings.color_text }});
+    color: var(--color-text, {{ settings.color_text | default: '#000000' }});
     opacity: 0.6;
   }
 
@@ -457,7 +457,7 @@
     gap: 1rem;
     padding: 0.75rem 1rem;
     text-decoration: none;
-    color: var(--color-text, {{ settings.color_text }});
+    color: var(--color-text, {{ settings.color_text | default: '#000000' }});
     transition: background-color 0.2s ease;
     cursor: pointer;
   }
@@ -484,7 +484,7 @@
     font-weight: 600;
     font-size: 0.9375rem;
     margin: 0 0 0.25rem 0;
-    color: var(--color-text, {{ settings.color_text }});
+    color: var(--color-text, {{ settings.color_text | default: '#000000' }});
     white-space: nowrap;
     overflow: hidden;
     text-overflow: ellipsis;
@@ -560,7 +560,7 @@
     gap: var(--spacing-element, 0.375rem);
     padding: var(--spacing-element, 0.75rem) var(--spacing-element, 1rem);
     text-decoration: none;
-    color: var(--color-text, {{ settings.color_text }});
+    color: var(--color-text, {{ settings.color_text | default: '#000000' }});
     font-weight: var(--font-weight-normal, 400);
     font-size: var(--text-xl, 2rem);
     letter-spacing: var(--letter-spacing-heading, 0);
@@ -592,7 +592,7 @@
     
     .mobile-header-wrapper {
       display: block;
-      background: var(--color-background, {{ settings.color_background }});
+      background: var(--color-background, {{ settings.color_background | default: '#ffffff' }});
       border-bottom: 1px solid var(--color-border, rgba(15, 23, 42, 0.08));
     }
     
@@ -614,7 +614,7 @@
       border: none;
       cursor: pointer;
       padding: 0;
-      color: var(--color-text, {{ settings.color_text }});
+      color: var(--color-text, {{ settings.color_text | default: '#000000' }});
       border-radius: var(--border-radius-small, 0);
       transition: background-color var(--transition-fast, 0.2s) var(--ease-out, ease);
       flex-shrink: 0;
@@ -676,7 +676,7 @@
     .mobile-header__logo-text {
       font-size: var(--text-lg, 1.125rem);
       font-weight: var(--font-weight-bold, 600);
-      color: var(--color-text, {{ settings.color_text }});
+      color: var(--color-text, {{ settings.color_text | default: '#000000' }});
       white-space: nowrap;
       overflow: hidden;
       text-overflow: ellipsis;
@@ -699,7 +699,7 @@
       border: none;
       cursor: pointer;
       padding: 0;
-      color: var(--color-text, {{ settings.color_text }});
+      color: var(--color-text, {{ settings.color_text | default: '#000000' }});
       border-radius: var(--border-radius-small, 0);
       transition: background-color var(--transition-fast, 0.2s) var(--ease-out, ease);
     }
@@ -723,8 +723,8 @@
       min-width: 18px;
       height: 18px;
       padding: 0 var(--spacing-element, 5px);
-      background: var(--color-text, {{ settings.color_text }});
-      color: var(--color-background, {{ settings.color_background }});
+      background: var(--color-text, {{ settings.color_text | default: '#000000' }});
+      color: var(--color-background, {{ settings.color_background | default: '#ffffff' }});
       border-radius: var(--border-radius-medium, 9px);
       font-size: var(--text-xs, 0.75rem);
       font-weight: var(--font-weight-bold, 600);
@@ -770,8 +770,8 @@
       border-radius: var(--border-radius-medium, 8px);
       font-size: var(--text-base, 0.875rem);
       font-family: var(--font-primary, var(--font-body));
-      color: var(--color-text, {{ settings.color_text }});
-      background-color: var(--color-background, {{ settings.color_background }});
+      color: var(--color-text, {{ settings.color_text | default: '#000000' }});
+      background-color: var(--color-background, {{ settings.color_background | default: '#ffffff' }});
       min-height: 44px;
       transition: border-color var(--transition-fast, 0.2s) var(--ease-out, ease);
     }
@@ -813,7 +813,7 @@
       max-width: 85vw;
       height: 100vh;
       max-height: 100vh;
-      background-color: var(--color-background, {{ settings.color_background }});
+      background-color: var(--color-background, {{ settings.color_background | default: '#ffffff' }});
       padding: var(--spacing-component, 1.5rem);
       overflow-y: auto;
       overflow-x: hidden;
@@ -843,7 +843,7 @@
     .mobile-nav-drawer-title {
       font-size: var(--text-lg, 1.125rem);
       font-weight: var(--font-weight-bold, 600);
-      color: var(--color-text, {{ settings.color_text }});
+      color: var(--color-text, {{ settings.color_text | default: '#000000' }});
       margin: 0;
     }
     
@@ -855,7 +855,7 @@
       height: 40px;
       background: none;
       border: none;
-      color: var(--color-text, {{ settings.color_text }});
+      color: var(--color-text, {{ settings.color_text | default: '#000000' }});
       cursor: pointer;
       border-radius: var(--border-radius-small, 0);
       transition: background-color var(--transition-fast, 0.2s) var(--ease-out, ease);
@@ -885,7 +885,7 @@
     .mobile-nav-drawer-link {
       display: block;
       padding: var(--spacing-component, 1rem) 0;
-      color: var(--color-text, {{ settings.color_text }});
+      color: var(--color-text, {{ settings.color_text | default: '#000000' }});
       text-decoration: none;
       font-weight: var(--font-weight-medium, 500);
       font-size: var(--text-base, 1rem);
@@ -935,7 +935,7 @@
     .mobile-nav-drawer-nav .site-header__menu-link {
       display: block;
       padding: var(--spacing-component, 1rem) 0;
-      color: var(--color-text, {{ settings.color_text }});
+      color: var(--color-text, {{ settings.color_text | default: '#000000' }});
       text-decoration: none;
       font-weight: var(--font-weight-medium, 500);
       font-size: var(--text-base, 1rem);
@@ -1267,7 +1267,7 @@
 </div>
 
 <script>
-  document.addEventListener('DOMContentLoaded', function() {
+  document.addEventListener('DOMContentLoaded', () => {
     const menuToggle = document.querySelector('[data-mobile-menu-toggle]');
     const menuClose = document.querySelector('[data-mobile-menu-close]');
     const drawer = document.getElementById('mobile-nav-drawer');
@@ -1294,7 +1294,7 @@
     }
     
     if (menuToggle) {
-      menuToggle.addEventListener('click', function(e) {
+      menuToggle.addEventListener('click', (e) => {
         e.preventDefault();
         e.stopPropagation();
         if (drawer && drawer.classList.contains('active')) {
@@ -1306,7 +1306,7 @@
     }
     
     if (menuClose) {
-      menuClose.addEventListener('click', function(e) {
+      menuClose.addEventListener('click', (e) => {
         e.preventDefault();
         e.stopPropagation();
         closeMenu();
@@ -1314,13 +1314,13 @@
     }
     
     if (drawer) {
-      drawer.addEventListener('click', function(e) {
+      drawer.addEventListener('click', (e) => {
         if (e.target === drawer) {
           closeMenu();
         }
       });
       
-      document.addEventListener('keydown', function(e) {
+      document.addEventListener('keydown', (e) => {
         if (e.key === 'Escape' && drawer.classList.contains('active')) {
           closeMenu();
         }
@@ -1481,21 +1481,21 @@
     }
     
     if (searchToggle) {
-      searchToggle.addEventListener('click', function(e) {
+      searchToggle.addEventListener('click', (e) => {
         e.preventDefault();
         openSearch();
       });
     }
     
     if (searchClose) {
-      searchClose.addEventListener('click', function(e) {
+      searchClose.addEventListener('click', (e) => {
         e.preventDefault();
         closeSearch();
       });
     }
     
     if (searchOverlay) {
-      searchOverlay.addEventListener('click', function(e) {
+      searchOverlay.addEventListener('click', (e) => {
         if (e.target === searchOverlay) {
           closeSearch();
         }
@@ -1503,7 +1503,7 @@
     }
     
     if (searchInput) {
-      searchInput.addEventListener('input', function(e) {
+      searchInput.addEventListener('input', (e) => {
         const query = e.target.value.trim();
         clearTimeout(searchTimeout);
         searchTimeout = setTimeout(() => {
@@ -1511,7 +1511,7 @@
         }, 300);
       });
       
-      searchInput.addEventListener('keydown', function(e) {
+      searchInput.addEventListener('keydown', (e) => {
         if (e.key === 'Escape') {
           closeSearch();
         }
@@ -1519,7 +1519,7 @@
     }
     
     if (mobileSearchInput) {
-      mobileSearchInput.addEventListener('input', function(e) {
+      mobileSearchInput.addEventListener('input', (e) => {
         const query = e.target.value.trim();
         if (query.length > 0 && searchOverlay && !searchOverlay.classList.contains('active')) {
           openSearch();
@@ -1540,7 +1540,7 @@
     
     if (searchForms && searchForms.length > 0) {
       searchForms.forEach(form => {
-        form.addEventListener('submit', function(e) {
+        form.addEventListener('submit', (e) => {
           e.preventDefault();
           const input = form.querySelector('[data-search-input], input[type="search"], input[name="q"]');
           if (input && input.value.trim()) {
@@ -1554,7 +1554,7 @@
       });
     }
     
-    document.addEventListener('keydown', function(e) {
+    document.addEventListener('keydown', (e) => {
       if (e.key === 'Escape' && searchOverlay && searchOverlay.classList.contains('active')) {
         closeSearch();
       }
@@ -1577,7 +1577,7 @@
 
       // Debounce function
       function debounce(func, wait) {
-        return function(...args) {
+        return (...args) => {
           clearTimeout(searchTimeout);
           searchTimeout = setTimeout(() => func.apply(this, args), wait);
         };

package/test-theme/sections/header.liquid

@@ -24,7 +24,7 @@
 {% liquid
   assign widget_settings = header_menu_widget.settings | default: section.settings
   assign widget_data = header_menu_widget.data | default: null
-  
+
   assign content_data = widget_data.content | default: null
   if content_data and content_data.size > 0
     assign header_config = content_data.first
@@ -33,7 +33,9 @@
   endif
 
   assign logo_url = nil
-  if widget_settings.logo and widget_settings.logo != blank
+  if settings.logo and settings.logo != blank
+    assign logo_url = settings.logo
+  elsif widget_settings.logo and widget_settings.logo != blank
     assign logo_url = widget_settings.logo
   elsif shop.logo and shop.logo != blank
     assign logo_url = shop.logo
@@ -59,13 +61,7 @@
     assign show_cart = true
   endif
 
-  assign show_account = widget_settings.showAccount
-  if show_account == nil and header_config and header_config.ShowAccount != nil
-    assign show_account = header_config.ShowAccount
-  endif
-  if show_account == nil
-    assign show_account = true
-  endif
+  assign show_account = true
 %}
 
 <header class="site-header" data-section-id="{{ section.id }}">
@@ -81,7 +77,7 @@
                data-widget-type="{{ widget.type }}"
                data-widget-position="{{ widget.Position | default: widget.position | default: forloop.index }}">
             {% if widget and widget.template_path %}
-              {% render widget.template_path, widget: widget, settings: settings %}
+              {% render widget.template_path, widget: widget, settings: settings, shop: shop %}
             {% else %}
               <div class="widget-error" data-widget-error>
                 <p>Widget template not found: {{ widget.template_path | default: 'unknown' }}</p>
@@ -117,7 +113,7 @@
         <div class="site-header__search-container" data-search-container>
           <form class="site-header__search-form" action="/search" method="get" role="search">
             <div class="site-header__search-input-wrapper">
-              <svg class="site-header__search-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5">
+              <svg class="site-header__search-icon" width="20" height="20" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5">
                 <circle cx="11" cy="11" r="8"></circle>
                 <path d="m21 21-4.35-4.35"></path>
               </svg>
@@ -130,7 +126,7 @@
                 autocomplete="off"
                 data-search-input>
               <button type="button" class="site-header__search-clear" aria-label="Clear search" data-search-clear style="display: none;">
-                <svg class="site-header__search-clear-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2">
+                <svg class="site-header__search-clear-icon" width="16" height="16" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2">
                   <line x1="18" y1="6" x2="6" y2="18"></line>
                   <line x1="6" y1="6" x2="18" y2="18"></line>
                 </svg>
@@ -152,37 +148,29 @@
       </div>
 
       <div class="site-header__actions">
-        {% if show_account %}
-          {% comment %}Debug: customer={{ customer | json }}{% endcomment %}
+        {% if show_account == true %}
+          <a href="/account" class="site-header__action site-header__action--account" aria-label="Account">
+            <svg class="site-header__account-icon" width="20" height="20" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5">
+              <path d="M20 21v-2a4 4 0 0 0-4-4H8a4 4 0 0 0-4 4v2"></path>
+              <circle cx="12" cy="7" r="4"></circle>
+            </svg>
+          </a>
           {% if customer and customer.isAuthenticated %}
-            <a href="/account" class="site-header__action site-header__action--account" aria-label="Account">
-              <svg class="site-header__account-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5">
-                <path d="M20 21v-2a4 4 0 0 0-4-4H8a4 4 0 0 0-4 4v2"></path>
-                <circle cx="12" cy="7" r="4"></circle>
-              </svg>
-            </a>
             <form method="post" action="/webstoreapi/customer/logout" class="site-header__logout-form">
               <button type="submit" class="site-header__action site-header__action--logout" aria-label="Logout">
-                <svg class="site-header__logout-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5">
+                <svg class="site-header__logout-icon" width="20" height="20" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5">
                   <path d="M9 21H5a2 2 0 0 1-2-2V5a2 2 0 0 1 2-2h4"></path>
                   <polyline points="16 17 21 12 16 7"></polyline>
                   <line x1="21" y1="12" x2="9" y2="12"></line>
                 </svg>
               </button>
             </form>
-          {% else %}
-            <a href="#" class="site-header__action site-header__action--login" aria-label="Login" data-login-modal-trigger>
-              <svg class="site-header__login-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5">
-                <circle cx="12" cy="8" r="4"></circle>
-                <path d="M6 20a6 6 0 0 1 12 0"></path>
-              </svg>
-            </a>
           {% endif %}
         {% endif %}
 
         {% if show_cart %}
           <button class="site-header__action site-header__action--cart" aria-label="Shopping cart" data-cart-toggle type="button">
-            <svg class="site-header__cart-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5">
+            <svg class="site-header__cart-icon" width="20" height="20" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5">
               <circle cx="9" cy="21" r="1"></circle>
               <circle cx="20" cy="21" r="1"></circle>
               <path d="M1 1h4l2.68 13.39a2 2 0 0 0 2 1.61h9.72a2 2 0 0 0 2-1.61L23 6H6"></path>
@@ -200,7 +188,7 @@
       <div class="site-header__mobile-search">
         <form class="site-header__mobile-search-form" action="/search" method="get" role="search">
           <div class="site-header__mobile-search-input-wrapper">
-            <svg class="site-header__mobile-search-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2">
+            <svg class="site-header__mobile-search-icon" width="20" height="20" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2">
               <circle cx="11" cy="11" r="8"></circle>
               <path d="m21 21-4.35-4.35"></path>
             </svg>
@@ -235,7 +223,7 @@
              data-widget-id="{{ header_menu_widget.id }}" 
              data-widget-type="{{ header_menu_widget.type }}">
           {% if header_menu_widget and header_menu_widget.template_path %}
-            {% render header_menu_widget.template_path, widget: header_menu_widget, settings: settings %}
+            {% render header_menu_widget.template_path, widget: header_menu_widget, settings: settings, shop: shop %}
           {% else %}
             <div class="widget-error">
               <p>Widget template not found</p>
@@ -791,7 +779,7 @@
 </style>
 
 <script>
-  document.addEventListener('DOMContentLoaded', function() {
+  document.addEventListener('DOMContentLoaded', () => {
     // Desktop Search
     const searchContainer = document.querySelector('[data-search-container]');
     if (searchContainer) {
@@ -805,8 +793,8 @@
       let searchTimeout;
 
       if (searchInput) {
-        searchInput.addEventListener('input', function() {
-          const query = this.value.trim();
+        searchInput.addEventListener('input', () => {
+          const query = searchInput.value.trim();
           
           if (searchClear) {
             searchClear.style.display = query ? 'flex' : 'none';
@@ -822,7 +810,7 @@
       }
 
       if (searchClear) {
-        searchClear.addEventListener('click', function() {
+        searchClear.addEventListener('click', () => {
           searchInput.value = '';
           searchClear.style.display = 'none';
           searchDropdown.style.display = 'none';
@@ -884,7 +872,7 @@
         return div.innerHTML;
       }
 
-      document.addEventListener('click', function(e) {
+      document.addEventListener('click', (e) => {
         if (searchContainer && !searchContainer.contains(e.target)) {
           searchDropdown.style.display = 'none';
         }
@@ -904,8 +892,8 @@
       let mobileSearchTimeout;
 
       if (mobileSearchInput) {
-        mobileSearchInput.addEventListener('input', function() {
-          const query = this.value.trim();
+        mobileSearchInput.addEventListener('input', () => {
+          const query = mobileSearchInput.value.trim();
 
           if (query.length >= 2) {
             clearTimeout(mobileSearchTimeout);
@@ -917,7 +905,7 @@
       }
 
       if (mobileSearchForm) {
-        mobileSearchForm.addEventListener('submit', function(e) {
+        mobileSearchForm.addEventListener('submit', (e) => {
           const query = mobileSearchInput.value.trim();
           if (!query || query.length < 2) {
             e.preventDefault();
@@ -979,7 +967,7 @@
         return div.innerHTML;
       }
 
-      document.addEventListener('click', function(e) {
+      document.addEventListener('click', (e) => {
         if (mobileSearch && !mobileSearch.contains(e.target)) {
           mobileSearchDropdown.style.display = 'none';
         }
@@ -1006,6 +994,11 @@
       const logoutForm = document.querySelector('.site-header__logout-form');
       const loginIcon = document.querySelector('.site-header__action--login');
 
+      // Keep account icon visible in all states; unauthenticated clicks open login modal.
+      if (accountIcon) {
+        accountIcon.style.display = '';
+      }
+
       if (authenticated) {
         // User is authenticated - show account and logout icons
         // Create account icon if it doesn't exist
@@ -1015,7 +1008,7 @@
           accountLink.href = '/account';
           accountLink.className = 'site-header__action site-header__action--account';
           accountLink.setAttribute('aria-label', 'Account');
-          accountLink.innerHTML = '<svg class="site-header__account-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5"><path d="M20 21v-2a4 4 0 0 0-4-4H8a4 4 0 0 0-4 4v2"></path><circle cx="12" cy="7" r="4"></circle></svg>';
+          accountLink.innerHTML = '<svg class="site-header__account-icon" width="20" height="20" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5"><path d="M20 21v-2a4 4 0 0 0-4-4H8a4 4 0 0 0-4 4v2"></path><circle cx="12" cy="7" r="4"></circle></svg>';
           actionsContainer.insertBefore(accountLink, actionsContainer.firstChild);
         }
         
@@ -1026,7 +1019,7 @@
           logoutFormEl.method = 'post';
           logoutFormEl.action = '/webstoreapi/customer/logout';
           logoutFormEl.className = 'site-header__logout-form';
-          logoutFormEl.innerHTML = '<button type="submit" class="site-header__action site-header__action--logout" aria-label="Logout"><svg class="site-header__logout-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5"><path d="M9 21H5a2 2 0 0 1-2-2V5a2 2 0 0 1 2-2h4"></path><polyline points="16 17 21 12 16 7"></polyline><line x1="21" y1="12" x2="9" y2="12"></line></svg></button>';
+          logoutFormEl.innerHTML = '<button type="submit" class="site-header__action site-header__action--logout" aria-label="Logout"><svg class="site-header__logout-icon" width="20" height="20" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5"><path d="M9 21H5a2 2 0 0 1-2-2V5a2 2 0 0 1 2-2h4"></path><polyline points="16 17 21 12 16 7"></polyline><line x1="21" y1="12" x2="9" y2="12"></line></svg></button>';
           const accountIconEl = document.querySelector('.site-header__action--account');
           if (accountIconEl) {
             accountIconEl.parentNode.insertBefore(logoutFormEl, accountIconEl.nextSibling);
@@ -1048,10 +1041,7 @@
           loginIcon.style.display = 'none';
         }
       } else {
-        // User is not authenticated - hide account and logout icons
-        if (accountIcon && accountIcon.offsetParent !== null) {
-          accountIcon.style.display = 'none';
-        }
+        // User is not authenticated - keep account icon visible, hide logout icon
         if (logoutForm && logoutForm.offsetParent !== null) {
           logoutForm.style.display = 'none';
         }
@@ -1066,12 +1056,12 @@
     
     // Also listen for login success events and update UI immediately
     // This handles cases where login happens without page refresh
-    document.addEventListener('loginSuccess', function() {
+    document.addEventListener('loginSuccess', () => {
       setTimeout(updateAuthUI, 200);
     });
     
     // Monitor for login modal success view being shown
-    const observer = new MutationObserver(function(mutations) {
+    const observer = new MutationObserver((mutations) => {
       const successView = document.querySelector('[data-login-view="success"]');
       if (successView && !successView.hidden) {
         // Login success detected - update UI
@@ -1091,7 +1081,7 @@
     
     // Also check periodically (every 2 seconds) if user becomes authenticated
     // This handles cases where cookies are set but UI hasn't updated
-    setInterval(function() {
+    setInterval(() => {
       const authenticated = isAuthenticated();
       const accountIcon = document.querySelector('.site-header__action--account');
       const logoutForm = document.querySelector('.site-header__logout-form');
@@ -1105,7 +1095,7 @@
     // Handle account icon click - check authentication and open login modal if needed
     const accountIcon = document.querySelector('.site-header__action--account');
     if (accountIcon) {
-      accountIcon.addEventListener('click', function(e) {
+      accountIcon.addEventListener('click', (e) => {
         // If not authenticated, prevent navigation and open login modal
         if (!isAuthenticated()) {
           e.preventDefault();
@@ -1142,4 +1132,4 @@
     }
   ]
 }
-{% endschema %}
+{% endschema %}

package/test-theme/sections/hero-fallback.liquid

@@ -7,7 +7,7 @@
 {% assign hero_title = section.settings.hero_title | default: shop.name %}
 {% assign hero_subtitle = section.settings.hero_subtitle | default: 'Discover our curated collection' %}
 {% assign hero_button_text = section.settings.hero_button_text | default: 'Shop Now' %}
-{% assign hero_button_url = section.settings.hero_button_url | default: '/collections/all' %}
+{% assign hero_button_url = section.settings.hero_button_url | default: '/categories' %}
 {% assign hero_button_2_text = section.settings.hero_button_2_text %}
 {% assign hero_button_2_url = section.settings.hero_button_2_url %}
 {% assign hero_text_position = section.settings.text_position | default: 'center' %}