more dark logo tests

gordonwoodhull · gordonwoodhull · commit 7d61969e166c · 2025-07-01T11:25:09.000-04:00
diff --git a/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/.gitignore b/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/.gitignore
@@ -0,0 +1 @@
+/.quarto/
diff --git a/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/_brand.yml b/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/_brand.yml
@@ -0,0 +1,7 @@
+logo:
+  images:
+    light-logo:
+      path: light-logo.png
+      alt: light logo
+  small:
+    light: light-logo
diff --git a/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/_quarto.yml b/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/_quarto.yml
@@ -0,0 +1,14 @@
+project:
+  type: website
+theme:
+  light: default
+  dark: darkly
+website:
+  navbar:
+    logo:
+      light: quarto.png
+    style: "docked"
+    search: true
+    contents:
+      - index.qmd
+      - conclusion.qmd
diff --git a/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/conclusion.qmd b/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/conclusion.qmd
@@ -0,0 +1,3 @@
+---
+title: conclusion
+---
diff --git a/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/dark-fixups.scss b/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/dark-fixups.scss
@@ -0,0 +1,9 @@
+/*-- scss:rules --*/
+
+nav.sidebar.sidebar-navigation:not(.rollup) {
+  background-color: #282b30;
+}
+
+nav.navbar {
+  background-color: rgba(13, 108, 251, 25%);
+}
diff --git a/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/index.qmd b/tests/docs/smoke-all/brand/logo/navbar/theme-dark-enables-dark-logo/index.qmd
@@ -0,0 +1,15 @@
+---
+title: Yes dark logo if theme enables dark mode
+_quarto:
+  tests:
+    html:
+      ensureHtmlElements:
+        -
+          - 'img[class*="light-content"][src="./quarto.png"][alt=""]'
+          - 'img[class*="dark-content"][src="./quarto.png"][alt=""]'
+        - []
+---
+
+Chooses the project's light logo because there is no dark logo in the brand.
+
+{{< lipsum 4 >}}
diff --git a/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/.gitignore b/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/.gitignore
@@ -0,0 +1 @@
+/.quarto/
diff --git a/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/_brand.yml b/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/_brand.yml
@@ -0,0 +1,7 @@
+logo:
+  images:
+    light-logo:
+      path: light-logo.png
+      alt: light logo
+  small:
+    light: light-logo
diff --git a/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/_quarto.yml b/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/_quarto.yml
@@ -0,0 +1,14 @@
+project:
+  type: website
+theme:
+  light: default
+  dark: darkly
+website:
+  sidebar:
+    logo:
+      light: quarto.png
+    style: "docked"
+    search: true
+    contents:
+      - index.qmd
+      - conclusion.qmd
diff --git a/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/conclusion.qmd b/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/conclusion.qmd
@@ -0,0 +1,3 @@
+---
+title: conclusion
+---
diff --git a/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/dark-fixups.scss b/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/dark-fixups.scss
@@ -0,0 +1,9 @@
+/*-- scss:rules --*/
+
+nav.sidebar.sidebar-navigation:not(.rollup) {
+  background-color: #282b30;
+}
+
+nav.navbar {
+  background-color: rgba(13, 108, 251, 25%);
+}
diff --git a/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/index.qmd b/tests/docs/smoke-all/brand/logo/sidebar/theme-dark-mode-enables-dark-logo/index.qmd
@@ -0,0 +1,15 @@
+---
+title: Yes dark logo if theme enables dark mode
+_quarto:
+  tests:
+    html:
+      ensureHtmlElements:
+        -
+          - 'img[class*="light-content"][src="./quarto.png"][alt=""]'
+          - 'img[class*="dark-content"][src="./quarto.png"][alt=""]'
+        - []
+---
+
+Chooses the project's light logo because there is no dark logo in the brand.
+
+{{< lipsum 4 >}}
diff --git a/tests/docs/smoke-all/brand/logo/theme-dark-mode-brand-file-light-doc-dark-logo.qmd b/tests/docs/smoke-all/brand/logo/theme-dark-mode-brand-file-light-doc-dark-logo.qmd
@@ -0,0 +1,26 @@
+---
+title: Yes dark logo if theme enables dark mode
+format:
+  dashboard:
+    theme:
+      light: default
+      dark: darkly  # Explicitly enable dark mode
+brand: logo-light-only.yml
+logo:
+  dark:
+    path: doc-dark-logo.png
+    alt: doc dark logo
+_quarto:
+  tests:
+    dashboard:
+      ensureHtmlElements:
+        -
+          - 'img[src="light-logo.png"][alt="light logo"][class="navbar-logo light-content d-inline-block"]'
+          - 'img[src="doc-dark-logo.png"][alt="doc dark logo"][class="navbar-logo dark-content d-inline-block"]'
+        - []
+---
+
+
+Chooses the brand's light logo because there is no dark logo in the brand or document.
+
+{{< lipsum 4 >}}
diff --git a/tests/docs/smoke-all/brand/logo/theme-dark-mode-brand-file-light-no-doc-logo-yes-dark-logo.qmd b/tests/docs/smoke-all/brand/logo/theme-dark-mode-brand-file-light-no-doc-logo-yes-dark-logo.qmd
@@ -0,0 +1,22 @@
+---
+title: Yes dark logo if theme enables dark mode
+format:
+  dashboard:
+    theme:
+      light: default
+      dark: darkly  # Explicitly enable dark mode
+brand: logo-light-only.yml
+_quarto:
+  tests:
+    dashboard:
+      ensureHtmlElements:
+        -
+          - 'img[src="light-logo.png"][alt="light logo"][class="navbar-logo light-content d-inline-block"]'
+          - 'img[src="light-logo.png"][alt="light logo"][class="navbar-logo dark-content d-inline-block"]'
+        - []
+---
+
+
+Chooses the brand's light logo because there is no dark logo in the brand or document.
+
+{{< lipsum 4 >}}
diff --git a/tests/docs/smoke-all/brand/logo/theme-dark-mode-brand-file-light-yes-dark-logo.qmd b/tests/docs/smoke-all/brand/logo/theme-dark-mode-brand-file-light-yes-dark-logo.qmd
@@ -14,17 +14,11 @@ _quarto:
       ensureHtmlElements:
         -
           - 'img[src="quarto.png"][alt=""][class="navbar-logo light-content d-inline-block"]'
