From 16778bc3132f82f41f413c8c84dfd7a6fb707b66 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Mon, 19 Feb 2024 13:55:13 +0530
Subject: [PATCH 1/8] fix: update column width on column resize by dragging

---
 frontend/src/components/ColumnItem.vue     | 51 ++++++++++++++++++++++
 frontend/src/components/ColumnSettings.vue | 33 +++-----------
 2 files changed, 58 insertions(+), 26 deletions(-)
 create mode 100644 frontend/src/components/ColumnItem.vue

diff --git a/frontend/src/components/ColumnItem.vue b/frontend/src/components/ColumnItem.vue
new file mode 100644
index 00000000..b8089d3f
--- /dev/null
+++ b/frontend/src/components/ColumnItem.vue
@@ -0,0 +1,51 @@
+<template>
+  <div
+    class="flex cursor-grab items-center justify-between gap-6 rounded px-2 py-1.5 text-base text-gray-800 hover:bg-gray-100"
+  >
+    <div class="flex items-center gap-2">
+      <DragIcon class="h-3.5" />
+      <div>{{ column.label }}</div>
+    </div>
+    <div class="flex cursor-pointer items-center gap-1">
+      <Button
+        variant="ghost"
+        class="!h-5 w-5 !p-1"
+        @click="emit('edit', column)"
+      >
+        <EditIcon class="h-3.5" />
+      </Button>
+      <Button
+        variant="ghost"
+        class="!h-5 w-5 !p-1"
+        @click="emit('remove', column)"
+      >
+        <FeatherIcon name="x" class="h-3.5" />
+      </Button>
+    </div>
+  </div>
+</template>
+<script setup>
+import EditIcon from '@/components/Icons/EditIcon.vue'
+import DragIcon from '@/components/Icons/DragIcon.vue'
+import { watchDebounced } from '@vueuse/core'
+
+const props = defineProps({
+  column: {
+    type: Object,
+    required: true,
+  },
+})
+
+const emit = defineEmits(['edit', 'remove', 'update'])
+
+watchDebounced(
+  () => props.column.width,
+  (val, old_val) => {
+    val = val.replace(/[^\d.]/g, '')
+    old_val = old_val.replace(/[^\d.]/g, '')
+    if (Math.abs(val - old_val) > 1) return
+    emit('update')
+  },
+  { debounce: 1000 }
+)
+</script>
diff --git a/frontend/src/components/ColumnSettings.vue b/frontend/src/components/ColumnSettings.vue
index 9e619af8..890be9ef 100644
--- a/frontend/src/components/ColumnSettings.vue
+++ b/frontend/src/components/ColumnSettings.vue
@@ -19,30 +19,12 @@
             class="list-group"
           >
             <template #item="{ element }">
-              <div
-                class="flex cursor-grab items-center justify-between gap-6 rounded px-2 py-1.5 text-base text-gray-800 hover:bg-gray-100"
-              >
-                <div class="flex items-center gap-2">
-                  <DragIcon class="h-3.5" />
-                  <div>{{ element.label }}</div>
-                </div>
-                <div class="flex cursor-pointer items-center gap-1">
-                  <Button
-                    variant="ghost"
-                    class="!h-5 w-5 !p-1"
-                    @click="editColumn(element)"
-                  >
-                    <EditIcon class="h-3.5" />
-                  </Button>
-                  <Button
-                    variant="ghost"
-                    class="!h-5 w-5 !p-1"
-                    @click="removeColumn(element)"
-                  >
-                    <FeatherIcon name="x" class="h-3.5" />
-                  </Button>
-                </div>
-              </div>
+              <ColumnItem
+                :column="element"
+                @edit="editColumn"
+                @remove="removeColumn"
+                @update="apply"
+              />
             </template>
           </Draggable>
           <div class="mt-1.5 flex flex-col gap-1 border-t pt-1.5">
@@ -135,8 +117,7 @@
 
 <script setup>
 import ColumnsIcon from '@/components/Icons/ColumnsIcon.vue'
-import EditIcon from '@/components/Icons/EditIcon.vue'
-import DragIcon from '@/components/Icons/DragIcon.vue'
+import ColumnItem from '@/components/ColumnItem.vue'
 import ReloadIcon from '@/components/Icons/ReloadIcon.vue'
 import NestedPopover from '@/components/NestedPopover.vue'
 import Autocomplete from '@/components/frappe-ui/Autocomplete.vue'

From b6f86390a31106fcce28f1e0b706ecd9ea7957aa Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Mon, 19 Feb 2024 14:08:46 +0530
Subject: [PATCH 2/8] fix: disabled tooltip from leads listview

---
 frontend/src/components/ListViews/LeadsListView.vue | 8 ++++++--
 frontend/src/pages/Leads.vue                        | 1 +
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/frontend/src/components/ListViews/LeadsListView.vue b/frontend/src/components/ListViews/LeadsListView.vue
index 1c12e13b..1773365a 100644
--- a/frontend/src/components/ListViews/LeadsListView.vue
+++ b/frontend/src/components/ListViews/LeadsListView.vue
@@ -6,6 +6,7 @@
     :options="{
       getRowRoute: (row) => ({ name: 'Lead', params: { leadId: row.name } }),
       selectable: options.selectable,
+      showTooltip: options.showTooltip,
     }"
     row-key="name"
   >
@@ -57,7 +58,8 @@
               <PhoneIcon class="h-4 w-4" />
             </div>
           </template>
-          <div
+          <Tooltip
+            :text="item.label"
             v-if="
               [
                 'modified',
@@ -70,7 +72,7 @@
             class="truncate text-base"
           >
             {{ item.timeAgo }}
-          </div>
+          </Tooltip>
           <div
             v-else-if="column.key === 'sla_status'"
             class="truncate text-base"
@@ -144,6 +146,7 @@ import {
   ListFooter,
   Dropdown,
   call,
+  Tooltip,
 } from 'frappe-ui'
 import { setupBulkActions, createToast } from '@/utils'
 import { globalStore } from '@/stores/global'
@@ -163,6 +166,7 @@ const props = defineProps({
     type: Object,
     default: () => ({
       selectable: true,
+      showTooltip: true,
       totalCount: 0,
       rowCount: 0,
     }),
diff --git a/frontend/src/pages/Leads.vue b/frontend/src/pages/Leads.vue
index 0b49cfd7..887ef884 100644
--- a/frontend/src/pages/Leads.vue
+++ b/frontend/src/pages/Leads.vue
@@ -23,6 +23,7 @@
     :rows="rows"
     :columns="leads.data.columns"
     :options="{
+      showTooltip: false,
       rowCount: leads.data.row_count,
       totalCount: leads.data.total_count,
     }"

From dd9080a69a7c20aa0a7c3a6a96399c6870d48fc8 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Mon, 19 Feb 2024 14:09:30 +0530
Subject: [PATCH 3/8] fix: disabled tooltip from deals listview

---
 frontend/src/components/ListViews/DealsListView.vue | 8 ++++++--
 frontend/src/pages/Contact.vue                      | 2 +-
 frontend/src/pages/Deals.vue                        | 1 +
 frontend/src/pages/Organization.vue                 | 2 +-
 4 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/frontend/src/components/ListViews/DealsListView.vue b/frontend/src/components/ListViews/DealsListView.vue
index ccf1cb3e..77570cc7 100644
--- a/frontend/src/components/ListViews/DealsListView.vue
+++ b/frontend/src/components/ListViews/DealsListView.vue
@@ -6,6 +6,7 @@
     :options="{
       getRowRoute: (row) => ({ name: 'Deal', params: { dealId: row.name } }),
       selectable: options.selectable,
+      showTooltip: options.showTooltip,
     }"
     row-key="name"
   >
@@ -48,7 +49,8 @@
               <PhoneIcon class="h-4 w-4" />
             </div>
           </template>
-          <div
+          <Tooltip
+            :text="item.label"
             v-if="
               [
                 'modified',
@@ -61,7 +63,7 @@
             class="truncate text-base"
           >
             {{ item.timeAgo }}
-          </div>
+          </Tooltip>
           <div
             v-else-if="column.key === 'sla_status'"
             class="truncate text-base"
@@ -135,6 +137,7 @@ import {
   ListFooter,
   Dropdown,
   call,
+  Tooltip,
 } from 'frappe-ui'
 import { setupBulkActions, createToast } from '@/utils'
 import { globalStore } from '@/stores/global'
@@ -154,6 +157,7 @@ const props = defineProps({
     type: Object,
     default: () => ({
       selectable: true,
+      showTooltip: true,
       totalCount: 0,
       rowCount: 0,
     }),
diff --git a/frontend/src/pages/Contact.vue b/frontend/src/pages/Contact.vue
index 7b219b20..1b9f6c80 100644
--- a/frontend/src/pages/Contact.vue
+++ b/frontend/src/pages/Contact.vue
@@ -175,7 +175,7 @@
           class="mt-4"
           :rows="rows"
           :columns="columns"
-          :options="{ selectable: false }"
+          :options="{ selectable: false, showTooltip: false }"
         />
         <div
           v-if="!rows.length"
diff --git a/frontend/src/pages/Deals.vue b/frontend/src/pages/Deals.vue
index ee5fa0c6..cc6f25f5 100644
--- a/frontend/src/pages/Deals.vue
+++ b/frontend/src/pages/Deals.vue
@@ -22,6 +22,7 @@
     :rows="rows"
     :columns="deals.data.columns"
     :options="{
+      showTooltip: false,
       rowCount: deals.data.row_count,
       totalCount: deals.data.total_count,
     }"
diff --git a/frontend/src/pages/Organization.vue b/frontend/src/pages/Organization.vue
index 0e988c28..dec6b871 100644
--- a/frontend/src/pages/Organization.vue
+++ b/frontend/src/pages/Organization.vue
@@ -184,7 +184,7 @@
           v-if="tab.label === 'Deals' && rows.length"
           :rows="rows"
           :columns="columns"
-          :options="{ selectable: false }"
+          :options="{ selectable: false, showTooltip: false }"
         />
         <ContactsListView
           class="mt-4"

From 175d711abc310b05e0376dfb375fadf211022ee0 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Mon, 19 Feb 2024 14:14:19 +0530
Subject: [PATCH 4/8] fix: disabled tooltip from contacts listview

---
 .../src/components/ListViews/ContactsListView.vue  | 14 +++++++++++---
 frontend/src/pages/Contacts.vue                    |  1 +
 frontend/src/pages/Organization.vue                |  2 +-
 3 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/frontend/src/components/ListViews/ContactsListView.vue b/frontend/src/components/ListViews/ContactsListView.vue
index fe234620..ea3c0781 100644
--- a/frontend/src/components/ListViews/ContactsListView.vue
+++ b/frontend/src/components/ListViews/ContactsListView.vue
@@ -9,6 +9,7 @@
         params: { contactId: row.name },
       }),
       selectable: options.selectable,
+      showTooltip: options.showTooltip,
     }"
     row-key="name"
   >
