Don’t hide banner style titles unless there is a sidebar there

quarto-dev · Apr 30, 2022 · e1a9c37 · e1a9c37
1 parent f66cd5f
commit e1a9c37
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/project/types/website/website-navigation.ts b/src/project/types/website/website-navigation.ts
@@ -323,6 +323,7 @@ export async function websiteNavigationExtras(
       [kHtmlPostprocessors]: [
         navigationHtmlPostprocessor(
           project,
+          format,
           source,
           markdownPipeline,
           format.language,
@@ -362,6 +363,7 @@ export function writeRedirectPage(path: string, href: string) {
 
 function navigationHtmlPostprocessor(
   project: ProjectContext,
+  format: Format,
   source: string,
   markdownPipeline: MarkdownPipeline,
   language: FormatLanguage,
@@ -374,6 +376,12 @@ function navigationHtmlPostprocessor(
     // Process any markdown rendered through the render envelope
     markdownPipeline.processRenderedMarkdown(doc);
 
+    // Mark the body with a sidebar, if present
+    const sidebar = sidebarForHref(href, format);
+    if (sidebar) {
+      doc.body.classList.add("nav-sidebar");
+    }
+
     // Note sidebar style on body
     // TODO: Should we compute this using the using project instead?
     // It is slightly complicated since we need to compute the sidebar

diff --git a/src/resources/formats/html/bootstrap/_bootstrap-rules.scss b/src/resources/formats/html/bootstrap/_bootstrap-rules.scss
@@ -1573,8 +1573,7 @@ nav.quarto-secondary-nav.color-navbar .quarto-btn-toggle {
   }    
 }
 
-body.floating,
-body.docked {
+body.nav-sidebar {
   @include media-breakpoint-down(lg) {
     .quarto-title-banner,
     .quarto-title-banner {