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

Changing between different wallets switches to overview tab #83

Closed
1 task done
Tracachang opened this issue Jun 9, 2024 · 6 comments
Closed
1 task done

Changing between different wallets switches to overview tab #83

Tracachang opened this issue Jun 9, 2024 · 6 comments
Labels

Comments

@Tracachang
Copy link

Tracachang commented Jun 9, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current behaviour

When using multiple wallets (switching them) If you do this in the Transactions tab, instead of displaying the transactions of the selected wallet it will switch to the Overview tab.

It only happens in the Transactions tab, the other tabs (Overview, Send or Receive) work as expected.

Expected behaviour

It should display the transactions of the selected wallet instead of switching to overview.

Steps to reproduce

Go to the Transactions tab and select a different wallet.

Relevant log output

No response

How did you obtain Bitcoin Knots

Pre-built binaries

What version of Bitcoin Knots are you using?

Bitcoin Knots version v26.1.knots20240513

Operating system and version

Fedora Workstation 40 (gnome)

Machine specifications

No response

@Tracachang Tracachang added the bug label Jun 9, 2024
@Tracachang Tracachang changed the title Changing between different wallets resets to overview window Changing between different wallets switches to overview window Jun 9, 2024
@Tracachang Tracachang changed the title Changing between different wallets switches to overview window Changing between different wallets switches to overview tab Jun 9, 2024
@luke-jr
Copy link
Member

luke-jr commented Jun 10, 2024

Confirmed. Bug was introduced by bitcoin-core/gui#815, and will be resolved in next version

@luke-jr
Copy link
Member

luke-jr commented Jun 23, 2024

Please confirm this is fixed in 27.1.knots20260621 and close if so

@Tracachang
Copy link
Author

I confirm, fixed

@pablomartin4btc
Copy link

Steps to reproduce

Go to the Transactions tab and select a different wallet.

@Tracachang, just to be clear, you meant: selecting a different wallet by "Open Wallet" menu action? Switching among wallets from the wallets (already loaded) combo box (at the top right section of the main window), won't produce such change in the behaviour. Thanks!

@Tracachang
Copy link
Author

Steps to reproduce

Go to the Transactions tab and select a different wallet.

@Tracachang, just to be clear, you meant: selecting a different wallet by "Open Wallet" menu action? Switching among wallets from the wallets (already loaded) combo box (at the top right section of the main window), won't produce such change in the behaviour. Thanks!

yes, switching among wallets from the wallets (already loaded) combo box (at the top right section of the main window)

@pablomartin4btc
Copy link

yes, switching among wallets from the wallets (already loaded) combo box (at the top right section of the main window)

Mmm... the line that was causing that behaviour of switching from transactions to overview tab is this one:

if (historyAction->isChecked()) gotoOverviewPage();

That's inside the function void BitcoinGUI::enableHistoryAction(bool privacy) which was referenced/ called from 2 places... when the user ticks the "mask values" from the Settings menu (pls click here to watch animated gif).

mask values - switch from transaction tab to overview tab

The idea (introduced in #708) was to move from the transaction tab to the overview tab since it was going to be disabled anyways.

The 2nd place where this function was called was in addWallet (still is in bitcoin-qt but bitcoin-core/gui/pull/815 is removing it), which is called when a new wallet is created or opened/ loaded, and here we can see the undesired secondary effect you were mentioning on this issue (pls click here to watch animated gif).

switch from transaction tab to overview tab when opening wallets

Switching among loaded wallets from the combo box doesn't switch between tabs (this is still in bitcoin-qt, unless there's something in bitcoinknots that causes it and I'm not aware of:

switch wallets from combo dont switch from transaction tab to overview tab

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants