From 122207f5804074effbb4d2506952b0120f533593 Mon Sep 17 00:00:00 2001
From: wjames111 <william.james@cru.org>
Date: Mon, 13 Jan 2025 12:46:06 -0500
Subject: [PATCH 1/3] Fix codestyle in emailField.tpl.html.

---
 .../contactInfo/emailField/emailField.tpl.html | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/common/components/contactInfo/emailField/emailField.tpl.html b/src/common/components/contactInfo/emailField/emailField.tpl.html
index 0c21433ce..f1af68f71 100644
--- a/src/common/components/contactInfo/emailField/emailField.tpl.html
+++ b/src/common/components/contactInfo/emailField/emailField.tpl.html
@@ -1,11 +1,11 @@
 <div class="form-group is-required" ng-class="{'has-error': ($ctrl.detailsForm.email | showErrors)}">
-	<label>
-		<span translate>{{'EMAIL'}}</span>
-		<input type="email" class="form-control  form-control-subtle" name="email" ng-model="$ctrl.donorDetails['email']" required ng-maxlength="100" ng-disabled="$ctrl.donorDetails.staff">
-	</label>
-	<div role="alert" ng-messages="$ctrl.detailsForm.email.$error" ng-if="($ctrl.detailsForm.email | showErrors)">
-		<div class="help-block" ng-message="required" translate>{{'EMAIL_MISSING_ERROR'}}</div>
-		<div class="help-block" ng-message="email" translate>{{'EMAIL_INVALID_ERROR'}}</div>
-		<div class="help-block" ng-message="maxlength" translate>{{'EMAIL_LENGTH_ERROR'}}</div>
-	</div>
+  <label>
+    <span translate>{{'EMAIL'}}</span>
+    <input type="email" class="form-control  form-control-subtle" name="email" ng-model="$ctrl.donorDetails['email']" required ng-maxlength="100" ng-disabled="$ctrl.donorDetails.staff">
+  </label>
+  <div role="alert" ng-messages="$ctrl.detailsForm.email.$error" ng-if="($ctrl.detailsForm.email | showErrors)">
+    <div class="help-block" ng-message="required" translate>{{'EMAIL_MISSING_ERROR'}}</div>
+    <div class="help-block" ng-message="email" translate>{{'EMAIL_INVALID_ERROR'}}</div>
+    <div class="help-block" ng-message="maxlength" translate>{{'EMAIL_LENGTH_ERROR'}}</div>
+  </div>
 </div>

From 67586aa1b177a527e411e21b2f8630f172f2c7ab Mon Sep 17 00:00:00 2001
From: wjames111 <william.james@cru.org>
Date: Mon, 13 Jan 2025 12:57:25 -0500
Subject: [PATCH 2/3] fix styling for first and last name.

---
 .../step-1/branded-checkout-step-1.tpl.html   | 22 +++++++++----------
 .../contactInfo/contactInfo.tpl.html          | 19 ++++++++--------
 2 files changed, 21 insertions(+), 20 deletions(-)

diff --git a/src/app/branded/step-1/branded-checkout-step-1.tpl.html b/src/app/branded/step-1/branded-checkout-step-1.tpl.html
index 192ff348e..1dad7589e 100644
--- a/src/app/branded/step-1/branded-checkout-step-1.tpl.html
+++ b/src/app/branded/step-1/branded-checkout-step-1.tpl.html
@@ -8,17 +8,17 @@
     <loading ng-if="$ctrl.loadingProductConfig">
       <translate>Loading gift details...</translate>
     </loading>
-    <product-config-form code="$ctrl.code"
-                         is-edit="$ctrl.isEdit"
-                         item-config="$ctrl.itemConfig"
-                         default-frequency="$ctrl.defaultFrequency"
-                         uri="$ctrl.item.uri"
-                         submitted="$ctrl.submitted"
-                         on-state-change="$ctrl.onGiftConfigStateChange(state)"
-                         disable-session-restart="true"
-                         ng-if="!$ctrl.loadingProductConfig && !$ctrl.errorLoadingProductConfig"
-                         use-v3="$ctrl.useV3"
-                         >
+    <product-config-form 
+      code="$ctrl.code"
+      is-edit="$ctrl.isEdit"
+      item-config="$ctrl.itemConfig"
+      default-frequency="$ctrl.defaultFrequency"
+      uri="$ctrl.item.uri"
+      submitted="$ctrl.submitted"
+      on-state-change="$ctrl.onGiftConfigStateChange(state)"
+      disable-session-restart="true"
+      ng-if="!$ctrl.loadingProductConfig && !$ctrl.errorLoadingProductConfig"
+      use-v3="$ctrl.useV3">
     </product-config-form>
     <div ng-if="$ctrl.errorLoadingProductConfig" class="alert alert-danger" role="alert">
       <p translate='LOADING_ERROR' translate-values="{loadData: '$ctrl.loadData()'}" translate-compile></p>
