fix(nc-gui): show image

nocodb · May 11, 2024 · c4440e4 · c4440e4
1 parent 69ded65
commit c4440e4
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/packages/nc-gui/components/project/AccessSettings.vue b/packages/nc-gui/components/project/AccessSettings.vue
@@ -158,7 +158,7 @@ const selected = reactive<{
 
 const toggleSelectAll = (value: boolean) => {
   filteredCollaborators.value.forEach((_, i) => {
-    selected[i] = value
+    selected[_.id] = value
   })
 }
 
@@ -267,10 +267,13 @@ watch(currentBase, () => {
             <div
               v-for="(collab, i) of sortedCollaborators"
               :key="i"
-              class="user-row flex flex-row border-b-1 py-1 min-h-14 items-center"
+              :class="{
+                'bg-[#F0F3FF]': selected[collab.id],
+              }"
+              class="user-row flex hover:bg-[#F0F3FF] flex-row border-b-1 py-1 min-h-14 items-center"
             >
               <div class="py-3 px-6">
-                <NcCheckbox v-model:checked="selected[i]" />
+                <NcCheckbox v-model:checked="selected[collab.id]" />
               </div>
               <div class="flex gap-3 items-center users-email-grid">
                 <GeneralUserIcon size="base" :email="collab.email" />

diff --git a/packages/nc-gui/components/workspace/CollaboratorsList.vue b/packages/nc-gui/components/workspace/CollaboratorsList.vue
@@ -42,7 +42,7 @@ const filterCollaborators = computed(() => {
 
   return collaborators.value.filter(
     (collab) =>
-      collab.display_name.toLowerCase().includes(userSearchText.value.toLowerCase()) ||
+      collab.display_name?.toLowerCase().includes(userSearchText.value.toLowerCase()) ||
       collab.email?.toLowerCase().includes(userSearchText.value.toLowerCase()),
   )
 })
@@ -148,7 +148,10 @@ onMounted(async () => {
           <div
             v-for="(collab, i) of sortedCollaborators"
             :key="i"
-            class="user-row flex hover:bg-gray-50 flex-row last:border-b-0 border-b-1 py-1 min-h-14 items-center"
+            :class="{
+              'bg-[#F0F3FF]': selected[i],
+            }"
+            class="user-row flex hover:bg-[#F0F3FF] flex-row last:border-b-0 border-b-1 py-1 min-h-14 items-center"
           >
             <div class="py-3 px-6">
               <NcCheckbox v-model:checked="selected[i]" />