GUI: start rows in askpassphrasedialog.ui at 0 (not 1) #1365
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
|
Does this change anything visually? |
Author
|
I took a screenshot of both initial screens (encrypt wallet) and compared them ... no difference. |
laanwj
added a commit
that referenced
this pull request
May 21, 2012
GUI: start rows in askpassphrasedialog.ui at 0 (not 1)
coblee
pushed a commit
to litecoin-project/litecoin
that referenced
this pull request
Jul 17, 2012
GUI: start rows in askpassphrasedialog.ui at 0 (not 1)
lateminer
pushed a commit
to lateminer/bitcoin
that referenced
this pull request
May 6, 2020
aa7ff86 [GUI][Wallet] Use unlock context for HD upgrade (random-zebra) 060790f [Doc] Add Context Lock/Unlock to release notes (random-zebra) dee68e3 [Refactor] Use isWalletUnlocked() to check valid in requestUnlock() (random-zebra) 556a176 [GUI][Model] Remove verifyWalletUnlocked and connect everything (random-zebra) 1c803f0 [UI][Model] Connect WalletModel::requireUnlock (random-zebra) 31cbef7 [Refactor] Rename ...AnonymizationOnly --> ...StakingOnly (random-zebra) 1a7aa56 [Cleanup] Remove un-used walletpassworddialog (random-zebra) a85deb7 [Wallet][Model] Fix and refactor UnlockContext (random-zebra) 05b1360 [Trivial] Styling: spaces and brackets (random-zebra) Pull request description: Fix `UnlockContext` class and connect it where needed. Now, once an action requires the wallet to be fully unlocked, the askpassphrase dialog is presented to the user (if the status is "Locked" or "UnlockedForStaking"). This way, the password can be entered directly, and the operation can resume without needing additional clicks by the user. The previous locking status is restored when the action completes. Interested flows: - Send transaction (closes bitcoin#1365) - Create New Receive Address - Create New Payment Request - Create Masternode Controller - Create Cold Staking Address (closes bitcoin#1171) - Send Cold Stake Delegation - Wallet Data: Add multisend recipient - Tools: Sign Message - Tools: BIP38 tool - Upgrade wallet to HD Also Closes bitcoin#553 Closes bitcoin#1035 And maybe bitcoin#1152 becomes not needed now. ACKs for top commit: Fuzzbawls: ACK aa7ff86 Tree-SHA512: 21590ca025b8c3a26824fea9c1107b68e0508cc080acbf0e1bf5b1c86da15f5f6696ea680c57989cc0cb8b5f54d328f32a60d24dc3c5ca8526179ef1750b3482
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.