[Design] New ‘Create Wallet’ user flow. #77

issue Bosch-0 openend this issue on August 27, 2020
  1. Bosch-0 commented at 10:47 am on August 27, 2020: none

    Overview of current and new create wallet user flow

    image

    1.0 Who is this being designed for?

    I am under the assumption that currently the Bitcoin core GUI primarily caters to Bitcoin power users and or developers who are okay with usability trade offs for improved security and more robustly audited code. With the GUI being one of the most secure, simple ways to run a full node, and if a goal of the Bitcoin community is to have as many full nodes as possible (which I presume it is), then we need the GUI to cater to a wider, less technical audience.

    The aim of my designs is to remove friction for beginner and power users without having any security and or privacy trade offs, as well as have as minimal technical hurdles. Primarily, however, my goal is to make the GUI more beginner friendly. The Bitcoin core GUI in my view should be a benchmark / reference implementation and should not aim to be a flashy node software like umbrel or mynode – but this does not mean usability improvements cant be made. A well designed GUI will also further improve the security and privacy of its users - for this reason core should put more emphasis on design implementations. With good design, both the beginners and power users will be able to send and receive bitcoin backed by their own full node using the GUI without trade offs.

    2.0 Problems being solved

    The current user flow for creating a new wallet does not explain to users the types of wallets you can create or how each wallet type is different. Furthermore, the types of wallets available to create use terminology that is likely unfamiliar to many users (e.g. ‘Disable private keys’ could just be named ‘Watch Only Wallet’ which is more common and describes the same kind of wallet) - Luke Jr brang up this concern in https://github.com/bitcoin/bitcoin/pull/17879.

    The encrypt wallet option should be part of the create new wallet user flow and not optional (but also not mandatory – see below). The current initial modal that is presented does nothing to educate the user of the importance and pitfalls (not recoverable) of encrypting your wallet. This information could be included in the first modal but to keep things minimal and clean this should focus on ‘Choose your wallet type’ options whilst ‘Encrypt your wallet’ should have its on modal to fully explain why/why not to encrypt. Having encrypt wallet as part of the user flow by default will likely lead to better security of users bitcoin – An example being users will be more likely encrypt their wallets if informed why they should, this will lead to less stolen (but possibly more lost) bitcoins. If we want beginners using the GUI (meaning many more nodes) simple content changes such as this are important. Some discussions around this occurred in https://github.com/bitcoin/bitcoin/pull/17879.

    A general clean up of the UI without any drastic changes was also made. The design scope is quite limited due to Qt Widgets inheriting most components from the host OS. A cleaner UI generally leads to a smoother user experience. Nothing too fancy was done here, mostly some hierarchical separation of text, cleaner inputs and more consistent padding.

    From a wider GUI perspective I chose to focus on the create new wallet feature first as creating a wallet and creating it securely is one of the most important things a bitcoin user can do so I figured this was a top priority.

    3.0 Solutions

    Below are the designs I have made with comments, along with the current screens for comparison. You can also view the figma file (opens in your desktop fine) if you want to view the source files and more specific details of the design. You can leave comments on my figma file by clicking the speech bubble in the top left and clicking the area of the design you want to comment on. Designs are done using the WindowsOS Qt widgets inherited styling. Linux and Mac versions will inherit their own styling but the primary elements will be consistent.

    3.1 Create Wallet Meta

    image

    This is the first modal that pops up when creating a wallet, it is quite vague what it is the user needs to do here #69. Here are some questions that users likely have but are unanswered of which I aimed to answer or avoid the user having in the above new design – Why do I encrypt my wallet? It is selected by default so it must be important, but why is it important? What does disable private keys mean and why would I want that? Why is it disabled by default? What is a blank wallet and why would I want that? What are these options anyways? When I click create what exactly am I creating?

    For a cleaner more practical flow, the first modal should focus on the meta of wallet creation such as the name / wallet type and move secondary steps like encrypting to a subsequent modal. Several wallet types can be created which is not clearly communicated, nor is what each of those wallet types can do. I included a standard wallet option which should be selected by default as this is likely the most common wallet type users will create. The encrypt wallet option has been moved to its own modal and should be a default option for creating any type of wallet (details in design below).

    Minor Changes

    The wallet name input should be pre-filled in with text such as ‘MyWallet1’ for users who want to quickly make a wallet. If the user already has say one wallet then the pre-filled text should be ‘MyWallet2,’ they have two wallets ‘MyWallet3,’ etc. This creates a quicker user flow and better user experience. Electrum wallet does this nicely when creating a new wallet.

    Removed the help button ‘?’ at the top – currently does nothing and the new design should clarify any confusion the user may have- #74.

    3.2 Create Wallet Encryption

    image

    Currently the encrypt wallet option is a toggled feature in the first modal, but as explained above this modal should focus on wallet meta options like wallet type/name. The encrypt wallet modal should be presented to all users after picking a wallet type regardless of the wallet type chosen and should clearly emphasize the pros and cons of encrypting. It should also still be optional to not encrypt your wallet which can be done by leaving the passphrase input blank – though the warnings should push users more towards encrypting for better security / privacy. A modal should pop up (last frame shown above) confirming the user does not want to encrypt if they leave the passphrase input blank. https://github.com/bitcoin/bitcoin/pull/17879.

    Much of this emphasis on the pros and cons of encrypting is placed AFTER the user encrypts their wallet in subsequent modals which does not make sense – the emphasis should be placed before they decide to encrypt or not. I did this by including the details from the modals that usually pop up after entering a passphrase at the header of this modal. I also added a warning in red to clearly indicate that cons of encrypting. Lastly, I added a checkbox input that must be clicked before the ‘create’ button can be clicked which finishes the wallet creation process.

    Minor Changes:

    Moved the passphrase visibility toggle to icons on each passphrase input. This is a cleaner, more user friendly approach. Also made each input have their own toggle rather than a toggle that just shows both inputs when clicked like in the current design. These are two new icons not currently in the GUI that will need to be added to implement this design.

    3.3 Create Wallet extra redundant modals

    image

    With a more detailed encrypt wallet modal the two subsequent modals (shown above) can be removed making the create wallet process more seamless and involve less steps.

    Next Steps

    This was mostly a re-modelling of what currently exists - no extra frames / steps have been added. However, going forward I would also like to setup a user flow in this create wallet section of the GUI for watch only wallets and blank wallets. Such as having the ability to enter a xpub for watch only wallets in the creation process rather than in the settings after the wallet is created like it currently is.

    Looking forward to some feedback!

  2. Bosch-0 added the label Feature on Aug 27, 2020
  3. hebasto commented at 10:55 am on August 27, 2020: member
  4. achow101 commented at 3:56 pm on August 27, 2020: member

    Seems reasonable, but a few comments:

    1. Each of the options in the current Create Wallet Dialog have a tooltip. Hovering over them will display some additional information, although they aren’t necessarily helpful.
    2. The Encrypt Wallet Dialog shouldn’t be shown for Watch Only Wallet. Maybe we can have in its explanation that those types of wallets cannot be encrypted.
    3. Descriptor wallets is an option in the master branch and we need to incorporate that into this as well. However descriptor wallets aren’t another distinct wallet type but rather a feature that can be enabled for any of the wallet types you have listed.
  5. jonasschnelli commented at 4:02 pm on August 27, 2020: contributor

    General Concept ACK

    the “Encryption problem”

    Since it doesn’t really “encrypt the wallet”, we should use the term “encrypt private keys”. Other than that, people think their transactions and additional client side comments are also encrypted. Users might think their wallet privacy is protected when storing it (“encrypted”) in an unsafe place. Or at least warn that transactions (and therefore exposed addresses) and tx comments are not encrypted.

  6. MarcoFalke commented at 4:09 pm on August 27, 2020: contributor

    Concept ACK.

    Agree with @jonasschnelli . The warning “You have not entered a passphrase …” should probably not be added. See also https://github.com/bitcoin/bitcoin/issues/18085

  7. ghost commented at 3:17 am on August 28, 2020: none

    Everything looks good however I noticed that confirming wallet encryption form mentioned “bitcoins”. Recently saw one video and also read at few places in past that “bitcoin” is used for both singular and plural. Then I realized its not used only in one form and used at lot of places: https://github.com/search?l=&q=BITCOINS+repo%3Abitcoin-core%2Fgui&type=Code

    One video in which Andreas mentioned about it: https://youtu.be/FYo5E7zT-vM?t=640

    Not sure if it needs to be changed. Just an observation.

  8. Sjors commented at 11:55 am on August 28, 2020: member

    I like the larger layout this design implements. It’s much more clear than the tooltips. It also leaves room for an Advanced section at the bottom, where stuff like descriptor wallets (and perhaps an early HWI integration) could be handled.

    Agree with @achow101’s point (2), but I would just skip the “encrypt private keys” step without explaining. It also needs to be easier to skip this step, e.g. by adding a “Skip” button between “Back” and “Create”. I don’t like using a blank input field as the method of indicating not wanting a password.

    Later on you could add a Recover / Import dialog (or option inside the dialog), where people can restore from a backup file, BIP39 mnemonic, bunch of xpubs.

  9. Bosch-0 commented at 4:50 am on August 29, 2020: none

    Seems reasonable, but a few comments:

    1. Each of the options in the current Create Wallet Dialog have a tooltip. Hovering over them will display some additional information, although they aren’t necessarily helpful.
    2. The Encrypt Wallet Dialog shouldn’t be shown for Watch Only Wallet. Maybe we can have in its explanation that those types of wallets cannot be encrypted.
    3. Descriptor wallets is an option in the master branch and we need to incorporate that into this as well. However descriptor wallets aren’t another distinct wallet type but rather a feature that can be enabled for any of the wallet types you have listed.

    yes I should have mentioned this currently exists, though I don’t think this method of explaining the wallet types is the best UX and the current tooltips info isn’t as informative as they could be.

    Does this mean blank only wallets can also not be encrypted (because no pkeys are generated)? It would be handy if you could encrypt the watch only / blank wallets for privacy reasons though. I’ll get the user flows up for the other wallet creation types soon.

    Instead of having descriptor wallets as an option in the first modal I think it would be better to have it in subsequent wallet creation steps as an advanced option as @Sjors mentioned, or just something that can be turned on after the wallet is created. I prefer this second method as to not clutter the wallet creation process / leave this to advanced users / avoid scaring beginners with more new terminology.

    Agree with @jonasschnelli . The warning “You have not entered a passphrase …” should probably not be added. See also bitcoin/bitcoin#18085

    I agree with not having that modal, it isn’t entirely unnecessary, my main motivation behind adding it was to push users towards encrypting as well prevent accidentally not encrypting (unlikely but could happen). Though I think the more descriptive encrypt wallet modal educates the user enough.

    Recently saw one video and also read at few places in past that “bitcoin” is used for both singular and plural.

    My mistake, I usually use Andreas terminology, similar to other commodities you wouldn’t call gold golds or wheat wheats.

    …but I would just skip the “encrypt private keys” step without explaining. It also needs to be easier to skip this step, e.g. by adding a “Skip” button between “Back” and “Create”. I don’t like using a blank input field as the method of indicating not wanting a password.

    If watch only wallets are unable to be encrypted then yeah just skipping this step would be ideal, though giving watch only wallets the ability to be encrypted is good for privacy. I’ll do a watch only and blank wallet user flow soon.

    I’ll make it easier to skip this step and just have this as the ’no password option’ rather than leaving the field blank. I’d rather leave the skip option as a tertiary button as to still lean towards users encrypting to encourage good OPSEC.

    …(and perhaps an early HWI integration)… Later on you could add a Recover / Import dialog (or option inside the dialog), where people can restore from a backup file, BIP39 mnemonic, bunch of xpubs.

    Yeah I would like to eventually have all of this as part of the flows here, ideally having the first modal as having the options to either create a new wallet or import existing (with xpub / pkeys). I’ll get design mockups done for these in the coming weeks for feedback.

  10. achow101 commented at 5:32 am on August 29, 2020: member

    Does this mean blank only wallets can also not be encrypted (because no pkeys are generated)? It would be handy if you could encrypt the watch only / blank wallets for privacy reasons though.

    Wallet encryption only encrypts private keys. No transactions, watch only scripts, or literally anything else in the wallet is encrypted. There are no privacy gains.

    or just something that can be turned on after the wallet is created.

    Absolutely not. Descriptor wallets must be a thing for wallet creation. In fact, I will give an automatic Concept NACK if it is not an option. Descriptor wallets is not a feature that can just be turned on and it is not reasonable to make people migrate a newly created legacy wallet to a descriptor wallet if they just wanted a descriptor wallet. Furthermore, migration will probably have some bugs and be horribly inefficient.

  11. Bosch-0 commented at 5:45 am on August 29, 2020: none

    Wallet encryption only encrypts private keys. No transactions, watch only scripts, or literally anything else in the wallet is encrypted. There are no privacy gains.

    I’m not too familiar with how the technical details work here, but what I had in mind is how say in Electrum you can encrypt your watch only wallet. How do they achieve this? Why is not possible on the core GUI? I know there is no keys generated to encrypt, but is there another way to conceal that information being easily accessible to someone simply opening core on your device?

    Absolutely not. Descriptor wallets must be a thing for wallet creation. In fact, I will give an automatic Concept NACK if it is not an option. Descriptor wallets is not a feature that can just be turned on and it is not reasonable to make people migrate a newly created legacy wallet to a descriptor wallet if they just wanted a descriptor wallet. Furthermore, migration will probably have some bugs and be horribly inefficient.

    Apologies, I am not too familiar with the technical details of how descriptor wallets work. I will include it in the flow and upload a new design - is this something you would be comfortable with under a advanced options dialogue in a modal after choosing your wallet type?

  12. achow101 commented at 5:53 am on August 29, 2020: member

    I’m not too familiar with how the technical details work here, but what I had in mind is how say in Electrum you can encrypt your watch only wallet. How do they achieve this? Why is not possible on the core GUI?

    Electrum encrypts more information, IIRC they do encrypt the entire wallet file. Bitcoin Core only encrypts the private keys.

    I know there is no keys generated to encrypt, but is there another way to conceal that information being easily accessible to someone simply opening core on your device?

    Nope.

    Apologies, I am not too familiar with the technical details of how descriptor wallets work. I will include it in the flow and upload a new design - is this something you would be comfortable with under a advanced options dialogue in a modal after choosing your wallet type?

    I guess advanced options is fine. I prefer that it is an option up front just so that we can promote people to use it more, but the option just needs to exist in a reasonably accessible place.

  13. Sjors commented at 12:01 pm on August 30, 2020: member

    I imagine at some point we make descriptor wallets the default and it’s nice if that doesn’t lead to a confusing UI change. A check box hidden in an Advanced section helps with that.

    In #4 I automatically detect if a hardware wallet is connected, in which case descriptor wallet is selected automatically.

  14. Rspigler commented at 0:45 am on August 31, 2020: contributor

    Does this mean blank only wallets can also not be encrypted (because no pkeys are generated)?….

    Wallet encryption only encrypts private keys….

    So what happens currently if you select ‘blank only wallet’ and ’encrypt wallet’? In this new modal, should it ask the user if they wish to encrypt private keys when (if) private keys are imported?

  15. achow101 commented at 0:54 am on August 31, 2020: member

    So what happens currently if you select ‘blank only wallet’ and ’encrypt wallet’?

    An encryption key is added to the wallet. Any keys that are added to the wallet (either via import of sethdseed) will be encrypted before they are written to the wallet file.

  16. Bosch-0 commented at 5:06 am on August 31, 2020: none
    What are the UI hurdles when descriptor wallets are a default for all wallets created with core? From my understanding it would actually improve UI/UX a bit (for example allowing xpub imports during the wallet creation process for watch only / blank wallets).
  17. GBKS commented at 11:17 am on August 31, 2020: none

    Awesome to see this issue, and I like this direction a lot. A few thoughts on copy:

    “Please choose a name for your wallet and the type of wallet you want to create.”

    • Do we want to use words like “please”?
    • How about shortening to “Choose a wallet name and type.”?

    “A standard wallet will generate a set of private keys. You will be able to send and receive bitcoin with this wallet.”

    I think you can avoid repeating the wallet type from the beginning of these description sentences and just write “Generates a set of private keys…”.

    “Encrypt your wallet”

    Encrypting the wallet is what the app does, not me. I only choose a passphrase. How about changing the name of this screen to “Choose a passphrase”?

    “We recommend using…”

    • Do we want to use “we”? Who is “we” to a user? How about “A good passphrase consists of …”?
    • How about having a “Skip” button instead of leaving the input field blank as a way to not use a passphrase? That way it becomes a more intentional action.

    “If you lose this passphrase you will lose all of your bitcoin”

    How about “… you will lose access to…”?

    “You have not entered a passphrase for your wallet, not encrypting your wallet lowers your privacy and security. Are you sure you do not wish to encrypt your wallet?”

    A pretty long first sentence and lots of negation that makes both sentences hard to parse. How about something like “Do you want to continue with low security and accept the resulting risks of potentially losing your bitcoin?”

    Overall I prefer simple, clear, short sentences. Should be much easier to parse and understand, particularly by non-native English speakers. Good copy goes a long way.

  18. Bosch-0 commented at 11:40 am on August 31, 2020: none
    Cheers @GBKS I have integrated these changes into the figma file, agree with having as simple explanations as possible.
  19. Sjors commented at 8:27 am on September 18, 2020: member
    I made a few modifications to the create dialog based on the above, but nowhere near a full implementation: #96
  20. ghost commented at 7:02 pm on November 4, 2020: none

    An encryption key is added to the wallet. @achow101 Some people are using this to sell wallet.dat files

    Example: http://web.archive.org/web/20210406135133/https://twitter.com/UnderTheBreach/status/1303316723186139136 https://cointelegraph.com/news/no-a-quantum-computer-won-t-instantly-reward-you-with-69-000-bitcoins-yet

    https://cointelegraph.com/news/1b-of-bitcoin-from-silk-road-wallet-moves-for-first-time-since-2015

    image

    The HD symbol in above image which is from the tweet shows that “HD key generation is disabled” in this wallet

    So I am assuming most of the wallet.dat files with screenshots sold at https://allprivatekeys.com/wallet.dat are created like this:

    1. Create “blank wallet” with encryption
    2. Import any bitcoin address with big amount using importaddress or importmulti (watch-only)
    3. Share wallet.dat file and screenshots
  21. achow101 commented at 7:25 pm on November 4, 2020: member

    Some people are using this to sell wallet.dat files … The ~HD~ symbol in above image which is from the tweet shows that “HD key generation is disabled” in this wallet

    So I am assuming most of the wallet.dat files with screenshots sold at https://allprivatekeys.com/wallet.dat are created like this:

    1. Create “blank wallet” with encryption
    2. Import any bitcoin address with big amount using importaddress or importmulti (watch-only)
    3. Share wallet.dat file and screenshots

    Nope, that’s not how those wallets are made. If it were, you would see the balance as a watch only balance.

    Typically scammers make wallets like that by directly modifying the wallet.dat file to include a bogus ckey record. This record has the pubkey as the key, and the encrypted private key as the value. Usually the scammers put a pubkey for a high value address, and then garbage for the encrypted privkey. Of course there’s no way to verify that the encrypted privkey is correct without decrypting it.

  22. ghost commented at 9:12 pm on November 4, 2020: none

    Nope, that’s not how those wallets are made. If it were, you would see the balance as a watch only balance.

    Yes. Just noticed that it shows as “watch-only” if you import address in a blank wallet and “available” in a wallet with private keys disabled.

    Typically scammers make wallets like that by directly modifying the wallet.dat file to include a bogus ckey record. This record has the pubkey as the key, and the encrypted private key as the value. Usually the scammers put a pubkey for a high value address, and then garbage for the encrypted privkey. Of course there’s no way to verify that the encrypted privkey is correct without decrypting it.

    Interesting

  23. sipa commented at 7:36 am on November 5, 2020: contributor

    Of course there’s no way to verify that the encrypted privkey is correct without decrypting it.

    What if the ckey record included a signature of the encrypted data with the private key it claims to have? That would prevent scammers from inserting public keys they don’t have, though it would leave the option open for them to insert a public key for which they have the private key (but don’t insert it in the wallet file).

  24. achow101 commented at 4:40 pm on November 5, 2020: member

    What if the ckey record included a signature of the encrypted data with the private key it claims to have? That would prevent scammers from inserting public keys they don’t have, though it would leave the option open for them to insert a public key for which they have the private key (but don’t insert it in the wallet file).

    Anything that we do now won’t change the fact that prior wallets did not have the check, and because we have to maintain backwards compatibility, scammers would just use an old version of the wallet format which did not include signatures with the ckey.

  25. ryanofsky commented at 4:58 pm on November 5, 2020: contributor

    because we have to maintain backwards compatibility, scammers would just use an old version of the wallet format which did not include signatures with the ckey.

    Backwards compatibility just requires support for opening these wallets, not for displaying the balance as if it were spendable. It could show the balance as unconfirmed or something, until the wallet is upgraded

  26. MarcoFalke referenced this in commit e7986c51bc on Nov 17, 2020
  27. hebasto removed the label Feature on Mar 6, 2021
  28. hebasto added the label Design on Mar 6, 2021
  29. hebasto added the label UX on May 1, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/gui. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-21 12:20 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me