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
[UI] passphrase redesign #12731
base: master
Are you sure you want to change the base?
[UI] passphrase redesign #12731
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NACK
Why are we showing the passphrase dialog before the recovery words dialog?
As described in BIP39: a user may decide to protect their mnemonic with a passphrase
, so the passphrase is an extension to the recovery words, it is a string added to the recovery words, so it should naturally be displayed after the recovery words not the other way around.
Years ago Wasabi 1.0 used to do exactly what this PR is doing; to ask for a passphrase after naming the wallet and before showing the recovery words and that got changed because the UX was bad and many users thought it is not part of the backup.
Here is the issue and the PR where we fixed that in Wasabi 2.0: #6930 & #8867. Why are we going back to this?
Last thing, why using 13th seed word
is a bad idea:
(clarification: sometimes we use to say that the passphrase is the 13th word but that's just a mental shortcut for explaining the concept the newbies, the passphrase is not a word, or at least not necessarily, but multiple extra words separated by white spaces)
When I read 13th seed word, I think that the word should be in the BIP39 words list. It is not the case (I took a long time to understand that myself because you guys call it 13th seed word), therefore 13th word is not good, as it's not a 13th word. And I don't think it's relevant to say "13th word is used by some people", that's not an argument, if other people want to make a mistake: good for them, it doesn't mean we have to follow.
I NACK the NACK |
This doesn't answer the questions and concerns mentioned above. Here are some guidelines from the Bitcoin Core repo on how they should be used:
https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#conceptual-review
https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#peer-review Side note: We can learn a lot from Bitcoin Core's contributing guidelines to improve our review process which is a big bottleneck in the development process of the project. |
The PR is not ready yet. Wait until it is finished.
Because it will be displayed in the next dialog where you write down your recovery words.
The situation is the same now despite the fact we moved it after the recovery words. Why? It is not up to that. Those changes were made for different reasons.
It won't be used. We will use passphrase everywhere and only passphrase. Please wait until the PR is ready to review. |
You can use Shift+Tab to go to previous word. |
No, I'm trying to do this in the video |
The usage of 13th word and passphrasePinging: @Kruwed Using both "passphrase" and "13th word" in the same instruction could potentially confuse a layman user, especially if they are not familiar with the specific jargon associated with bitcoin and cryptographic security.
We need to use consistent terminology and avoid mixing it. We agreed to stick with the passphrase but we can reconsider it based on: we should use the one that is more commonly used.
So for this the text needs to be changed at the Add Passphrase dialog.I like @yahiheb idea to change the dialog text to this: |
Recovery Words screen
|
"No added value in this design compared to the current one"I went through carefully all of your comments @yahiheb. Here are my answers:
|
@wieslawsoltes I gather the suggestion that makes sense from the other's comments. Here is the list to work on: TODO:
(There are still several issues, but fix this list first and we will do another round of testing) |
… first, then cancel to make Enter key binding work
Fixes #12614
TODO:
I have written down
checkbox, and pressing ENTER again should execute the continue button. (We do the same in BuyAnything, check the code) 763e1a6, 8a16824Fill your 12 words in the correct order and enter the same passphrase that you used on wallet creation. If you have more words than 12 go to the advanced view.
(Guys can polish it later) abc1789