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

Always show the "I understand my password can't be recovered" disclaimer on password creation screen #15518

Closed
qoqobolo opened this issue Mar 28, 2023 · 3 comments · Fixed by #15650
Assignees

Comments

@qoqobolo
Copy link
Contributor

qoqobolo commented Mar 28, 2023

Bug Report

Followup of #15405

Problem

Currently, according to design, we only show the disclaimer I understand my password can't be recovered when the user sets the cursor to the second input field (Repeat password) .

But if the user after that presses on the first input field (Type password) again for some reason, for example, if they add more characters to the password in the second field first and then back to the first and close the keyboard, then the disclaimer, that is needed to complete this step, won't be shown on the screen, which may be a bit confusing.

video_2023-03-28_14-39-32.mp4

Expected behavior

We should consider the option to show the disclaimer on the screen always from the beginning / always when both password fields are filled

Actual behavior

The disclaimer is only shown after focusing on the second field

Reproduction

  1. Start creating a new account
  2. Enter a password
  3. Repeat it
  4. Then focus back on the Type password field and try to complete the password creation flow

Additional Information

  • Status version: nightly 28/03
  • Operating System: Android, iOS
@SilentCruzer
Copy link
Collaborator

if this issue issue is still open, I would like to work on this issue

@qoqobolo
Copy link
Contributor Author

if this issue issue is still open, I would like to work on this issue

@cammellos pinging you to confirm the assignment

@cammellos
Copy link
Contributor

Sure, I'll assign it to you @SilentCruzer !

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

Successfully merging a pull request may close this issue.

3 participants