From b0290e2eb2a81bbe558d6b09e3e0ea750352dd09 Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Tue, 20 Sep 2022 11:30:50 +0530
Subject: [PATCH 1/3] refactor(gui): show fullscreen button always

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 .../pages/[projectType]/[projectId]/index/index.vue    | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/packages/nc-gui/pages/[projectType]/[projectId]/index/index.vue b/packages/nc-gui/pages/[projectType]/[projectId]/index/index.vue
index d8346776a5..397d7eec5f 100644
--- a/packages/nc-gui/pages/[projectType]/[projectId]/index/index.vue
+++ b/packages/nc-gui/pages/[projectType]/[projectId]/index/index.vue
@@ -148,14 +148,4 @@ function onEdit(targetKey: number, action: 'add' | 'remove' | string) {
 :deep(.ant-tabs-tab-remove) {
   @apply mt-[3px];
 }
-
-.nc-tab-bar {
-  :deep(.nc-fullscreen-icon) {
-    @apply opacity-0 transition;
-  }
-
-  &:hover :deep(.nc-fullscreen-icon) {
-    @apply opacity-100;
-  }
-}
 </style>

From b1a7c6272da7a26f7f917bf24561f01027300d95 Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Tue, 20 Sep 2022 11:31:46 +0530
Subject: [PATCH 2/3] fix(gui): show fullscreen button tooltip on left side to
 avoid unexpected scrollbar

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 packages/nc-gui/components/general/FullScreen.vue | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/packages/nc-gui/components/general/FullScreen.vue b/packages/nc-gui/components/general/FullScreen.vue
index 4667c01daa..53694bb1b9 100644
--- a/packages/nc-gui/components/general/FullScreen.vue
+++ b/packages/nc-gui/components/general/FullScreen.vue
@@ -14,9 +14,11 @@ const isSidebarsOpen = computed({
 </script>
 
 <template>
-  <a-tooltip>
+  <a-tooltip placement="left">
     <!-- todo: i18n -->
-    <template #title> {{ isSidebarsOpen ? 'Full width' : 'Exit full width' }}</template>
+    <template #title>
+      <span class="text-xs">{{ isSidebarsOpen ? 'Full width' : 'Exit full width' }}</span>
+    </template>
     <div
       v-e="['c:toolbar:fullscreen']"
       class="nc-fullscreen-btn cursor-pointer flex align-center self-center px-2 py-2 mr-2"

From 84a4da6b3f69b4e6400d8bb06770e0b781da878f Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Tue, 20 Sep 2022 11:33:06 +0530
Subject: [PATCH 3/3] chore(gui): lint

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 .../nc-gui/components/smartsheet-toolbar/ColumnFilter.vue     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/nc-gui/components/smartsheet-toolbar/ColumnFilter.vue b/packages/nc-gui/components/smartsheet-toolbar/ColumnFilter.vue
index 181a0d18d3..db3fb430ed 100644
--- a/packages/nc-gui/components/smartsheet-toolbar/ColumnFilter.vue
+++ b/packages/nc-gui/components/smartsheet-toolbar/ColumnFilter.vue
@@ -108,7 +108,7 @@ watch(
 )
 
 watch(
-  () => filters.value.length ,
+  () => filters.value.length,
   (length) => {
     emit('update:filtersLength', length ?? 0)
   },
@@ -281,7 +281,7 @@ defineExpose({
           {{ $t('activity.addFilter') }}
         </div>
       </a-button>
-      <a-button class="text-capitalize !text-gray-500" v-if="!webHook" @click.stop="addFilterGroup">
+      <a-button v-if="!webHook" class="text-capitalize !text-gray-500" @click.stop="addFilterGroup">
         <div class="flex items-center gap-1">
           <!--      <v-icon small color="grey"> mdi-plus </v-icon> -->
           <!--          Add Filter Group -->