-          - 'img[src="light-logo.png"][alt="light logo"][class="navbar-logo dark-content d-inline-block"]'
+          - 'img[src="quarto.png"][alt=""][class="navbar-logo dark-content d-inline-block"]'
         - []
 ---
 
 
-There is ambiguity whether this should use `quarto.png` for the dark logo, since
-the brand provides no dark logos, but if that's the intention then use
-
-```yaml
-logo: quarto.png
-```
-
+Chooses the document-specified light logo because there is no dark logo in the brand
 
 {{< lipsum 4 >}}
diff --git a/tests/docs/smoke-all/brand/logo/theme-dark-mode-yes-dark-logo.qmd b/tests/docs/smoke-all/brand/logo/theme-dark-mode-yes-dark-logo.qmd
@@ -21,16 +21,11 @@ _quarto:
       ensureHtmlElements:
         -
           - 'img[src="quarto.png"][alt=""][class="navbar-logo light-content d-inline-block"]'
-          - 'img[src="light-logo.png"][alt="light logo"][class="navbar-logo dark-content d-inline-block"]'
+          - 'img[src="quarto.png"][alt=""][class="navbar-logo dark-content d-inline-block"]'
         - []
 ---
 
-There is ambiguity whether this should use `quarto.png` for the dark logo, since
-the brand provides no dark logos, but if that's the intention then use
-
-```yaml
-logo: quarto.png
-```
+Chooses the documents's light logo because there is no dark logo in the brand or document.
 
 
 {{< lipsum 4 >}}
diff --git a/tests/docs/smoke-all/brand/logo/theme-dark-project-brand-file-light/yes-dark-logo.qmd b/tests/docs/smoke-all/brand/logo/theme-dark-project-brand-file-light/yes-dark-logo.qmd
@@ -8,17 +8,12 @@ _quarto:
       ensureHtmlElements:
         -
           - 'img[src="quarto.png"][alt=""][class="navbar-logo light-content d-inline-block"]'
-          - 'img[src="light-logo.png"][alt="light logo"][class="navbar-logo dark-content d-inline-block"]'
+          - 'img[src="quarto.png"][alt=""][class="navbar-logo dark-content d-inline-block"]'
         - []
 ---
 
 
-There is ambiguity whether this should use `quarto.png` for the dark logo, since
-the brand provides no dark logos, but if that's the intention then use
-
-```yaml
-logo: quarto.png
-```
+Uses the light logo as a dark logo because there is no dark logo in the brand.
 
 
 {{< lipsum 4 >}}
