From 42a0ef21c17da20c8f22565807578a05a0461df6 Mon Sep 17 00:00:00 2001
From: Divyansh Singh <40380293+brc-dd@users.noreply.github.com>
Date: Tue, 21 Mar 2023 16:20:45 +0530
Subject: [PATCH] fix(theme): don't show outline when no header is there
 (#2117)

---
 .../components/VPDocOutlineDropdown.vue       | 16 ++++++++++++----
 .../components/VPLocalNavOutlineDropdown.vue  | 19 +++++++++++++++----
 2 files changed, 27 insertions(+), 8 deletions(-)

diff --git a/src/client/theme-default/components/VPDocOutlineDropdown.vue b/src/client/theme-default/components/VPDocOutlineDropdown.vue
index 22d16c821954..5cd44a091a55 100644
--- a/src/client/theme-default/components/VPDocOutlineDropdown.vue
+++ b/src/client/theme-default/components/VPDocOutlineDropdown.vue
@@ -1,7 +1,7 @@
 <script setup lang="ts">
-import { ref } from 'vue'
+import { ref, shallowRef } from 'vue'
 import { useData } from '../composables/data'
-import { getHeaders, resolveTitle } from '../composables/outline'
+import { getHeaders, resolveTitle, type MenuItem } from '../composables/outline'
 import VPDocOutlineItem from './VPDocOutlineItem.vue'
 import { onContentUpdated } from 'vitepress'
 import VPIconChevronRight from './icons/VPIconChevronRight.vue'
@@ -12,16 +12,24 @@ const open = ref(false)
 onContentUpdated(() => {
   open.value = false
 })
+
+const headers = shallowRef<MenuItem[]>([])
+
+onContentUpdated(() => {
+  headers.value = getHeaders(
+    frontmatter.value.outline ?? theme.value.outline
+  )
+})
 </script>
 
 <template>
-  <div class="VPDocOutlineDropdown">
+  <div class="VPDocOutlineDropdown" v-if="headers.length > 0">
     <button @click="open = !open" :class="{ open }">
       {{ resolveTitle(theme) }}
       <VPIconChevronRight class="icon" />
     </button>
     <div class="items" v-if="open">
-      <VPDocOutlineItem :headers="getHeaders(frontmatter.outline ?? theme.outline)" />
+      <VPDocOutlineItem :headers="headers" />
     </div>
   </div>
 </template>
diff --git a/src/client/theme-default/components/VPLocalNavOutlineDropdown.vue b/src/client/theme-default/components/VPLocalNavOutlineDropdown.vue
index 10e26c8ee7d8..b68890c8b229 100644
--- a/src/client/theme-default/components/VPLocalNavOutlineDropdown.vue
+++ b/src/client/theme-default/components/VPLocalNavOutlineDropdown.vue
@@ -1,7 +1,7 @@
 <script setup lang="ts">
-import { ref, nextTick } from 'vue'
+import { ref, nextTick, shallowRef } from 'vue'
 import { useData } from '../composables/data'
-import { getHeaders, resolveTitle } from '../composables/outline'
+import { getHeaders, resolveTitle, type MenuItem } from '../composables/outline'
 import VPDocOutlineItem from './VPDocOutlineItem.vue'
 import { onContentUpdated } from 'vitepress'
 import VPIconChevronRight from './icons/VPIconChevronRight.vue'
@@ -36,14 +36,25 @@ function scrollToTop() {
   open.value = false
   window.scrollTo({ top: 0, left: 0, behavior: 'smooth' })
 }
+
+const headers = shallowRef<MenuItem[]>([])
+
+onContentUpdated(() => {
+  headers.value = getHeaders(
+    frontmatter.value.outline ?? theme.value.outline
+  )
+})
 </script>
 
 <template>
   <div class="VPLocalNavOutlineDropdown" :style="{ '--vp-vh': vh + 'px' }">
-    <button @click="toggle" :class="{ open }">
+    <button @click="toggle" :class="{ open }" v-if="headers.length > 0">
       {{ resolveTitle(theme) }}
       <VPIconChevronRight class="icon" />
     </button>
+    <button @click="scrollToTop" v-else>
+      {{ theme.returnToTopLabel || 'Return to top' }}
+    </button>
     <Transition name="flyout">
       <div v-if="open"
         ref="items"
@@ -53,7 +64,7 @@ function scrollToTop() {
         <a class="top-link" href="#" @click="scrollToTop">
           {{ theme.returnToTopLabel || 'Return to top' }}
         </a>
-        <VPDocOutlineItem :headers="getHeaders(frontmatter.outline ?? theme.outline)" />
+        <VPDocOutlineItem :headers="headers" />
       </div>
     </Transition>
   </div>