From 09c6e02d863e64d9c61676fb104e45b424cc7995 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Tue, 10 Dec 2024 17:23:06 +0530
Subject: [PATCH] fix: removed detailMode code from Contact modal

---
 .../src/components/Modals/ContactModal.vue    | 136 +++---------------
 .../components/Modals/OrganizationModal.vue   |  26 ++--
 frontend/src/pages/Contacts.vue               |  13 +-
 3 files changed, 35 insertions(+), 140 deletions(-)

diff --git a/frontend/src/components/Modals/ContactModal.vue b/frontend/src/components/Modals/ContactModal.vue
index 8487985ae..a3515e8b8 100644
--- a/frontend/src/components/Modals/ContactModal.vue
+++ b/frontend/src/components/Modals/ContactModal.vue
@@ -10,10 +10,10 @@
           </div>
           <div class="flex items-center gap-1">
             <Button
-              v-if="isManager() || detailMode"
+              v-if="isManager()"
               variant="ghost"
               class="w-7"
-              @click="detailMode ? (detailMode = false) : openQuickEntryModal()"
+              @click="openQuickEntryModal"
             >
               <EditIcon class="h-4 w-4" />
             </Button>
@@ -22,77 +22,36 @@
             </Button>
           </div>
         </div>
-        <div>
-          <div v-if="detailMode" class="flex flex-col gap-3.5">
-            <div
-              v-for="field in detailFields"
-              :key="field.name"
-              class="flex h-7 items-center gap-2 text-base text-ink-gray-8"
-            >
-              <div class="grid w-7 place-content-center">
-                <component :is="field.icon" />
-              </div>
-              <div v-if="field.type == 'dropdown'">
-                <Dropdown
-                  :options="field.options"
-                  class="form-control -ml-2 mr-2 w-full flex-1"
-                >
-                  <template #default="{ open }">
-                    <Button
-                      variant="ghost"
-                      :label="contact.data[field.name]"
-                      class="dropdown-button w-full justify-between truncate hover:bg-surface-white"
-                    >
-                      <div class="truncate">{{ contact.data[field.name] }}</div>
-                      <template #suffix>
-                        <FeatherIcon
-                          :name="open ? 'chevron-up' : 'chevron-down'"
-                          class="h-4 text-ink-gray-5"
-                        />
-                      </template>
-                    </Button>
-                  </template>
-                </Dropdown>
-              </div>
-              <div v-else>{{ field.value }}</div>
-            </div>
-          </div>
-          <FieldLayout
-            v-else-if="filteredSections.length"
-            :tabs="filteredSections"
-            :data="_contact"
-          />
+        <div v-if="filteredSections.length">
+          <FieldLayout :tabs="filteredSections" :data="_contact" />
         </div>
       </div>
-      <div v-if="!detailMode" class="px-4 pb-7 pt-4 sm:px-6">
+      <div class="px-4 pb-7 pt-4 sm:px-6">
         <div class="space-y-2">
           <Button
             class="w-full"
             v-for="action in dialogOptions.actions"
             :key="action.label"
             v-bind="action"
-          >
-            {{ __(action.label) }}
-          </Button>
+            :label="__(action.label)"
+          />
         </div>
       </div>
     </template>
   </Dialog>
   <AddressModal v-model="showAddressModal" v-model:address="_address" />
+  <QuickEntryModal
+    v-if="showQuickEntryModal"
+    v-model="showQuickEntryModal"
+    doctype="Contact"
+  />
 </template>
 
 <script setup>
+import QuickEntryModal from '@/components/Modals/QuickEntryModal.vue'
 import FieldLayout from '@/components/FieldLayout.vue'
 import AddressModal from '@/components/Modals/AddressModal.vue'
-import ContactIcon from '@/components/Icons/ContactIcon.vue'
-import GenderIcon from '@/components/Icons/GenderIcon.vue'
-import Email2Icon from '@/components/Icons/Email2Icon.vue'
-import PhoneIcon from '@/components/Icons/PhoneIcon.vue'
-import OrganizationsIcon from '@/components/Icons/OrganizationsIcon.vue'
-import AddressIcon from '@/components/Icons/AddressIcon.vue'
-import CertificateIcon from '@/components/Icons/CertificateIcon.vue'
 import EditIcon from '@/components/Icons/EditIcon.vue'
-import Dropdown from '@/components/frappe-ui/Dropdown.vue'
 import { usersStore } from '@/stores/users'
 import { capture } from '@/telemetry'
 import { call, createResource } from 'frappe-ui'
@@ -108,7 +67,6 @@ const props = defineProps({
     type: Object,
     default: {
       redirect: true,
-      detailMode: false,
       afterInsert: () => {},
     },
   },
@@ -119,7 +77,6 @@ const { isManager } = usersStore()
 const router = useRouter()
 const show = defineModel()
 
-const detailMode = ref(false)
 const editMode = ref(false)
 let _contact = ref({})
 let _address = ref({})
@@ -186,63 +143,17 @@ function handleContactUpdate(doc) {
 const dialogOptions = computed(() => {
   let title = !editMode.value ? 'New Contact' : _contact.value.full_name
 
-  let size = detailMode.value ? '' : 'xl'
-  let actions = detailMode.value
-    ? []
-    : [
-        {
-          label: editMode.value ? 'Save' : 'Create',
-          variant: 'solid',
-          disabled: !dirty.value,
-          onClick: () => (editMode.value ? updateContact() : callInsertDoc()),
-        },
-      ]
-
-  return { title, size, actions }
-})
-
-const detailFields = computed(() => {
-  let details = [
-    {
-      icon: ContactIcon,
-      name: 'full_name',
-      value:
-        (_contact.value.salutation ? _contact.value.salutation + '. ' : '') +
-        _contact.value.full_name,
-    },
-    {
-      icon: GenderIcon,
-      name: 'gender',
-      value: _contact.value.gender,
-    },
+  let size = 'xl'
+  let actions = [
     {
-      icon: Email2Icon,
-      name: 'email_id',
-      value: _contact.value.email_id,
-    },
-    {
-      icon: PhoneIcon,
-      name: 'mobile_no',
-      value: _contact.value.actual_mobile_no,
-    },
-    {
-      icon: OrganizationsIcon,
-      name: 'company_name',
-      value: _contact.value.company_name,
-    },
-    {
-      icon: CertificateIcon,
-      name: 'designation',
-      value: _contact.value.designation,
-    },
-    {
-      icon: AddressIcon,
-      name: 'address',
-      value: _contact.value.address,
+      label: editMode.value ? 'Save' : 'Create',
+      variant: 'solid',
+      disabled: !dirty.value,
+      onClick: () => (editMode.value ? updateContact() : callInsertDoc()),
     },
   ]
 
-  return details.filter((detail) => detail.value)
+  return { title, size, actions }
 })
 
 const tabs = createResource({
@@ -287,7 +198,6 @@ watch(
   () => show.value,
   (value) => {
     if (!value) return
-    detailMode.value = props.options.detailMode
     editMode.value = false
     nextTick(() => {
       _contact.value = { ...props.contact.data }
@@ -298,13 +208,11 @@ watch(
   },
 )
 
-const showQuickEntryModal = defineModel('quickEntry')
+const showQuickEntryModal = ref(false)
 
 function openQuickEntryModal() {
   showQuickEntryModal.value = true
-  nextTick(() => {
-    show.value = false
-  })
+  nextTick(() => (show.value = false))
 }
 </script>
 
diff --git a/frontend/src/components/Modals/OrganizationModal.vue b/frontend/src/components/Modals/OrganizationModal.vue
index 8f62b587d..585b9b78a 100644
--- a/frontend/src/components/Modals/OrganizationModal.vue
+++ b/frontend/src/components/Modals/OrganizationModal.vue
@@ -22,23 +22,21 @@
             </Button>
           </div>
         </div>
-        <div>
-          <FieldLayout
-            v-if="filteredSections.length"
-            :tabs="filteredSections"
-            :data="_organization"
-          />
+        <div v-if="filteredSections.length">
+          <FieldLayout :tabs="filteredSections" :data="_organization" />
         </div>
       </div>
       <div class="px-4 pb-7 pt-4 sm:px-6">
-        <Button
-          class="w-full"
-          v-for="action in dialogOptions.actions"
-          :key="action.label"
-          v-bind="action"
-          :label="__(action.label)"
-          :loading="loading"
-        />
+        <div class="space-y-2">
+          <Button
+            class="w-full"
+            v-for="action in dialogOptions.actions"
+            :key="action.label"
+            v-bind="action"
+            :label="__(action.label)"
+            :loading="loading"
+          />
+        </div>
       </div>
     </template>
   </Dialog>
diff --git a/frontend/src/pages/Contacts.vue b/frontend/src/pages/Contacts.vue
index 30ed45517..dca88f309 100644
--- a/frontend/src/pages/Contacts.vue
+++ b/frontend/src/pages/Contacts.vue
@@ -59,16 +59,7 @@
       </Button>
     </div>
   </div>
-  <ContactModal
-    v-model="showContactModal"
-    v-model:quickEntry="showQuickEntryModal"
-    :contact="{}"
-  />
-  <QuickEntryModal
-    v-if="showQuickEntryModal"
-    v-model="showQuickEntryModal"
-    doctype="Contact"
-  />
+  <ContactModal v-model="showContactModal" :contact="{}" />
 </template>
 
 <script setup>
@@ -77,7 +68,6 @@ import CustomActions from '@/components/CustomActions.vue'
 import ContactsIcon from '@/components/Icons/ContactsIcon.vue'
 import LayoutHeader from '@/components/LayoutHeader.vue'
 import ContactModal from '@/components/Modals/ContactModal.vue'
-import QuickEntryModal from '@/components/Modals/QuickEntryModal.vue'
 import ContactsListView from '@/components/ListViews/ContactsListView.vue'
 import ViewControls from '@/components/ViewControls.vue'
 import { organizationsStore } from '@/stores/organizations.js'
@@ -87,7 +77,6 @@ import { ref, computed } from 'vue'
 const { getOrganization } = organizationsStore()
 
 const showContactModal = ref(false)
-const showQuickEntryModal = ref(false)
 
 const contactsListView = ref(null)