Page MenuHomePhabricator

Merge #15149: gui: Show current wallet name in window title

Authored by jasonbcox on Nov 27 2019, 20:23.



fe7048b39 gui: Show current wallet name in window title (João Barbosa)
8a7926112 gui: Keep network style in BitcoinGUI (João Barbosa)
f411c8b35 gui: Remove unused return type in some BitcoinGUI methods (João Barbosa)

Pull request description:

<img width="876" alt="screenshot 2019-01-11 at 23 58 26" src="">

Tree-SHA512: 5c43f615834983bc1c5045e07c6e119044dd78ca947fd2679d302b519d5ce1d08d29ca00b1c11e88c4bbc4d56f2e6f4a8adc42084f3503e751e642e8a13112dc

Backport of Core PR15149

Depends on D4567
Completes T699

Test Plan

ninja check
Verify bitcoin-qt shows wallet name in window title.

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jasonbcox created this revision.Nov 27 2019, 20:23
jasonbcox edited the test plan for this revision. (Show Details)Nov 27 2019, 20:24
Fabien accepted this revision.Nov 27 2019, 20:42
Fabien added a subscriber: Fabien.
Fabien added inline comments.
76 ↗(On Diff #14470)

Nit: put config before platform to keep ordering consistent

This revision is now accepted and ready to land.Nov 27 2019, 20:42
jasonbcox updated this revision to Diff 14474.Nov 27 2019, 21:52
  • Rebase
  • changed member initializer ordering per feedback
  • removed enableWallet(false) which was erroneously left in D1959
Fabien accepted this revision.Nov 28 2019, 06:34