gui: HD icon is displayed when no wallet is loaded #17927

issue fanquake opened this issue on January 15, 2020
  1. fanquake commented at 6:37 AM on January 15, 2020: member

    Compile master, and run with src/qt/bitcoin-qt -regtest -nowallet. As expected, no icon is displayed. Load and unload a wallet. The HD icon remains.

    On startup: on_startup

    After a wallet load and unload: after_load_unload

  2. fanquake added the label GUI on Jan 15, 2020
  3. fanquake added the label good first issue on Jan 15, 2020
  4. theStack commented at 3:05 PM on January 15, 2020: contributor

    The same behaviour also applies to the encryption icon, if you load a wallet that is encrypted.

  5. brakmic cross-referenced this on Jan 15, 2020 from issue gui: hide HD & encryption icons when no wallet loaded by brakmic
  6. brakmic commented at 8:31 PM on January 15, 2020: contributor

    I could also show/hide the lock icon the same way I did with HD icon, but I am not sure if this should be packed together in a single PR. It's actually nothing complex, so it could maybe be better to avoid opening another one only because of an icon.

    For now, I have only changed the handling of HD icon.

  7. fanquake commented at 9:57 PM on January 15, 2020: member

    but I am not sure if this should be packed together in a single PR. @brakmic Given how simple the changes are here, I think it makes sense to just fix both in the same PR.

  8. promag commented at 10:01 PM on January 15, 2020: member

    the same PR.

    Agree.

  9. brakmic commented at 10:04 PM on January 15, 2020: contributor

    Ok, have updated the PR and renamed it too.

  10. fanquake closed this on Jan 16, 2020

  11. bitcoin locked this on Feb 15, 2022
  12. Munkybooty referenced this in commit 451216e956 on Jun 25, 2022
  13. Munkybooty referenced this in commit 61355ef19d on Aug 3, 2022
  14. Munkybooty referenced this in commit 592ed6e7dc on Aug 22, 2022
  15. Munkybooty referenced this in commit f7551aa380 on Aug 24, 2022
  16. Munkybooty referenced this in commit 40c48c06ac on Aug 31, 2022
  17. Munkybooty referenced this in commit 07da6f6b92 on Sep 19, 2022
  18. Munkybooty referenced this in commit afa583fb39 on Oct 3, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-20 06:54 UTC