Skip to content

Fix transactions disable problem#922

Closed
beati wants to merge 1 commit intobitcoin-core:masterfrom
beati:gui-fix-transactions-disable-problem-2
Closed

Fix transactions disable problem#922
beati wants to merge 1 commit intobitcoin-core:masterfrom
beati:gui-fix-transactions-disable-problem-2

Conversation

@beati
Copy link
Copy Markdown

@beati beati commented Jan 9, 2026

  • In bitcoin-qt, there is under Settings the "Mask values" option
  • When a wallet is open and the "Mask values" checkbox is checked, the "Transactions" tab is disabled
  • When no wallet was open and the "Mask values" checkbox was checked and a wallet was opened, the "Transactions" tab was enabled
  • This fix changes this, so that there is always the same behavior
  • The "Transactions" tab is now always disabled when the "Mask values" checkbox is checked
  • This fix is manually tested on the fork

@DrahtBot
Copy link
Copy Markdown
Contributor

DrahtBot commented Jan 9, 2026

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.
A summary of reviews will appear here.

@hebasto hebasto removed the CI failed label Jan 19, 2026
@hebasto hebasto changed the title gui: fix transactions disable problem Fix transactions disable problem Jan 19, 2026
@hebasto
Copy link
Copy Markdown
Member

hebasto commented Jan 19, 2026

Does this PR solve the same issue as #815?

@beati
Copy link
Copy Markdown
Author

beati commented Jan 19, 2026

Does this PR solve the same issue as #815?

Yes, it looks like.
But this PR has a different solution.

@hebasto
Copy link
Copy Markdown
Member

hebasto commented Mar 22, 2026

@beati

Thank you for your contribution!

Does this PR solve the same issue as #815?

Yes, it looks like. But this PR has a different solution.

An alternative approach was recently merged, as there was a consensus among developers regarding the preferred code structure (see #815 (comment) and #815 (review)).

Because that solution has already landed, I am closing this PR.

@hebasto hebasto closed this Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants