From 255647d2a931bb045bf9e5e3621bffb5ad2496a5 Mon Sep 17 00:00:00 2001 From: Brian Talbot Date: Thu, 25 Sep 2025 10:55:40 -0400 Subject: [PATCH] fixing theme.js logic (#1492) * to adhere to original work in talbs/update-theme-logic * to also account for work in talbs/update-header --- packages/webawesome/docs/assets/scripts/theme.js | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/packages/webawesome/docs/assets/scripts/theme.js b/packages/webawesome/docs/assets/scripts/theme.js index 6f1b2fe41..1f8e1eeb4 100644 --- a/packages/webawesome/docs/assets/scripts/theme.js +++ b/packages/webawesome/docs/assets/scripts/theme.js @@ -9,9 +9,9 @@ async function updateTheme(value, isInitialLoad = false) { // Get brand, palette, and theme name from the selected option const themeSelector = document.querySelector('.theme-selector'); const selectedOption = themeSelector?.querySelector(`wa-dropdown-item[value="${value}"]`); - const brand = selectedOption?.getAttribute('data-brand') || 'blue'; - const palette = selectedOption?.getAttribute('data-palette') || 'default'; - const themeName = selectedOption?.textContent.trim() || 'Unknown'; + let brand = selectedOption?.getAttribute('data-brand') || 'blue'; + let palette = selectedOption?.getAttribute('data-palette') || 'default'; + let themeName = selectedOption?.textContent.trim() || 'Unknown'; if (!isInitialLoad) { // Disable tooltip during theme transition @@ -39,12 +39,14 @@ async function updateTheme(value, isInitialLoad = false) { let href = `/dist/styles/themes/${value}.css`; if (document.querySelector('wa-page').dataset.pageType === 'site') { - value = 'site'; + brand = 'orange'; href = `/assets/styles/theme-site.css`; + palette = 'default'; + value = 'site'; } else { - localStorage.setItem('theme', value); localStorage.setItem('brand', brand); localStorage.setItem('palette', palette); + localStorage.setItem('theme', value); } // Update theme classes