Skip to content

Close console autocomplete when leaving Console tab - #809

Closed
uqlidi wants to merge 1 commit into
bitcoin-core:qt6from
uqlidi:qml/close-console-autocomplete
Closed

Close console autocomplete when leaving Console tab#809
uqlidi wants to merge 1 commit into
bitcoin-core:qt6from
uqlidi:qml/close-console-autocomplete

Conversation

@uqlidi

@uqlidi uqlidi commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

The RPC console lives as a persistent StackLayout child and its
autocomplete Popup renders in the window overlay layer, so hiding
the console on a tab switch does not hide the popup. It lingers over
the newly selected tab.

Add a tabActive property to CommandConsole, bound by DesktopWallets
to consoleTabButton.checked, and close the popup when the tab is
left. Keyed off the tab ButtonGroup rather than StackLayout visibility,
which never reports the transition.

fix #782

Co-authored-by: johnny9 <johnny9dev@pm.me>

@johnny9 johnny9 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 5021838

@hebasto hebasto changed the title qml: close console autocomplete when leaving Console tab Close console autocomplete when leaving Console tab Jul 22, 2026

@jarolrod jarolrod left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 5021838

hebasto added a commit that referenced this pull request Jul 23, 2026
5021838 qml: close console autocomplete when leaving Console tab (Uqlidi)

Pull request description:

  The RPC console lives as a persistent StackLayout child and its
  autocomplete Popup renders in the window overlay layer, so hiding
  the console on a tab switch does not hide the popup. It lingers over
  the newly selected tab.

  Add a `tabActive` property to CommandConsole, bound by DesktopWallets
  to `consoleTabButton.checked`, and close the popup when the tab is
  left. Keyed off the tab ButtonGroup rather than StackLayout visibility,
  which never reports the transition.

  fix #782

ACKs for top commit:
  johnny9:
    ACK 5021838
  jarolrod:
    ACK 5021838

Tree-SHA512: 884dbacc6653bc3e38c1bf157a8b58cd0091f5986c00eb2079670a81e44021e0f13721055cae02645ef376d85817c30e41680e9b0683ee91ffbfdea21bd5a188
@hebasto

hebasto commented Jul 23, 2026

Copy link
Copy Markdown
Member

This PR is merged.

@hebasto hebasto closed this Jul 23, 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.

Close RPC console autocomplete when changing desktop tabs

4 participants