Skip logo normalization for sidebars without explicit logo

Preserve inheritance: sidebars that don't define logo should inherit
from the first sidebar via propagation spread. Add changelog entry
and test coverage for the inheritance case.

Author: cderv

news/changelog-1.10.md

@@ -13,6 +13,12 @@ All changes included in 1.10:
 
 - ([#14261](https://github.com/quarto-dev/quarto-cli/issues/14261)): Fix theorem/example block titles containing inline code producing invalid Typst markup when syntax highlighting is applied.
 
+## Projects
+
+### Websites
+
+- ([#13565](https://github.com/quarto-dev/quarto-cli/issues/13565), [#14353](https://github.com/quarto-dev/quarto-cli/issues/14353)): Fix sidebar logo not appearing on secondary sidebars in multi-sidebar website layouts.
+
 ## Commands
 
 ### `quarto preview`

src/project/types/website/website-shared.ts

@@ -172,6 +172,7 @@ export async function websiteNavigationConfig(project: ProjectContext) {
     }
 
     for (const sb of sidebars) {
+      if (sb.logo === undefined) continue; // let propagation handle inheritance
       let sideLogo = sb.logo;
       if (sideLogo !== false) { // don't do anything logo processing when sidebar logo is opt-out
         if (sideLogo && sb[kLogoAlt]) {

tests/docs/smoke-all/website-sidebar/issue-14353-multi-sidebar-logo/_quarto.yml

@@ -5,6 +5,7 @@ project:
 render:
   - index.qmd
   - page.qmd
+  - page2.qmd
 
 website:
   title: "Issue 14353"
@@ -19,6 +20,10 @@ website:
       contents:
         - page.qmd
 
+    - title: "Group 3"
+      contents:
+        - page2.qmd
+
 format:
   html:
     theme: cosmo

tests/docs/smoke-all/website-sidebar/issue-14353-multi-sidebar-logo/page2.qmd

@@ -0,0 +1,12 @@
+---
+title: "Group 3 Page"
+_quarto:
+  tests:
+    html:
+      ensureHtmlElements:
+        -
+          - 'a.sidebar-logo-link img[src$="logo-a.png"][class*="sidebar-logo"]'
+        - []
+---
+
+This page belongs to the third sidebar entry which has no explicit logo. It should inherit logo-a.png from the first sidebar.