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

Implement login with more than 12 words - Closes #1748 #1787

Merged
merged 12 commits into from
Feb 21, 2019

Conversation

massao
Copy link
Contributor

@massao massao commented Feb 18, 2019

What issue have I solved?

#1748

How have I implemented/fixed it?

Implemented logic on passphraseInputV2 component to show 24 fields, when the user paste a passphrase with more than 12 words, or if the user presses space, tab or right arrow on the 12th field.

How has this been tested?

Go to /login and paste a passphrase with more than 12 words, or press space, tab or right arrow on the 12th field, if wants to go back to having 12 fields, it's needed to go back to the previous page an go again to /login

Review checklist

@massao massao self-assigned this Feb 18, 2019
@massao massao requested a review from slaweet February 18, 2019 16:28
@massao massao changed the base branch from 1.12.0 to development February 18, 2019 16:34
@massao massao changed the base branch from development to 1.12.0 February 18, 2019 16:34
@slaweet slaweet requested review from osvaldovega and removed request for slaweet February 19, 2019 08:24
osvaldovega
osvaldovega previously approved these changes Feb 20, 2019
Copy link
Contributor

@osvaldovega osvaldovega left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

@Efefefef Efefefef left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐛 left arrow and cmd+del move caret to the word beginning in the 13-24 words

@Efefefef Efefefef added the ready label Feb 20, 2019
@Efefefef
Copy link
Contributor

Good

@massao massao merged commit a41e906 into 1.12.0 Feb 21, 2019
@massao massao deleted the 1748-login-with-more-than-12-words branch February 21, 2019 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants