From d052bba97979630213dda564a93e6715c912c2f0 Mon Sep 17 00:00:00 2001
From: Dima K <dima1086@hotmail.com>
Date: Wed, 28 Jun 2023 21:18:12 -0700
Subject: [PATCH 1/2] Fix editing Home Owners

---
 .../mhrRegistration/HomeOwners/HomeOwnersTable.vue   | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/ppr-ui/src/components/mhrRegistration/HomeOwners/HomeOwnersTable.vue b/ppr-ui/src/components/mhrRegistration/HomeOwners/HomeOwnersTable.vue
index 7dbf7b806..6d42d8777 100644
--- a/ppr-ui/src/components/mhrRegistration/HomeOwners/HomeOwnersTable.vue
+++ b/ppr-ui/src/components/mhrRegistration/HomeOwners/HomeOwnersTable.vue
@@ -27,7 +27,7 @@
             <div
               v-if="showGroups && !(disableGroupHeader(group.groupId) && (hideRemovedOwners || isReadonlyTable))"
               :colspan="4"
-              class="py-3 px-7 pl-0 group-header-slot"
+              class="py-3 px-7 group-header-slot"
               :class="{
                 'spacer-header': disableGroupHeader(group.groupId),
                 'border-error-left': isInvalidOwnerGroup(group.groupId)
@@ -71,7 +71,7 @@
                 >
                   <div
                     class="error-text my-6 text-center"
-                    :data-test-id="`no-owners-msg-group-${group.owners.indexOf(item)}`"
+                    :data-test-id="`no-owners-msg-group-${homeOwners.indexOf(item)}`"
                   >
                     <HomeOwnersGroupError :groupId="group.groupId" />
                   </div>
@@ -85,7 +85,7 @@
                 />
               </tr>
 
-              <template v-if="isCurrentlyEditing(group.owners.indexOf(item))">
+              <template v-if="isCurrentlyEditing(homeOwners.indexOf(item))">
                 <div class="pa-0" :colspan="homeOwnersTableHeaders.length">
                   <v-expand-transition>
                     <AddEditHomeOwner
@@ -101,7 +101,7 @@
               </template>
 
                 <tr v-else-if="item.ownerId"
-                  :key="`owner-row-key-${group.owners.indexOf(item)}`"
+                  :key="`owner-row-key-${homeOwners.indexOf(item)}`"
                   class="owner-info"
                   :data-test-id="`owner-info-${item.ownerId}`"
                 >
@@ -186,7 +186,7 @@
                         class="mr-n4"
                         :ripple="false"
                         :disabled="isAddingMode || isEditingMode || isGlobalEditingMode"
-                        @click="openForEditing(group.owners.indexOf(item))"
+                        @click="openForEditing(homeOwners.indexOf(item))"
                         data-test-id="table-edit-btn"
                       >
                         <v-icon small>mdi-pencil</v-icon>
@@ -283,7 +283,7 @@
                           <v-list class="actions-dropdown actions__more-actions">
                             <!-- Menu Edit Option -->
                             <v-list-item class="my-n2">
-                              <v-list-item-subtitle class="pa-0" @click="openForEditing(group.owners.indexOf(item))">
+                              <v-list-item-subtitle class="pa-0" @click="openForEditing(homeOwners.indexOf(item))">
                                 <v-icon small class="mb-1">mdi-pencil</v-icon>
                                 <span class="ml-1 remove-btn-text">Change Details</span>
                               </v-list-item-subtitle>

From 2a01bd7f9b2b67bde29386f1ab2138c54cb08792 Mon Sep 17 00:00:00 2001
From: Dima K <dima1086@hotmail.com>
Date: Fri, 30 Jun 2023 14:46:30 -0700
Subject: [PATCH 2/2] UXA updates for Home Owners table conversion

---
 .../HomeOwners/HomeOwnersTable.vue            | 42 +++++++++++--------
 1 file changed, 24 insertions(+), 18 deletions(-)

diff --git a/ppr-ui/src/components/mhrRegistration/HomeOwners/HomeOwnersTable.vue b/ppr-ui/src/components/mhrRegistration/HomeOwners/HomeOwnersTable.vue
index 6d42d8777..6845512f5 100644
--- a/ppr-ui/src/components/mhrRegistration/HomeOwners/HomeOwnersTable.vue
+++ b/ppr-ui/src/components/mhrRegistration/HomeOwners/HomeOwnersTable.vue
@@ -24,10 +24,27 @@
         <tr v-for="(group, groupIndex) in homeOwnerGroups" :key="`${group}: ${groupIndex}`">
           <td class="pa-0" colspan="4">
             <!-- Start of Home Owner Group -->
+
+            <div v-if="
+                groupIndex === 0 &&
+                isMhrTransfer &&
+                !hasActualOwners(group.owners) &&
+                group.owners.length > 0 &&
+                hasRemovedAllHomeOwnerGroups() &&
+                !isTransferToExecutorProbateWill &&
+                !isTransferToExecutorUnder25Will &&
+                !isTransferToAdminNoWill
+              "
+            >
+              <div class="pa-6 fs-14 text-center no-owners-head-row" data-test-id="no-data-msg">
+                No owners added yet.
+              </div>
+            </div>
+
             <div
               v-if="showGroups && !(disableGroupHeader(group.groupId) && (hideRemovedOwners || isReadonlyTable))"
               :colspan="4"
-              class="py-3 px-7 group-header-slot"
+              class="py-3 group-header-slot"
               :class="{
                 'spacer-header': disableGroupHeader(group.groupId),
                 'border-error-left': isInvalidOwnerGroup(group.groupId)
@@ -45,21 +62,6 @@
             <!-- End of Table Group Header -->
 
             <div v-for="(item, index) in group.owners" :key="`${item}: ${index}`" class="owner-row">
-              <div
-                v-if="
-                  isMhrTransfer &&
-                  !hasActualOwners(group.owners) &&
-                  group.owners.length > 0 &&
-                  hasRemovedAllHomeOwnerGroups() &&
-                  !isTransferToExecutorProbateWill &&
-                  !isTransferToExecutorUnder25Will &&
-                  !isTransferToAdminNoWill
-                "
-              >
-                <div class="pa-6 fs-14 text-center no-owners-head-row" data-test-id="no-data-msg">
-                  No owners added yet.
-                </div>
-              </div>
 
               <!-- Transfer scenario: Display error for groups that 'removed' all owners
               but they still exist in the table -->
@@ -416,8 +418,8 @@
                 </td>
               </tr>
             </div>
-            <div v-if="group.owners.length === 0" class="my-6 text-center" data-test-id="no-data-msg">
-              No owners added yet.
+            <div v-if="group.owners.length === 0" class="error-text my-6 text-center" data-test-id="no-owners-err-msg">
+              Group must contain at least one owner.
             </div>
           </td>
         </tr>
@@ -1024,6 +1026,10 @@ export default defineComponent({
   }
 }
 
+.home-owners-table:not(.review-mode) .group-header-slot {
+    padding: 0 28px;
+}
+
 .v-menu__content {
   cursor: pointer;
 }