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

Nag-bar and dialog for Passwordless Users to provide password #3602

Closed
lampholder opened this issue Apr 11, 2017 · 8 comments
Closed

Nag-bar and dialog for Passwordless Users to provide password #3602

lampholder opened this issue Apr 11, 2017 · 8 comments

Comments

@lampholder
Copy link
Member

The details of the new guest experience for Riot are on the project plan: element-hq/riot-meta#59

Users who have provided a username and completed a CAPTCHA should be prompted to complete their registration to avoid losing access to said username.

This should use the existing 'You are Rioting as a guest' top notification bar, and should include a link to the mechanism to complete registration.

@lampholder lampholder added this to the RW003 milestone Apr 24, 2017
@lukebarnard1 lukebarnard1 changed the title Improve Landing as Guest: Prompt Passwordless Users to complete registration Prompt Passwordless Users to complete registration Apr 25, 2017
@lukebarnard1 lukebarnard1 changed the title Prompt Passwordless Users to complete registration Nag bar for Passwordless Users to provide password Apr 25, 2017
@lukebarnard1 lukebarnard1 changed the title Nag bar for Passwordless Users to provide password Nag-bar and dialog for Passwordless Users to provide password Apr 25, 2017
@lampholder
Copy link
Member Author

@ara4n @AmandineLP we need some wording for the nag box.

We probably want to go for something like one of these four (steadily increasing in level of alarm):

  1. Please click here to complete registration of your Riot account.
  2. To log in on other devices, please complete registration by setting a password for this account.
  3. To claim @GreatName:matrix.org complete registration by setting a password.
  4. Without completing registration you risk losing @GreatName:matrix.org forever! Set your password now!

@lampholder
Copy link
Member Author

The current nag bar features links to log in or to register.

We might want to consider a button here instead since buttons are more actiony:
nag

@AmandineLP
Copy link
Contributor

M & I suggest:

To be able to return to your account, you need to set a password. [............]

@AmandineLP AmandineLP assigned lampholder and unassigned ara4n and AmandineLP May 4, 2017
@lampholder
Copy link
Member Author

Is that the password field in the nag bar itself?

@lukebarnard1
Copy link
Contributor

Is that the password field in the nag bar itself?

Conclusion was that, no it's the "Set Password" button.

@lampholder
Copy link
Member Author

lampholder commented May 11, 2017

The current registration password set UX has some issues:

  • password criteria (minimum length == 6) is not written and has to be discovered by typing
  • if there is anything wrong with you password (too short or mismatched) it just wears a red border and shakes itself side to side
  • the field names are only there as placeholder text (so they can be completely hidden by malfunctioning form auto-completes)

The strickenthrough issues above were experienced by password feedback being below the fold on my macbook air :\ #3893

To move forward, I suggest we:

  1. Implement a naive impl of the password by dropping the password set components from the registration page into here
  2. Prioritise reworking our password set UX as a separate task.

Thoughts?

@lampholder lampholder removed their assignment May 11, 2017
@lukebarnard1
Copy link
Contributor

@lampholder this feels like the wrong place for that. This issue is for the nag bar and the new dialog for changing passwords.

@lukebarnard1
Copy link
Contributor

^ Initial simple implementation of the dialog has now been merged.

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

5 participants