@@ -45,12 +46,13 @@
               <PhoneIcon class="h-4 w-4" />
             </div>
           </template>
-          <div
+          <Tooltip
+            :text="item.label"
             v-if="['modified', 'creation'].includes(column.key)"
             class="truncate text-base"
           >
             {{ item.timeAgo }}
-          </div>
+          </Tooltip>
           <div v-else-if="column.type === 'Check'">
             <FormControl
               type="checkbox"
@@ -64,7 +66,11 @@
     </ListRows>
     <ListSelectBanner>
       <template #actions="{ selections, unselectAll }">
-        <Button variant="subtle" label="Edit" @click="editValues(selections, unselectAll)">
+        <Button
+          variant="subtle"
+          label="Edit"
+          @click="editValues(selections, unselectAll)"
+        >
           <template #prefix>
             <EditIcon class="h-3 w-3" />
           </template>
@@ -103,6 +109,7 @@ import {
   ListSelectBanner,
   ListRowItem,
   ListFooter,
+  Tooltip,
 } from 'frappe-ui'
 import { ref, watch } from 'vue'
 
@@ -119,6 +126,7 @@ const props = defineProps({
     type: Object,
     default: () => ({
       selectable: true,
+      showTooltip: true,
       totalCount: 0,
       rowCount: 0,
     }),
diff --git a/frontend/src/pages/Contacts.vue b/frontend/src/pages/Contacts.vue
index 16224913..54d40484 100644
--- a/frontend/src/pages/Contacts.vue
+++ b/frontend/src/pages/Contacts.vue
@@ -21,6 +21,7 @@
     :rows="rows"
     :columns="contacts.data.columns"
     :options="{
+      showTooltip: false,
       rowCount: contacts.data.row_count,
       totalCount: contacts.data.total_count,
     }"
diff --git a/frontend/src/pages/Organization.vue b/frontend/src/pages/Organization.vue
index dec6b871..4e57e2a2 100644
--- a/frontend/src/pages/Organization.vue
+++ b/frontend/src/pages/Organization.vue
@@ -191,7 +191,7 @@
           v-if="tab.label === 'Contacts' && rows.length"
           :rows="rows"
           :columns="columns"
-          :options="{ selectable: false }"
+          :options="{ selectable: false, showTooltip: false, }"
         />
         <div
           v-if="!rows.length"

From 464a6d5098e178810f9b70b07617531f6174c266 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Mon, 19 Feb 2024 14:15:50 +0530
Subject: [PATCH 5/8] fix: disabled tooltip from organizations listview

---
 .../components/ListViews/OrganizationsListView.vue | 14 +++++++++++---
 frontend/src/pages/Organizations.vue               |  1 +
 2 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/frontend/src/components/ListViews/OrganizationsListView.vue b/frontend/src/components/ListViews/OrganizationsListView.vue
index 9dae1671..c0beabd6 100644
--- a/frontend/src/components/ListViews/OrganizationsListView.vue
+++ b/frontend/src/components/ListViews/OrganizationsListView.vue
@@ -8,6 +8,7 @@
         params: { organizationId: row.name },
       }),
       selectable: options.selectable,
