From 236a5071ae43198cf452b0c56bc878e2ced9c328 Mon Sep 17 00:00:00 2001
From: Hrvoje Vucic <hvucic@extensionengine.com>
Date: Fri, 27 Aug 2021 18:07:58 +0200
Subject: [PATCH] Integrate v-lazy with data table in workflow overview

---
 .../repository/Workflow/Overview/Assignee.vue |  2 +-
 .../repository/Workflow/Overview/Priority.vue |  2 +-
 .../repository/Workflow/Overview/index.vue    | 73 ++++++++++++-------
 3 files changed, 50 insertions(+), 27 deletions(-)

diff --git a/client/components/repository/Workflow/Overview/Assignee.vue b/client/components/repository/Workflow/Overview/Assignee.vue
index 4fb3448da..5813bace4 100644
--- a/client/components/repository/Workflow/Overview/Assignee.vue
+++ b/client/components/repository/Workflow/Overview/Assignee.vue
@@ -1,7 +1,7 @@
 <template>
   <v-tooltip open-delay="800" bottom>
     <template #activator="{ on }">
-      <div v-on="on" class="column-assignee d-flex align-center">
+      <div v-on="on" class="column-assignee">
         <assignee-avatar
           :label="label"
           :img-url="imgUrl"
diff --git a/client/components/repository/Workflow/Overview/Priority.vue b/client/components/repository/Workflow/Overview/Priority.vue
index 686989056..8240c66b7 100644
--- a/client/components/repository/Workflow/Overview/Priority.vue
+++ b/client/components/repository/Workflow/Overview/Priority.vue
@@ -1,5 +1,5 @@
 <template>
-  <div class="d-flex align-center">
+  <div>
     <v-icon class="priority-icon mr-2">
       {{ `$vuetify.icons.${icon}` }}
     </v-icon>
diff --git a/client/components/repository/Workflow/Overview/index.vue b/client/components/repository/Workflow/Overview/index.vue
index fbf0d0841..74dac54d9 100644
--- a/client/components/repository/Workflow/Overview/index.vue
+++ b/client/components/repository/Workflow/Overview/index.vue
@@ -1,29 +1,39 @@
 <template>
   <v-data-table
-    @click:row="selectActivity($event.id)"
     :headers="headers"
     :items="items"
     height="100%"
     item-class="class"
     disable-pagination hide-default-footer fixed-header
     class="overview primary lighten-5">
-    <template #item.name="{ item, value }">
-      <div class="d-flex">
-        <overview-name :value="value" />
-        <publishing-badge v-if="isAdmin || isRepositoryAdmin" :activity="item" />
-      </div>
-    </template>
-    <template #item.status="{ value }">
-      <overview-status v-bind="value" />
-    </template>
-    <template #item.assignee="{ value }">
-      <overview-assignee v-bind="value" />
-    </template>
-    <template #item.priority="{ value }">
-      <overview-priority v-bind="value" />
-    </template>
-    <template #item.dueDate="{ value }">
-      <overview-due-date v-if="value" :value="value" />
+    <template v-slot:item="{ item }">
+      <v-lazy
+        v-model="itemVisibility[item.id]"
+        @click="selectActivity(item.id)"
+        :class="{ selected: isActivitySelected(item.id) }"
+        tag="tr"
+        style="height: 48px;">
+        <div style="display: contents;">
+          <td class="text-left">
+            <v-col class="d-flex text-left">
+              <overview-name :value="item.name" />
+              <publishing-badge v-if="isAdmin || isRepositoryAdmin" :activity="item" />
+            </v-col>
+          </td>
+          <td class="text-left">
+            <overview-status v-bind="item.status" />
+          </td>
+          <td class="text-left">
+            <overview-assignee v-bind="item.assignee" />
+          </td>
+          <td class="text-left">
+            <overview-priority v-bind="item.priority" />
+          </td>
+          <td class="text-left">
+            <overview-due-date v-if="item.dueDate" :value="item.dueDate" />
+          </td>
+        </div>
+      </v-lazy>
     </template>
   </v-data-table>
 </template>
@@ -45,6 +55,9 @@ export default {
   props: {
     activities: { type: Array, default: () => [] }
   },
+  data: () => ({
+    itemVisibility: {}
+  }),
   computed: {
     ...mapGetters(['isAdmin']),
     ...mapGetters('repository', ['isRepositoryAdmin', 'workflow']),
@@ -116,12 +129,13 @@ export default {
 </script>
 
 <style lang="scss" scoped>
-$row-background: var(--v-primary-lighten4);
+$row-background: var(--v-primary-lighten5);
+$row-hover-background: var(--v-primary-lighten4);
 
 .overview ::v-deep {
   thead.v-data-table-header {
     tr th {
-      background: #eceff1;
+      background: $row-hover-background;
     }
   }
 
@@ -133,16 +147,25 @@ $row-background: var(--v-primary-lighten4);
     max-width: 11.5rem;
   }
 
-  tr:hover {
-    background: $row-background !important;
+  .v-lazy {
+    background-color: $row-background;
+    border-bottom: thin solid rgba(0, 0, 0, 0.12);
+
+    &:hover {
+      background-color: $row-hover-background !important;
+    }
+
+    &.selected {
+      background: $row-hover-background !important;
+    }
 
     &:not(.selected) {
       cursor: pointer;
     }
-  }
 
-  .selected {
-    background: $row-background;
+    td {
+      height: 48px;
+    }
   }
 }
 </style>