Issues with new create wallet dialogue #151

1. 
   Rspigler commented at 7:58 pm on December 14, 2020: contributor

   As commented here (https://github.com/bitcoin-core/gui/pull/96#issuecomment-727259580)

   Encrypt Wallet: “Encrypt the wallet. The wallet will be encrypted with a passphrase of your choice.” This is incorrect. The wallet is not encrypted, only the private keys are.

   Also, this inconsistency:

   Checking Disable Private Keys checks Make Blank Wallet (and disables Encrypt Wallet). This is correct. Disable Private Keys should only be checked when Make Blank Wallet is also checked. However, there is a way to have Disable Private Keys checked without having Make Blank Wallet checked (which doesn’t make sense).

   Check Disable Private Keys, then uncheck the auto-check Make Blank Wallet. You are now left with the settings:

   Disabled Encrypt Wallet, checked Disable Private Keys.

   This is a setting that you can’t enable yourself. (Checking Disable Private Keys also checks Make Blank Wallet normally).

   To fix, unchecking the auto-enabled Make Blank Wallet should also uncheck Disable Private Keys and re-enable Encrypt Wallet.

2. 
   hebasto commented at 0:26 am on March 5, 2021: member

   cc @achow101 @Sjors
3. hebasto added the label good first issue on Mar 5, 2021
4. 
   hebasto commented at 1:25 am on March 5, 2021: member

   @Rspigler

   Encrypt Wallet: “Encrypt the wallet. The wallet will be encrypted with a passphrase of your choice.” This is incorrect. The wallet is not encrypted, only the private keys are.

   This wording seems ubiquitous:

   

5. 
   Rspigler commented at 6:48 am on March 8, 2021: contributor

   Where is that screenshot from? Some wallets do encrypt the entire wallet. Bitcoin Core doesn’t.

   Compiling on master, I still see for the tooltip “Encrypt the wallet. The wallet will be encrypted with a passphrase of your choice”, which is incorrect. The wallet is not encrypted, the private keys are, which are two very different things https://github.com/bitcoin/bitcoin/issues/18085

   There are also various text boxes after encrypting the wallet which are incorrect “You wallet is now encrypted” etc.

6. 
   hebasto commented at 7:47 am on March 8, 2021: member

   @Rspigler

   Where is that screenshot from?

   The Bitcoin Core.

   Create a wallet without encryption. then in menu go to “Settings” -> “Encrypt Wallet”.

7. 
   Rspigler commented at 9:43 pm on March 8, 2021: contributor

   That should be changed as well then. For example, metadata is not encrypted.
8. maflcko referenced this in commit 19e3e65429 on Mar 26, 2021
9. hebasto added the label UX on May 1, 2021
10. 
    maflcko commented at 12:32 pm on May 22, 2024: contributor

    See also #722 (comment)

Contributors
Rspigler hebasto maflcko

Labels
good first issue UX