When opening or autoloading wallets there should be clear messages about rescanning in progress and wallets' names. #259

There are at least two ways to open a wallet.

1. File, Open wallet
2. or by manually appending settings.json file with wallet name.

When opening a wallet that has not been synchronized recently, in the first case there is a small window displayed with a message "opening wallet <walletname>". The node is rescanning at this time but nothing really indicates that, so unless user peeks in the debug.log file they may think bitcoin core is stuck.

<img width="680" alt="Screenshot 2021-03-24 at 18 21 13" src="https://user-images.githubusercontent.com/22457855/112633972-50d02e00-8e3a-11eb-92af-a9c3bd223c70.png">

In the second scenario, the initial window displays a message with just a single word "rescanning". This is better in a way that at least, as a user I am not confused about what is actually happening. However, there is a different problem: no indication about which wallet is being loaded. If a user has multiple wallets and don't keep them all opened all the time, but opens one, closes another and so on, upon (re)starting Bitcoin-qt they would not know for which wallet rescanning is happening.

<img width="592" alt="Screenshot 2021-03-25 at 01 13 54" src="https://user-images.githubusercontent.com/22457855/112633990-5463b500-8e3a-11eb-84ac-a6c678dbf7a1.png">

In my opinion, both windows should clearly state:

• what is going on right now exactly, i.e. rescanning
• names of wallets that are being loaded

Adding a named wallet to the string presented to the user should be a one-line change.

FIX: Explicitly enumerate all cases and be sure to not add the wallet name in one window and not leave others unnamed. E.g., "Sending/Receiveing addresses" windows.

Additionally, adding a comment to mention when not naming is required would be appreciated.

Keep in mind that there is also a case to cover in my second scenario where a json file is configured to load multiple wallets and more than one require rescanning. However, I am not familiar with how the rescan feature works under the hood specifically. Whether it rescans for each wallet one by one or rescans for all loaded wallets at once. If the latter is the case, is it really feasible to list all those wallets there? Two or three is fine but what if someone has dozens of them? (unless there is a limit to the number of wallets)

Whether it rescans for each wallet one by one or rescans for all loaded wallets at once

IIRC, the former one.

@khan201kp

What do you mean by wallet name?

A Wallet::getWalletName() return value: https://github.com/bitcoin-core/gui/blob/77a2f5d30c5ecb764b8a7c098492e1f5cdec90f0/src/interfaces/wallet.h#L82-L83

can I work on this issue?

@AdarshRawat1

can I work on this issue?

This project is permissionless.

Feel free to open a PR :)

Hey, I would love to work on this issue. Can I take it?

@Jaysinh146

Hey, I would love to work on this issue. Can I take it?

This project is permissionless.

Feel free to open a PR :)

You might also consider the previous attempt in #786.

Okay. Thank you