Confusion with Create Wallet menu options #69

1. 
   Rspigler commented at 6:47 pm on August 22, 2020: contributor

   This is in regards to https://github.com/bitcoin/bitcoin/pull/15450

   If you check ‘Disable Private Keys’, is that not initially a Blank Wallet, until you import an xpub? I don’t understand what the difference would be between checking just ‘Disable Private Keys’, and checking ‘Disable Private Keys’ and ‘Make Blank Wallet’.

   Should checking ‘Disable Private Keys’ automatically check ‘Make Blank Wallet’? (But not vice versa - Making a Blank Wallet with/without Disabling Private Keys affects the future ability to import.)

   But then I am concerned that this would be very confusing for the end user. Enabling ‘Encrypt Wallet’ already disables ‘Disable Private Keys’.

2. Rspigler added the label Bug on Aug 22, 2020
3. 
   Rspigler commented at 6:47 pm on August 22, 2020: contributor

   Did not mean to add a Bug label
4. MarcoFalke removed the label Bug on Aug 22, 2020
5. 
   achow101 commented at 8:36 pm on August 22, 2020: member

   Disable private keys means that the wallet cannot have private keys ever. Make blank wallet means that the wallet starts out with nothing in it, but private keys can be added to the wallet via import or sethdseed.

   Both options begin very similarly in that the wallet doesn’t have any keys or scripts.

   Encrypt wallet is mutually exclusive with disable private keys because encrypting only encrypts the private keys. When the wallet doesn’t have any private keys, there’s nothing to encrypt so the option is useless.

6. 
   Rspigler commented at 10:21 pm on August 22, 2020: contributor

   I understand all of that.

   My point then, is that there is no difference between a wallet created with ‘Disable Private Keys’ & ‘Make Blank Wallet’ vs. just ‘Disable Private Keys’. Should enabling ‘Disable Private Keys’ then automatically enable ‘Make Blank Wallet’?

   There is; however, a difference between a wallet created with ‘Make Blank Wallet’ vs. ‘Make Blank Wallet’ & ‘Disable Private Keys’, as it affects future imports of private keys/HD seeds. So they shouldn’t be combined into one item.

   ‘Encrypt Wallet’ correctly disables ‘Disable Private Keys’, but I’m starting to get concerned that there are just a lot of variables now that could confuse the end user, so I wanted to note that.

   Thinking about this more: Should enabling ‘Disable Private Keys’ also disable ‘Encrypt Wallet’, just as enabling ‘Encrypt Wallet’ disables ‘Disable Private Keys’, as encrypt wallet is mutually exclusive with disabled private keys?

7. 
   Bosch-0 commented at 8:44 am on August 23, 2020: none

   I’ve got a WIP design related to this issue that more clearly explains each option, you can view it here: https://www.figma.com/file/FJ02rY3m8V9ZCDvoXjW39W/Bitcoin-Core?node-id=281%3A0
8. 
   Rspigler commented at 8:42 pm on August 23, 2020: contributor

   I like that a lot! (I had some difficulty in opening this in a DispVM, maybe paste screenshots/code here for people who don’t want to leave github to review?)

   Some typos in the ‘Blank Wallet’ descriptions; example “A blank wallet has will generate…”

   In the 1.2 modal; are you able to click more than one wallet option? Should selecting one, grey out the others?

9. 
   Rspigler commented at 8:44 pm on August 23, 2020: contributor

   Referencing this here: https://github.com/bitcoin/bitcoin/pull/17879#issuecomment-578533079
10. 
    Bosch-0 commented at 5:24 am on August 25, 2020: none

    I’ll upload a screenshot once I have made some changes - though there are many UI/UX changes being made other than just the initial menu options so ill likely open a new issue.

    Fixed that typo cheers, no spellcheck on Figma - usually clean up those details at the end ;)

    I’ll change these to radio inputs rather than check boxes with only one being able to be selected.

11. 
    ghost commented at 11:43 pm on August 26, 2020: none

    I’ve got a WIP design related to this issue that more clearly explains each option, you can view it here: https://www.figma.com/file/FJ02rY3m8V9ZCDvoXjW39W/Bitcoin-Core?node-id=281%3A0 @Bosch-0 This looks good.

    

12. Rspigler closed this on Aug 30, 2020

    ---

13. bitcoin-core locked this on Feb 15, 2022

Contributors
Rspigler achow101 Bosch-0 ghost