Merge pull request #5383 from nocodb/fix/icon-issues

fix: add border for left tree view and icon alignment

packages/nc-gui/components/dashboard/TreeView.vue

@@ -1132,6 +1132,7 @@ const setIcon = async (icon: string, table: TableType) => {
 <style scoped lang="scss">
 .nc-treeview-container {
   @apply h-[calc(100vh_-_var(--header-height))];
+  border-right: 1px solid var(--navbar-border) !important;
 }
 
 .nc-treeview-footer-item {

packages/nc-gui/components/dashboard/settings/Modal.vue

@@ -227,7 +227,7 @@ watch(
                 @click="vDataState = DataSourcesSubTab.New"
               >
                 <div v-if="vDataState === ''" class="flex items-center gap-2 text-primary font-light">
-                  <component :is="iconMap.plusCircle" class="text-lg group-hover:text-accent" />
+                  <component :is="iconMap.plusCircle" class="group-hover:text-accent" />
                   New
                 </div>
               </a-button>

packages/nc-gui/components/smartsheet/toolbar/AddRow.vue

@@ -21,7 +21,7 @@ const onClick = () => {
     >
       <component
         :is="iconMap.plus"
-        :class="{ 'cursor-pointer text-gray-500 group-hover:(text-primary)': !isLocked, 'disabled': isLocked }"
+        :class="{ 'cursor-pointer group-hover:(text-primary)': !isLocked, 'disabled': isLocked }"
         @click="onClick"
       />
     </div>

packages/nc-gui/components/smartsheet/toolbar/Reload.vue

@@ -28,7 +28,7 @@ const onClick = () => {
     <div class="nc-toolbar-btn flex min-w-32px w-32px h-32px items-center justify-center select-none">
       <component
         :is="iconMap.reload"
-        class="cursor-pointer text-gray-500 group-hover:(text-primary) nc-toolbar-reload-btn"
+        class="cursor-pointer group-hover:(text-primary) nc-toolbar-reload-btn"
         :class="isReloading ? 'animate-spin' : ''"
         @click="onClick"
       />