-
Notifications
You must be signed in to change notification settings - Fork 717
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
[GUI] cs widget: minor usability improvements #1652
[GUI] cs widget: minor usability improvements #1652
Conversation
Do not try to unlock the wallet if either: - cold staking is disabled - the owner or staker address are not valid - the amount is below the minimum allowed - the external delegation warning dialog is not accepted - both addresses are internal
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionality tested, working as intended.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review ACK, nice adds.
"Delegating to" instead of "Sending to"
a1de007
to
ba7a8fd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
review ACK ba7a8fd .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK ba7a8fd
Three simple fixes to the cold staking widget:
give immediate feedback (with red outline) when the owner address is not valid (same as we have with SendMultiRow objects, e.g. the staker address in this widget). Closes [Feature request] Add warning if invalid address is entered into Owner address field #1596
on send: try to unlock the wallet only when all validity checks are passing.
since the address shown on the confirmation dialog is the staker, change the label "Sending to" in "Delegating to".