From 27516e4368a969e93e795bd78cf7fa919bbc48ac Mon Sep 17 00:00:00 2001
From: Yoones Khoshghadam <yooneskh@gmail.com>
Date: Thu, 5 Dec 2024 04:04:08 +0000
Subject: [PATCH] fix (nc-gui): icon size and color in grid view attachment
 cell

---
 .../components/cell/attachment/Modal.vue       | 18 ++++++++++++++++--
 .../components/cell/attachment/index.vue       |  4 ++--
 .../expanded-form/Sidebar/Audits.vue           |  7 ++++++-
 .../workspace/integrations/ConnectionsTab.vue  |  1 -
 4 files changed, 24 insertions(+), 6 deletions(-)

diff --git a/packages/nc-gui/components/cell/attachment/Modal.vue b/packages/nc-gui/components/cell/attachment/Modal.vue
index e54dbc0fd3..369b78302f 100644
--- a/packages/nc-gui/components/cell/attachment/Modal.vue
+++ b/packages/nc-gui/components/cell/attachment/Modal.vue
@@ -158,9 +158,23 @@ const handleFileDelete = (i: number) => {
               @click.stop="onClick(item)"
             />
 
-            <component :is="FileIcon(item.icon)" v-else-if="item.icon" :height="45" :width="45" @click.stop="onClick(item)" />
+            <component
+              :is="FileIcon(item.icon)"
+              v-else-if="item.icon"
+              :height="45"
+              :width="45"
+              class="text-white"
+              @click.stop="onClick(item)"
+            />
 
-            <IcOutlineInsertDriveFile v-else :height="45" :width="45" @click.stop="onClick(item)" />
+            <GeneralIcon
+              v-else
+              icon="ncFileTypeUnknown"
+              :height="45"
+              :width="45"
+              class="text-white"
+              @click.stop="onClick(item)"
+            />
           </div>
 
           <div class="relative px-1 pb-1 items-center flex" :title="item.title">
diff --git a/packages/nc-gui/components/cell/attachment/index.vue b/packages/nc-gui/components/cell/attachment/index.vue
index e3a0351357..88a89e2143 100644
--- a/packages/nc-gui/components/cell/attachment/index.vue
+++ b/packages/nc-gui/components/cell/attachment/index.vue
@@ -433,9 +433,9 @@ defineExpose({
               }"
               @click="onFileClick(item)"
             >
-              <component :is="FileIcon(item.icon)" v-if="item.icon" />
+              <component :is="FileIcon(item.icon)" v-if="item.icon" class="text-white max-h-full max-w-full" />
 
-              <GeneralIcon v-else icon="ncFileTypeUnknown" class="text-white" />
+              <GeneralIcon v-else icon="ncFileTypeUnknown" class="text-white max-h-full max-w-full" />
             </div>
           </NcTooltip>
         </template>
diff --git a/packages/nc-gui/components/smartsheet/expanded-form/Sidebar/Audits.vue b/packages/nc-gui/components/smartsheet/expanded-form/Sidebar/Audits.vue
index 88fe61e22e..8ace242c7d 100644
--- a/packages/nc-gui/components/smartsheet/expanded-form/Sidebar/Audits.vue
+++ b/packages/nc-gui/components/smartsheet/expanded-form/Sidebar/Audits.vue
@@ -83,7 +83,12 @@ watch(
         <div class="group gap-3 overflow-hidden px-3 py-2 hover:bg-gray-100">
           <div class="flex items-start justify-between">
             <div class="flex items-start gap-3 flex-1 w-full">
-              <GeneralUserIcon :email="audit?.created_by_email" :name="audit?.created_display_name" class="mt-0.5" size="medium" />
+              <GeneralUserIcon
+                :email="audit?.created_by_email"
+                :name="audit?.created_display_name"
+                class="mt-0.5"
+                size="medium"
+              />
               <div class="flex h-[28px] items-center gap-3 w-[calc(100%_-_40px)]">
                 <NcDropdown placement="topLeft" :trigger="['hover']" class="flex-none max-w-[calc(100%_-_72px)]">
                   <div class="truncate text-gray-800 font-medium !text-small !leading-[18px] overflow-hidden">
diff --git a/packages/nc-gui/components/workspace/integrations/ConnectionsTab.vue b/packages/nc-gui/components/workspace/integrations/ConnectionsTab.vue
index e219210d9e..c5ea772a2e 100644
--- a/packages/nc-gui/components/workspace/integrations/ConnectionsTab.vue
+++ b/packages/nc-gui/components/workspace/integrations/ConnectionsTab.vue
@@ -464,7 +464,6 @@ onKeyStroke('ArrowDown', onDown)
                         <GeneralUserIcon
                           :email="collaboratorsMap.get(integration.created_by)?.email"
                           :name="collaboratorsMap.get(integration.created_by)?.display_name"
-
                           size="base"
                           class="flex-none"
                           :class="{