diff --git a/src/common/components/contactInfo/contactInfo.tpl.html b/src/common/components/contactInfo/contactInfo.tpl.html
index 43c62f9dc..95a2ee40b 100644
--- a/src/common/components/contactInfo/contactInfo.tpl.html
+++ b/src/common/components/contactInfo/contactInfo.tpl.html
@@ -48,17 +48,18 @@
       <h4 ng-if="$ctrl.useV3 !== 'true'" class="panel-title  border-bottom-small  visible" translate>{{'YOUR_INFORMATION'}}</h4>
       <h4 ng-if="$ctrl.useV3 === 'true'" class="panel-title  border-bottom-small  visible" translate>{{'CONTACT_INFO'}}</h4>
       <div class="row">
-        <div class="col-sm-4">
+        <div ng-class="{'col-sm-4': $ctrl.useV3 !== 'true', 'col-sm-6': $ctrl.useV3 === 'true'}">
           <div class="form-group is-required" ng-class="{'has-error': ($ctrl.detailsForm.nameGivenName | showErrors)}">
             <label>
               <span translate>{{'FIRST_NAME'}}</span>
-              <input type="text"
-                      class="form-control  form-control-subtle"
-                      name="nameGivenName"
-                      ng-model="$ctrl.donorDetails['name']['given-name']"
-                      required
-                      ng-maxlength="50"
-                      ng-disabled="$ctrl.nameFieldsDisabled">
+              <input 
+                type="text"
+                class="form-control  form-control-subtle"
+                name="nameGivenName"
+                ng-model="$ctrl.donorDetails['name']['given-name']"
+                required
+                ng-maxlength="50"
+                ng-disabled="$ctrl.nameFieldsDisabled">
             </label>
             <div role="alert" ng-messages="$ctrl.detailsForm.nameGivenName.$error" ng-if="($ctrl.detailsForm.nameGivenName | showErrors)">
               <div class="help-block" ng-message="required" translate>{{'FIRST_NAME_ERROR'}}</div>
@@ -82,7 +83,7 @@ <h4 ng-if="$ctrl.useV3 === 'true'" class="panel-title  border-bottom-small  visi
             </div>
           </div>
         </div>
-        <div class="col-sm-4">
+        <div ng-class="{'col-sm-4': $ctrl.useV3 !== 'true', 'col-sm-6': $ctrl.useV3 === 'true'}">
           <div class="form-group is-required" ng-class="{'has-error': ($ctrl.detailsForm.nameFamilyName | showErrors)}">
             <label>
               <span translate>{{'LAST_NAME'}}</span>

From 015c7faac3056930e6d3343cf2415894efacb69d Mon Sep 17 00:00:00 2001
From: wjames111 <william.james@cru.org>
Date: Mon, 13 Jan 2025 13:00:58 -0500
Subject: [PATCH 3/3] account for no phone number on the form when adding
 custom validators on v3.

---
 src/common/components/contactInfo/contactInfo.component.js | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/common/components/contactInfo/contactInfo.component.js b/src/common/components/contactInfo/contactInfo.component.js
index 6b523a1d8..f6da5d8aa 100644
--- a/src/common/components/contactInfo/contactInfo.component.js
+++ b/src/common/components/contactInfo/contactInfo.component.js
@@ -70,8 +70,10 @@ class Step1Controller {
   }
 
   addCustomValidators () {
-    this.detailsForm.phoneNumber.$validators.phone = number => {
-      return !number || phoneNumberRegex.test(number)
+    if (this.useV3 !== 'true') {
+      this.detailsForm.phoneNumber.$validators.phone = number => {
+        return !number || phoneNumberRegex.test(number)
+      }
     }
   }