From 0d46b5e1d6d017a73c11b9ba997d8d85bcafd53b Mon Sep 17 00:00:00 2001
From: braks <78412429+bcakmakoglu@users.noreply.github.com>
Date: Fri, 5 Aug 2022 16:34:55 +0200
Subject: [PATCH] refactor(gui-v2): show header again

---
 packages/nc-gui-v2/app.vue              | 34 +++++++++++++------------
 packages/nc-gui-v2/assets/style-v2.scss |  2 +-
 2 files changed, 19 insertions(+), 17 deletions(-)

diff --git a/packages/nc-gui-v2/app.vue b/packages/nc-gui-v2/app.vue
index 3a11050c8d..123f9c6c6d 100644
--- a/packages/nc-gui-v2/app.vue
+++ b/packages/nc-gui-v2/app.vue
@@ -1,14 +1,14 @@
 <script lang="ts" setup>
 import { breakpointsTailwind } from '@vueuse/core'
 import { navigateTo } from '#app'
-import { computed, ref, useBreakpoints, useGlobal, useSidebar } from '#imports'
+import { computed, provideSidebar, ref, useBreakpoints, useGlobal } from '#imports'
 
 /** get current breakpoints (for enabling sidebar) */
 const breakpoints = useBreakpoints(breakpointsTailwind)
 
 const { signOut, signedIn, isLoading, user } = $(useGlobal())
 
-const { isOpen, toggle, hasSidebar } = useSidebar({ isOpen: signedIn && breakpoints.greater('md').value })
+const { isOpen, toggle, hasSidebar } = provideSidebar({ isOpen: signedIn && breakpoints.greater('md').value })
 
 const sidebar = ref<HTMLDivElement>()
 
@@ -21,8 +21,8 @@ const logout = () => {
 </script>
 
 <template>
-  <a-layout class="min-h-[100vh]">
-    <a-layout-header class="hidden flex !bg-primary items-center text-white px-4 shadow-md">
+  <a-layout>
+    <a-layout-header class="flex !bg-primary items-center text-white px-4 shadow-md">
       <material-symbols-menu v-if="signedIn && hasSidebar" class="text-xl cursor-pointer" @click="toggle" />
 
       <div class="flex-1" />
@@ -75,17 +75,19 @@ const logout = () => {
       </div>
     </a-layout-header>
 
-    <a-layout-sider
-      :collapsed="!isOpen"
-      width="300"
-      collapsed-width="0"
-      class="bg-white dark:!bg-gray-800 border-r-1 border-gray-200 dark:!border-gray-600 h-full"
-      :trigger="null"
-      collapsible
-    >
-      <div id="sidebar" ref="sidebar" class="w-full h-full" />
-    </a-layout-sider>
-
-    <NuxtPage />
+    <a-layout>
+      <a-layout-sider
+        :collapsed="!isOpen"
+        width="300"
+        collapsed-width="0"
+        class="bg-white dark:!bg-gray-800 border-r-1 border-gray-200 dark:!border-gray-600 h-full"
+        :trigger="null"
+        collapsible
+      >
+        <div id="sidebar" ref="sidebar" class="w-full h-full" />
+      </a-layout-sider>
+
+      <NuxtPage />
+    </a-layout>
   </a-layout>
 </template>
diff --git a/packages/nc-gui-v2/assets/style-v2.scss b/packages/nc-gui-v2/assets/style-v2.scss
index 5bf28b6b34..462ee5f05b 100644
--- a/packages/nc-gui-v2/assets/style-v2.scss
+++ b/packages/nc-gui-v2/assets/style-v2.scss
@@ -2,7 +2,7 @@
 @import 'ant-design-vue/dist/antd.min.css';
 
 :root {
-  --header-height: 0px;
+  --header-height: 56px;
 }
 
 .ant-layout-header {