Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[QT] Options UI cleanup #895

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@Alko89
Copy link

commented May 14, 2019

This PR is for issue #793

Hides zPIV options when wallet is disabled (disablewallet).

Main tab:
Screenshot_20190514_115224
Screenshot_20190514_130322

Display tab:
Screenshot_20190514_115202
Screenshot_20190514_130344

@Warrows
Copy link
Collaborator

left a comment

Concept ACK. I have a few nits though. And it would be better to have a rebase rather than a merge commit in the PR.

Show resolved Hide resolved src/qt/forms/optionsdialog.ui
Show resolved Hide resolved src/qt/forms/optionsdialog.ui
Show resolved Hide resolved src/qt/forms/optionsdialog.ui

@Warrows Warrows added the GUI label May 14, 2019

@Alko89 Alko89 force-pushed the Alko89:uicleanup branch from 766f651 to c424f0f May 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.