Resolve sidebar logo for all sidebars, not just the first

The logo resolution logic (resolveLogo + logoAddLeadingSlashes) only ran
for sidebars[0]. Other sidebars kept raw string logo values, but the EJS
template expects { light: { path }, dark: { path } } objects since the
dark logo support was added in #12996. This caused empty <a> anchors
with no <img> for any sidebar after the first.

Loop the resolution over all sidebar entries so each sidebar's logo is
properly transformed.

Author: cderv

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

@@ -171,40 +171,23 @@ export async function websiteNavigationConfig(project: ProjectContext) {
       sidebars[0].tools = [];
     }
 
-    let sideLogo = sidebars[0].logo;
-    if (sideLogo !== false) { // don't do anything logo processing when sidebar logo is opt-out
-      if (sideLogo && sidebars[0][kLogoAlt]) {
-        const alt = sidebars[0][kLogoAlt];
-        if (typeof sideLogo === "string") {
-          sideLogo = { path: sideLogo, alt };
+    for (const sb of sidebars) {
+      let sideLogo = sb.logo;
+      if (sideLogo !== false) { // don't do anything logo processing when sidebar logo is opt-out
+        if (sideLogo && sb[kLogoAlt]) {
+          const alt = sb[kLogoAlt];
+          if (typeof sideLogo === "string") {
+            sideLogo = { path: sideLogo, alt };
+          }
         }
-        // possible but absurd
-        // else if ("path" in sideLogo) {
-        //   sideLogo = { ...sideLogo, alt };
-        // } else {
-        //   sideLogo = {
-        //     light: !sideLogo.light ? undefined : typeof sideLogo.light === "string"
-        //       ? {
-        //         path: sideLogo.light,
-        //         alt,
-        //       }
-        //       : { ...sideLogo.light, alt },
-        //     dark: !sideLogo.dark ? undefined : typeof sideLogo.dark === "string"
-        //       ? {
-        //         path: sideLogo.dark,
-        //         alt,
-        //       }
-        //       : { ...sideLogo.dark, alt },
-        //   };
-        // }
+        let logo = resolveLogo(projectBrand, sideLogo, [
+          "medium",
+          "small",
+          "large",
+        ]);
+        logo = logoAddLeadingSlashes(logo, projectBrand, undefined);
+        sb.logo = logo;
       }
-      let logo = resolveLogo(projectBrand, sideLogo, [
-        "medium",
-        "small",
-        "large",
-      ]);
-      logo = logoAddLeadingSlashes(logo, projectBrand, undefined);
-      sidebars[0].logo = logo;
     }
 
     // convert contents: auto into items

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

@@ -0,0 +1,2 @@
+/.quarto/
+**/*.quarto_ipynb

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

@@ -0,0 +1,24 @@
+project:
+  type: website
+  output-dir: _site
+
+render:
+  - index.qmd
+  - page.qmd
+
+website:
+  title: "Issue 14353"
+  sidebar:
+    - title: "Group 1"
+      logo: /img/logo-a.png
+      contents:
+        - index.qmd
+
+    - title: "Group 2"
+      logo: /img/logo-b.png
+      contents:
+        - page.qmd
+
+format:
+  html:
+    theme: cosmo

tests/docs/smoke-all/website-sidebar/issue-14353-multi-sidebar-logo/img/logo-a.png

@@ -0,0 +1,12 @@
+---
+title: "Group 1 Page"
+_quarto:
+  tests:
+    html:
+      ensureHtmlElements:
+        -
+          - 'a.sidebar-logo-link img[src$="logo-a.png"][class*="sidebar-logo"]'
+        - []
+---
+
+This page belongs to the first sidebar entry. Its logo (logo-a.png) should render.

tests/docs/smoke-all/website-sidebar/issue-14353-multi-sidebar-logo/img/logo-b.png

@@ -0,0 +1,12 @@
+---
+title: "Group 2 Page"
+_quarto:
+  tests:
+    html:
+      ensureHtmlElements:
+        -
+          - 'a.sidebar-logo-link img[src$="logo-b.png"][class*="sidebar-logo"]'
+        - []
+---
+
+This page belongs to the second sidebar entry. Its logo (logo-b.png) should render.