+      showTooltip: options.showTooltip,
     }"
     row-key="name"
   >
@@ -32,12 +33,13 @@
               />
             </div>
           </template>
-          <div
+          <Tooltip
+            :text="item.label"
             v-if="['modified', 'creation'].includes(column.key)"
             class="truncate text-base"
           >
             {{ item.timeAgo }}
-          </div>
+          </Tooltip>
           <div v-else-if="column.type === 'Check'">
             <FormControl
               type="checkbox"
@@ -51,7 +53,11 @@
     </ListRows>
     <ListSelectBanner>
       <template #actions="{ selections, unselectAll }">
-        <Button variant="subtle" label="Edit" @click="editValues(selections, unselectAll)">
+        <Button
+          variant="subtle"
+          label="Edit"
+          @click="editValues(selections, unselectAll)"
+        >
           <template #prefix>
             <EditIcon class="h-3 w-3" />
           </template>
@@ -88,6 +94,7 @@ import {
   ListSelectBanner,
   ListRowItem,
   ListFooter,
+  Tooltip,
 } from 'frappe-ui'
 import { ref, watch } from 'vue'
 
@@ -104,6 +111,7 @@ const props = defineProps({
     type: Object,
     default: () => ({
       selectable: true,
+      showTooltip: true,
       totalCount: 0,
       rowCount: 0,
     }),
diff --git a/frontend/src/pages/Organizations.vue b/frontend/src/pages/Organizations.vue
index aac0de85..d05f89a7 100644
--- a/frontend/src/pages/Organizations.vue
+++ b/frontend/src/pages/Organizations.vue
@@ -25,6 +25,7 @@
     :rows="rows"
     :columns="organizations.data.columns"
     :options="{
+      showTooltip: false,
       rowCount: organizations.data.row_count,
       totalCount: organizations.data.total_count,
     }"

From 75eae614923fd01c989920f98c4af90f4ce825c3 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Mon, 19 Feb 2024 14:18:43 +0530
Subject: [PATCH 6/8] fix: disabled tooltip from tasks listview

---
 frontend/src/components/ListViews/TasksListView.vue | 7 +++++--
 frontend/src/pages/Tasks.vue                        | 1 +
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/frontend/src/components/ListViews/TasksListView.vue b/frontend/src/components/ListViews/TasksListView.vue
index 958af21b..2e74a13d 100644
--- a/frontend/src/components/ListViews/TasksListView.vue
+++ b/frontend/src/components/ListViews/TasksListView.vue
@@ -5,6 +5,7 @@
     :options="{
       onRowClick: (row) => emit('showTask', row.name),
       selectable: options.selectable,
+      showTooltip: options.showTooltip,
     }"
     row-key="name"
   >
@@ -47,12 +48,13 @@
               />
             </div>
           </template>
-          <div
+          <Tooltip
+            :text="item.label"
             v-if="['modified', 'creation'].includes(column.key)"
             class="truncate text-base"
           >
             {{ item.timeAgo }}
-          </div>
+          </Tooltip>
           <div v-else-if="column.type === 'Check'">
             <FormControl
               type="checkbox"
@@ -138,6 +140,7 @@ const props = defineProps({
     type: Object,
     default: () => ({
       selectable: true,
+      showTooltip: true,
       totalCount: 0,
       rowCount: 0,
     }),
diff --git a/frontend/src/pages/Tasks.vue b/frontend/src/pages/Tasks.vue
index 2f6335ed..70b54ca0 100644
--- a/frontend/src/pages/Tasks.vue
+++ b/frontend/src/pages/Tasks.vue
@@ -21,6 +21,7 @@
     :rows="rows"
     :columns="tasks.data.columns"
     :options="{
+      showTooltip: false,
       rowCount: tasks.data.row_count,
       totalCount: tasks.data.total_count,
     }"

From 966ab3ef8d0f53d256761dca871ea1dd21b9f711 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Mon, 19 Feb 2024 14:20:06 +0530
Subject: [PATCH 7/8] fix: disabled tooltip from call logs listview

---
 frontend/src/components/ListViews/CallLogsListView.vue | 8 ++++++--
 frontend/src/pages/CallLogs.vue                        | 1 +
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/frontend/src/components/ListViews/CallLogsListView.vue b/frontend/src/components/ListViews/CallLogsListView.vue
index ca9f9078..33f0c3c2 100644
--- a/frontend/src/components/ListViews/CallLogsListView.vue
+++ b/frontend/src/components/ListViews/CallLogsListView.vue
@@ -8,6 +8,7 @@
         params: { callLogId: row.name },
       }),
       selectable: options.selectable,
