Page MenuHomePhabricator

Merge #13248: [gui] Make proxy icon from statusbar clickable
ClosedPublic

Authored by nakihito on Oct 19 2019, 04:47.

Details

Summary

6d5fcad576962e5950641f7e7b113a6ac6f397e5 [gui] Make proxy icon from statusbar clickable (Cristian Mircea Messel)

Pull request description:

Clicking on the proxy icon will open settings showing the network tab

https://github.com/bitcoin/bitcoin/pull/11491#issuecomment-336685303

Tree-SHA512: c3549749296918818694a371326d1a3b1075478918aaee940b5c7119a7e2cb991dcfda78f20d44d6d001157b9b82951f0d5157b17f4f0d1a0a242795efade036

Backport of Core PR13248
https://github.com/bitcoin/bitcoin/pull/13248/

Depends on D4269

Test Plan
make check
./bitcoin-qt -proxy=<valid proxy>

Clicking the P icon in the bottom right corner should open up the network tab in the options menu.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR13248
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7817
Build 13656: Bitcoin ABC Buildbot (legacy)
Build 13655: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Oct 19 2019, 04:47
Fabien requested changes to this revision.Oct 21 2019, 06:32
Fabien added inline comments.
src/qt/bitcoingui.cpp
858

Braces

src/qt/optionsdialog.cpp
178

Braces, also the next line.

This revision now requires changes to proceed.Oct 21 2019, 06:32
This revision is now accepted and ready to land.Oct 23 2019, 10:49