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

Selected peer in the "Peers" tab gets unselected if any previous peer disconnects #160

Closed
hebasto opened this issue Dec 22, 2020 · 1 comment · Fixed by #164
Closed

Selected peer in the "Peers" tab gets unselected if any previous peer disconnects #160

hebasto opened this issue Dec 22, 2020 · 1 comment · Fixed by #164
Labels
Bug Something isn't working UX All about "how to get things done"

Comments

@hebasto
Copy link
Member

hebasto commented Dec 22, 2020

On master(209974d) the selected peer in the "Peers" tab gets unselected if any previous peer (in the upper rows) disconnects.

Step 1 -- connected three peers, and "peerC" has been selected:
Screenshot from 2020-12-22 15-04-19

Step 2 -- new connected peers ("peerD") do not hurt:
Screenshot from 2020-12-22 15-04-30

Step 3 -- but disconnection of any peer from upper rows (e.g., "peerB") causes the loss of peer selection:
Screenshot from 2020-12-22 15-05-28

@hebasto hebasto added the Bug Something isn't working label Dec 22, 2020
@hebasto hebasto added the UX All about "how to get things done" label May 1, 2021
@jarolrod
Copy link
Member

jarolrod commented Jun 7, 2021

This bug no longer operates in the way described. I have not investigated fully, but I believe that #18 fixed this bug as it is currently described. There is still a bug related to peer selection that exists and it is fixed by #164.

New Bug:
on master, if you select a peer, and a peer above it disconnects or is disconnected, the selection will slip to the next available peer.

Select Peer 6 Disconnect Peer 0, Peer 7 is now selected
Screen Shot 2021-06-06 at 9 17 11 PM Screen Shot 2021-06-06 at 9 17 48 PM

As shown in the screenshots above, we start by selecting Peer 6: 149.248.3.82:38333. Peer 0 is then disconnected, our selection then slips to Peer 7: 88.99.101.67:38333.

#164 Fixes New Bug:
So while the bug has changed, I can confirm that #164 fixes this.

Select Peer 9 Disconnect Peer 1, Peer 6 remains Selected
pr-start pr-disconnect

The screenshots above were taken on Linux on branch #164. We can see that we start with Peer 9: 149.248.3.82:38333, then we disconnect Peer 1. Our selection remains with Peer 9 and does not slip.

@hebasto hebasto closed this as completed in e638acf Jun 7, 2021
@jarolrod jarolrod mentioned this issue Jun 28, 2021
2140data added a commit to 2140data/gui that referenced this issue Aug 3, 2022
Line 73 - Adding a whitespace character (&bitcoin-core#160;) on the QT Receive Coins form instructions in order to show a space after the period in the application.
@bitcoin-core bitcoin-core locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Something isn't working UX All about "how to get things done"
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants
@jarolrod @hebasto and others