From e5be0633412de1590c38888afaefa45ac93ffdeb Mon Sep 17 00:00:00 2001
From: Vigneshkumar Chinnachamy M <chinnachamy.vignesh@gmail.com>
Date: Wed, 28 Aug 2019 22:40:29 +0530
Subject: [PATCH] fix alerts in profile settings

---
 .../components/ProfileSettingsForm.jsx        | 29 +++++++++++++++++--
 .../components/ProfileSettingsForm.scss       | 15 ++++++++--
 2 files changed, 38 insertions(+), 6 deletions(-)

diff --git a/src/routes/settings/routes/profile/components/ProfileSettingsForm.jsx b/src/routes/settings/routes/profile/components/ProfileSettingsForm.jsx
index 9ac27ddc2..098bbc7ab 100644
--- a/src/routes/settings/routes/profile/components/ProfileSettingsForm.jsx
+++ b/src/routes/settings/routes/profile/components/ProfileSettingsForm.jsx
@@ -28,6 +28,8 @@ class ProfileSettingsForm extends Component {
       dirty: false,
       businessPhoneValid: true,
       countrySelected: null,
+      businessPhoneDirty: false,
+      countrySelectionDirty: false
     }
     this.onSubmit = this.onSubmit.bind(this)
     this.onValid = this.onValid.bind(this)
@@ -42,11 +44,12 @@ class ProfileSettingsForm extends Component {
     if (country && country.value && this.state.countrySelected !== country.value) {
       this.setState({
         countrySelected: country.value,
+        countrySelectionDirty: true
       })
     }
   }
 
-  onBusinessPhoneCountryChange({ country }) {
+  onBusinessPhoneCountryChange({ country, externalChange }) {
     const { businessPhoneValid } = this.state
 
     if (country && country.code) {
@@ -68,6 +71,15 @@ class ProfileSettingsForm extends Component {
         businessPhoneValid: false
       })
     }
+
+    // external change means, the user didn't change the phone number field.
+    // But it was automatically changed due to country selection change. In such case, we should show
+    // the alert under country selection only.
+    if (!externalChange) {
+      this.setState({
+        businessPhoneDirty: true
+      })
+    }
   }
 
   getField(label, name, isRequired=false) {
@@ -108,6 +120,11 @@ class ProfileSettingsForm extends Component {
       ...data,
     }
     this.props.saveSettings(updatedData)
+
+    this.setState({
+      businessPhoneDirty: false,
+      countrySelectionDirty: false
+    })
   }
 
   onValid() {
@@ -167,7 +184,10 @@ class ProfileSettingsForm extends Component {
               value={this.props.values.settings.businessPhone}
               onChangeCountry={this.onBusinessPhoneCountryChange}
             />
-            <div styleName="warningText">Note: Changing the country code also updates your country selection</div>
+            {
+              this.state.businessPhoneDirty &&
+              <div styleName="warningText">Note: Changing the country code also updates your country selection</div>
+            }
           </div>
         </div>
         {this.getField('Company name', 'companyName', true)}
@@ -219,7 +239,10 @@ class ProfileSettingsForm extends Component {
               showDropdownIndicator
               setValueOnly
             />
-            <div styleName="warningText">Note: Changing the country also updates the country code of business phone.</div>
+            {
+              this.state.countrySelectionDirty &&
+              <div styleName="warningText">Note: Changing the country also updates the country code of business phone.</div>
+            }
           </div>
         </div>
         <div className="controls">
diff --git a/src/routes/settings/routes/profile/components/ProfileSettingsForm.scss b/src/routes/settings/routes/profile/components/ProfileSettingsForm.scss
index 7e020ac8f..1e448c3c9 100644
--- a/src/routes/settings/routes/profile/components/ProfileSettingsForm.scss
+++ b/src/routes/settings/routes/profile/components/ProfileSettingsForm.scss
@@ -173,7 +173,16 @@
 }
 
 .warningText {
-  font-size: 12px;
-  margin-top: 6px;
-  color: $tc-red-100;
+  margin: 5px auto;
+  @include roboto;
+
+  color: $tc-gray-70;
+  font-size: 13px;
+  line-height:20px;
+  font-style:italic;
+  border: 1px solid $tc-red-30;
+  background: $tc-red-10;
+  color: $tc-red-70;
+  padding:10px;
+  border-radius: 2px;
 }
\ No newline at end of file