[HOLD for payment 2021-12-15] Change password - Extra gap under validation button - Reported by: @parasharrajat

[isagoico]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

Navigate to Settings
Click on profile
Click on add phone number (secondary log in)
Check the button with dev tools

Expected Result:

Button area should only be consistent all around the button

Actual Result:

There's a empty gap below the button.

Workaround:

None needed.

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.1.16-10

Reproducible in staging?: Yes
Reproducible in production?: Yes

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:

Issue reported by: @parasharrajat
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1637865439012200

View all open jobs on GitHub

[MelvinBot]

Triggered auto assignment to @MariaHCD (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[isagoico]

Looks like it's the same issue for this button as - #6460. Created this issue separately, according to @parasharrajat it can be fixed in the same PR fix as the issue mentioned in this comment.

[parasharrajat]

Proposal

1. Remove the extra margin from the button

   App/src/pages/settings/PasswordPage.js

   Line 239 in 85d9a21

   style={[styles.mb2]}

FixedFooter already handles the correct gapping.

---

From the @isagoico Comment,
I think I can solve this into the same PR for #6460 Already Merged but I asked her to create a separate ticket as this was not known to us when 6460 was found. and As I come to know of this later while fixing some other issue. So this should be considered separate (Looking at the history of issues we have).

[MelvinBot]

@MariaHCD Whoops! This issue is 2 days overdue. Let's get this updated quick!

[MelvinBot]

@MariaHCD Huh... This is 4 days overdue. Who can take care of this?

[MelvinBot]

@MariaHCD Still overdue 6 days?! Let's take care of this!

[parasharrajat]

@mallenexpensify Could you please triage this? Maria seems to be OOO.