Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Confirm password box moves after setting removing focus from set new password box #5189

Closed
isagoico opened this issue Sep 10, 2021 · 3 comments
Assignees

Comments

@isagoico
Copy link

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:

  1. Navigate to settings
  2. Navigate to change password
  3. In New password box enter a password that meets all the requirements.
  4. Without clicking anywhere else, click directly over the confirm password field.

Expected Result:

User should be focused on the confirm password field in just 1 click.

Actual Result:

When removing focus from New Password field the requirement message disappears which causes the confirm password box to move upward and "escapes" the click from the user.

Workaround:

Just have to click on the confirm password box again.

Platform:

Where is this issue occurring?

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

Version Number: 1.0.96-0

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

Notes/Photos/Videos: Any additional supporting documentation

Bug5230848_new_Password_1009.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause
Slack conversation:

View all open jobs on GitHub

@MelvinBot
Copy link

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

@PrashantMangukiya
Copy link
Contributor

@isagoico I think this issue will be taken care via #4995 They are following this pattern for change password form #4995 (comment) At present pr #5130 submitted for that. So once pr merged this issue will be resolved.

@MonilBhavsar
Copy link
Contributor

MonilBhavsar commented Sep 13, 2021

Closing this as it will be resolved by #5130
Thanks @PrashantMangukiya for linking the issue and PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants