make ‘-min’ minimize wallet loading dialog #749

1. 
   furszy commented at 11:43 pm on August 13, 2023: member

   Simple fix for #748.

   When ‘-min’ is enabled, no loading dialog should be presented on screen during startup.

2. gui: make '-min' minimize wallet loading dialog

   When '-min' is enabled, no loading dialog should
   be presented on screen during startup.

   32db15450a
3. 
   DrahtBot commented at 11:43 pm on August 13, 2023: contributor

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Reviews

   See the guideline for information on the review process.

   Type	Reviewers
   ACK	hebasto

   If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

4. furszy renamed this:
   gui: make '-min' minimize wallet loading dialog
   make '-min' minimize wallet loading dialog
   on Aug 13, 2023
5. furszy cross-referenced this on Aug 14, 2023 from issue gui: make '-min' minimize wallet loading dialog by furszy
6. DrahtBot added the label CI failed on Aug 17, 2023
7. DrahtBot removed the label CI failed on Aug 22, 2023
8. 
   pablomartin4btc commented at 11:47 pm on August 22, 2023: contributor

   I’ve tested this change. Created several wallets in order to see the “Loading wallet…” dialog for a while. Without your change in master when I run bitcoin-qt -min the dialog is not shown and the app opens as usual. With your change when I run bitcoin-qt -min, the app gets minimized entirely, is that intentional?
9. 
   furszy commented at 12:26 pm on August 23, 2023: member

   I’ve tested this change. Created several wallets in order to see the “Loading wallet…” dialog for a while. Without your change in master when I run bitcoin-qt -min the dialog is not shown and the app opens as usual. With your change when I run bitcoin-qt -min, the app gets minimized entirely, is that intentional?

   Yes. Thats the -min flag purpose: start the app minimized.

10. hebasto approved
11. 
    hebasto commented at 11:21 am on August 31, 2023: member

    ACK 32db15450a9ef2a45de29b3b2ae60491a38edbd6, tested on Debian 11 + XFCE.

    The bug did not reveal itself on my Ubuntu + GNOME.

12. hebasto merged this on Aug 31, 2023
13. hebasto closed this on Aug 31, 2023

    ---

14. Frank-GER referenced this in commit a2ec5ec52a on Sep 8, 2023
15. 
    jonatack commented at 9:41 pm on October 30, 2023: contributor

    FWIW, a user on #bitcoin IRC mentioned today that this patch wasn’t backported to v25.x.
16. furszy deleted the branch on Oct 30, 2023

Contributors
furszy DrahtBot pablomartin4btc hebasto jonatack