From b1dfeac9bc1c8e1183cd753d0b520b6923d7e318 Mon Sep 17 00:00:00 2001
From: Muhammed Mustafa <mustafa010199@gmail.com>
Date: Tue, 23 Aug 2022 01:16:37 +0530
Subject: [PATCH] fix/Removed bottom padding and unnecassry ui elements in
 shared view

---
 packages/nc-gui-v2/components/shared-view/Grid.vue   | 3 ++-
 packages/nc-gui-v2/components/smartsheet/Grid.vue    | 2 +-
 packages/nc-gui-v2/components/smartsheet/Toolbar.vue | 6 +++---
 packages/nc-gui-v2/layouts/shared-view.vue           | 6 +++---
 4 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/packages/nc-gui-v2/components/shared-view/Grid.vue b/packages/nc-gui-v2/components/shared-view/Grid.vue
index 59f5310c5e..7c0c8690e9 100644
--- a/packages/nc-gui-v2/components/shared-view/Grid.vue
+++ b/packages/nc-gui-v2/components/shared-view/Grid.vue
@@ -26,7 +26,8 @@ useProvideSmartsheetStore(sharedView as Ref<TableType>, meta, true, sorts, neste
 
 <style scoped>
 .nc-container {
-  height: calc(100% - var(--header-height));
+  height: 100%;
+  padding-bottom: 0.5rem;
   flex: 1 1 100%;
 }
 </style>
diff --git a/packages/nc-gui-v2/components/smartsheet/Grid.vue b/packages/nc-gui-v2/components/smartsheet/Grid.vue
index aff46ce8f7..7852cfd50f 100644
--- a/packages/nc-gui-v2/components/smartsheet/Grid.vue
+++ b/packages/nc-gui-v2/components/smartsheet/Grid.vue
@@ -382,7 +382,7 @@ const onNavigate = (dir: NavigateDir) => {
                   <td key="row-index" class="caption nc-grid-cell pl-5 pr-1">
                     <div class="items-center flex gap-1 min-w-[55px]">
                       <div
-                        v-if="!readOnly && !isLocked"
+                        v-if="!readOnly || !isLocked"
                         class="nc-row-no text-xs text-gray-500"
                         :class="{ hidden: row.rowMeta.selected }"
                       >
diff --git a/packages/nc-gui-v2/components/smartsheet/Toolbar.vue b/packages/nc-gui-v2/components/smartsheet/Toolbar.vue
index 2287cab3e0..ec91b6cd7e 100644
--- a/packages/nc-gui-v2/components/smartsheet/Toolbar.vue
+++ b/packages/nc-gui-v2/components/smartsheet/Toolbar.vue
@@ -33,12 +33,12 @@ const { isOpen } = useSidebar()
     <SmartsheetToolbarExport v-if="!isUIAllowed('dataInsert')" />
     <div class="flex-1" />
 
-    <SmartsheetToolbarReload class="mx-1" />
-    <SmartsheetToolbarAddRow v-if="isUIAllowed('dataInsert')" class="mx-1" />
+    <SmartsheetToolbarReload v-if="!isPublic" class="mx-1" />
+    <SmartsheetToolbarAddRow v-if="isUIAllowed('dataInsert') && !isPublic" class="mx-1" />
 
     <SmartsheetToolbarSearchData v-if="(isGrid || isGallery) && !isPublic" class="shrink mr-2 ml-2" />
 
-    <ToggleDrawer v-if="!isOpen" class="mr-2" />
+    <ToggleDrawer v-if="!isOpen && !isPublic" class="mr-2" />
   </div>
 </template>
 
diff --git a/packages/nc-gui-v2/layouts/shared-view.vue b/packages/nc-gui-v2/layouts/shared-view.vue
index 60141394c8..0805a47a06 100644
--- a/packages/nc-gui-v2/layouts/shared-view.vue
+++ b/packages/nc-gui-v2/layouts/shared-view.vue
@@ -19,11 +19,11 @@ export default {
         </div>
         <div>
           <div class="flex justify-center items-center">
-            <div class="flex items-center gap-2 ml-3">
+            <div class="flex items-center gap-2 ml-3 text-white">
               <template v-if="isLoading">
-                {{ $t('general.loading') }}
+                <span class="text-white">{{ $t('general.loading') }}</span>
 
-                <MdiReload :class="{ 'animate-infinite animate-spin': isLoading }" />
+                <MdiReload :class="{ 'animate-infinite animate-spin ': isLoading }" />
               </template>
               <div v-else class="text-xl font-semibold truncate text-white">
                 {{ sharedView?.title }}