+      showTooltip: options.showTooltip,
     }"
     row-key="name"
   >
@@ -35,12 +36,13 @@
               <FeatherIcon :name="item.icon" class="h-3 w-3" />
             </div>
           </template>
-          <div
+          <Tooltip
+            :text="item.label"
             v-if="['modified', 'creation'].includes(column.key)"
             class="truncate text-base"
           >
             {{ item.timeAgo }}
-          </div>
+          </Tooltip>
           <div v-else-if="column.key === 'status'" class="truncate text-base">
             <Badge
               :variant="'subtle'"
@@ -82,6 +84,7 @@ import {
   ListSelectBanner,
   ListRowItem,
   ListFooter,
+  Tooltip,
 } from 'frappe-ui'
 import { watch } from 'vue'
 
@@ -98,6 +101,7 @@ const props = defineProps({
     type: Object,
     default: () => ({
       selectable: true,
+      showTooltip: true,
       totalCount: 0,
       rowCount: 0,
     }),
diff --git a/frontend/src/pages/CallLogs.vue b/frontend/src/pages/CallLogs.vue
index 7a2923d3..7a188d81 100644
--- a/frontend/src/pages/CallLogs.vue
+++ b/frontend/src/pages/CallLogs.vue
@@ -16,6 +16,7 @@
     :rows="rows"
     :columns="callLogs.data.columns"
     :options="{
+      showTooltip: false,
       rowCount: callLogs.data.row_count,
       totalCount: callLogs.data.total_count,
     }"

From 4569fa82fe04c30c1d7172138503c7910340d533 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Mon, 19 Feb 2024 14:22:26 +0530
Subject: [PATCH 8/8] fix: disabled tooltip from email templates listview

---
 .../src/components/ListViews/EmailTemplatesListView.vue   | 8 ++++++--
 frontend/src/pages/EmailTemplates.vue                     | 1 +
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/frontend/src/components/ListViews/EmailTemplatesListView.vue b/frontend/src/components/ListViews/EmailTemplatesListView.vue
index a424d05b..77ab123b 100644
--- a/frontend/src/components/ListViews/EmailTemplatesListView.vue
+++ b/frontend/src/components/ListViews/EmailTemplatesListView.vue
@@ -5,6 +5,7 @@
     :options="{
       onRowClick: (row) => emit('showEmailTemplate', row.name),
       selectable: options.selectable,
+      showTooltip: options.showTooltip,
     }"
     row-key="name"
   >
@@ -21,12 +22,13 @@
           <!-- <template #prefix>
 
           </template> -->
-          <div
+          <Tooltip
+            :text="item.label"
             v-if="['modified', 'creation'].includes(column.key)"
             class="truncate text-base"
           >
             {{ item.timeAgo }}
-          </div>
+          </Tooltip>
           <div v-else-if="column.key === 'status'" class="truncate text-base">
             <Badge
               :variant="'subtle'"
@@ -98,6 +100,7 @@ import {
   ListRowItem,
   ListFooter,
   call,
+  Tooltip,
 } from 'frappe-ui'
 import { ref, watch } from 'vue'
 
@@ -114,6 +117,7 @@ const props = defineProps({
     type: Object,
     default: () => ({
       selectable: true,
+      showTooltip: true,
       totalCount: 0,
       rowCount: 0,
     }),
diff --git a/frontend/src/pages/EmailTemplates.vue b/frontend/src/pages/EmailTemplates.vue
index d74def8f..75b7c148 100644
--- a/frontend/src/pages/EmailTemplates.vue
+++ b/frontend/src/pages/EmailTemplates.vue
@@ -21,6 +21,7 @@
     :rows="rows"
     :columns="emailTemplates.data.columns"
     :options="{
+      showTooltip: false,
       rowCount: emailTemplates.data.row_count,
       totalCount: emailTemplates.data.total_count,
     }"