From 5f3d512bb2830d13c925bf831d3f465dc6319adf Mon Sep 17 00:00:00 2001
From: Daniel Spaude <daniel@spaude.de>
Date: Thu, 29 Dec 2022 19:34:15 +0100
Subject: [PATCH] mobile optimisations - hide more elements when in mobile mode

---
 packages/nc-gui/components/smartsheet/Toolbar.vue             | 4 +++-
 packages/nc-gui/components/tabs/Smartsheet.vue                | 4 +++-
 .../nc-gui/pages/[projectType]/[projectId]/index/index.vue    | 4 ++--
 3 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/packages/nc-gui/components/smartsheet/Toolbar.vue b/packages/nc-gui/components/smartsheet/Toolbar.vue
index 2a028c286d..c9fcf2bede 100644
--- a/packages/nc-gui/components/smartsheet/Toolbar.vue
+++ b/packages/nc-gui/components/smartsheet/Toolbar.vue
@@ -5,6 +5,8 @@ const { isGrid, isForm, isGallery, isKanban, isSqlView } = useSmartsheetStoreOrT
 
 const isPublic = inject(IsPublicInj, ref(false))
 
+const { isMobileMode } = useGlobal()
+
 const { isUIAllowed } = useUIPermission()
 
 const { isOpen } = useSidebar('nc-right-sidebar')
@@ -45,7 +47,7 @@ const { allowCSVDownload } = useSharedView()
 
     <LazySmartsheetToolbarSearchData v-if="(isGrid || isGallery || isKanban) && !isPublic" class="shrink mx-2" />
 
-    <template v-if="!isOpen && !isPublic">
+    <template v-if="!isOpen && !isPublic && !isMobileMode">
       <div class="border-l-1 pl-3">
         <LazySmartsheetSidebarToolbarToggleDrawer class="mr-2" />
       </div>
diff --git a/packages/nc-gui/components/tabs/Smartsheet.vue b/packages/nc-gui/components/tabs/Smartsheet.vue
index 15e1f9863a..b0b1866e5b 100644
--- a/packages/nc-gui/components/tabs/Smartsheet.vue
+++ b/packages/nc-gui/components/tabs/Smartsheet.vue
@@ -31,6 +31,8 @@ const { isUIAllowed } = useUIPermission()
 
 const { metas } = useMetas()
 
+const { isMobileMode } = useGlobal()
+
 const activeTab = toRef(props, 'activeTab')
 
 const activeView = ref()
@@ -96,7 +98,7 @@ const onDecode = () => {
     <LazySmartsheetExpandedFormDetached />
 
     <!-- Lazy loading the sidebar causes issues when deleting elements, i.e. it appears as if multiple elements are removed when they are not -->
-    <SmartsheetSidebar v-if="meta" class="nc-right-sidebar" />
+    <SmartsheetSidebar v-if="meta && !isMobileMode" class="nc-right-sidebar" />
   </div>
 </template>
 
diff --git a/packages/nc-gui/pages/[projectType]/[projectId]/index/index.vue b/packages/nc-gui/pages/[projectType]/[projectId]/index/index.vue
index 8e050add53..69f1ae7d78 100644
--- a/packages/nc-gui/pages/[projectType]/[projectId]/index/index.vue
+++ b/packages/nc-gui/pages/[projectType]/[projectId]/index/index.vue
@@ -6,7 +6,7 @@ import { TabMetaInj, iconMap, provide, useGlobal, useSidebar, useTabs } from '#i
 
 const { tabs, activeTabIndex, activeTab, closeTab } = useTabs()
 
-const { isLoading } = useGlobal()
+const { isLoading, isMobileMode } = useGlobal()
 
 provide(TabMetaInj, activeTab)
 
@@ -83,7 +83,7 @@ function onEdit(targetKey: number, action: 'add' | 'remove' | string) {
         </div>
 
         <LazyGeneralShareBaseButton />
-        <LazyGeneralFullScreen class="nc-fullscreen-icon" />
+        <LazyGeneralFullScreen v-if="!isMobileMode" class="nc-fullscreen-icon" />
       </div>
 
       <div class="w-full min-h-[300px] flex-auto">