From 564d2b1ac5bdee71d4b54b6ef672cd08ae2a87a1 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Fri, 1 Aug 2025 17:37:02 +0530
Subject: [PATCH] refactor: update AssignTo component to use correct doctype
 for CRM Deal

---
 frontend/src/components/AssignTo.vue     | 16 +++++++++-------
 frontend/src/components/AssignToBody.vue |  8 ++------
 frontend/src/pages/Deal.vue              |  2 +-
 frontend/src/pages/MobileDeal.vue        |  2 +-
 4 files changed, 13 insertions(+), 15 deletions(-)

diff --git a/frontend/src/components/AssignTo.vue b/frontend/src/components/AssignTo.vue
index 6e9e2bdc..96d29cb6 100644
--- a/frontend/src/components/AssignTo.vue
+++ b/frontend/src/components/AssignTo.vue
@@ -1,13 +1,15 @@
 <template>
   <NestedPopover>
     <template #target>
-      <component
-        v-if="assignees?.length"
-        :is="assignees?.length == 1 ? 'Button' : 'div'"
-      >
-        <MultipleAvatar :avatars="assignees" />
-      </component>
-      <Button v-else :label="__('Assign to')" />
+      <div class="flex items-center">
+        <component
+          v-if="assignees?.length"
+          :is="assignees?.length == 1 ? 'Button' : 'div'"
+        >
+          <MultipleAvatar :avatars="assignees" />
+        </component>
+        <Button v-else :label="__('Assign to')" />
+      </div>
     </template>
     <template #body="{ open }">
       <AssignToBody
diff --git a/frontend/src/components/AssignToBody.vue b/frontend/src/components/AssignToBody.vue
index 675417cf..e4e226b0 100644
--- a/frontend/src/components/AssignToBody.vue
+++ b/frontend/src/components/AssignToBody.vue
@@ -1,6 +1,6 @@
 <template>
   <div
-    class="flex flex-col gap-2 my-2 w-96 rounded-lg bg-surface-modal shadow-2xl ring-1 ring-black p-3 ring-opacity-5 focus:outline-none"
+    class="flex flex-col gap-2 my-2 w-[470px] rounded-lg bg-surface-modal shadow-2xl ring-1 ring-black p-3 ring-opacity-5 focus:outline-none"
   >
     <div class="text-base text-ink-gray-5">{{ __('Assign to') }}</div>
     <Link
@@ -27,7 +27,7 @@
           >
             <div>
               <div
-                class="flex items-center text-sm text-ink-gray-6 border border-outline-gray-1 bg-surface-white rounded-full hover:bg-surface-white !p-0.5"
+                class="flex items-center text-sm p-0.5 text-ink-gray-6 border border-outline-gray-1 bg-surface-modal rounded-full cursor-pointer"
                 @click.stop
               >
                 <UserAvatar :user="assignee.name" size="sm" />
@@ -198,7 +198,6 @@ const addAssignees = createResource({
   }),
   onSuccess: () => {
     capture('assign_to', { doctype: props.doctype })
-    toast.success(__('Assignees added successfully.'))
   },
 })
 
@@ -209,8 +208,5 @@ const removeAssignees = createResource({
     name: props.docname,
     assignees: removedAssignees,
   }),
-  onSuccess: () => {
-    toast.success(__('Assignees removed successfully.'))
-  },
 })
 </script>
diff --git a/frontend/src/pages/Deal.vue b/frontend/src/pages/Deal.vue
index 80212a72..e11bff77 100644
--- a/frontend/src/pages/Deal.vue
+++ b/frontend/src/pages/Deal.vue
@@ -16,7 +16,7 @@
         v-if="document.actions?.length"
         :actions="document.actions"
       />
-      <AssignTo v-model="assignees.data" doctype="CRM Lead" :docname="dealId" />
+      <AssignTo v-model="assignees.data" doctype="CRM Deal" :docname="dealId" />
       <Dropdown
         v-if="doc"
         :options="
diff --git a/frontend/src/pages/MobileDeal.vue b/frontend/src/pages/MobileDeal.vue
index ca709b03..32d6b4c9 100644
--- a/frontend/src/pages/MobileDeal.vue
+++ b/frontend/src/pages/MobileDeal.vue
@@ -42,7 +42,7 @@
     v-if="doc.name"
     class="flex h-12 items-center justify-between gap-2 border-b px-3 py-2.5"
   >
-    <AssignTo v-model="assignees.data" doctype="CRM Lead" :docname="dealId" />
+    <AssignTo v-model="assignees.data" doctype="CRM Deal" :docname="dealId" />
     <div class="flex items-center gap-2">
       <CustomActions
         v-if="document._actions?.length"