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

Show Peer Info window only when left-clicking on avatar #6309

Merged
merged 1 commit into from
Aug 17, 2022

Conversation

Android-X13
Copy link
Contributor

Fixes minor glitch when right-clicking on a peer's avatar in PORTFOLIO

Steps to reproduce:

  • PORTFOLIO > HISTORY > Right-click on a peer's avatar > will show both PeerInfoWithTagEditor window and the "Create new offer like this..." context menu overlapping:

1

  • Click on "Create new offer like this..." > if you were maker, this will open the duplicate offer tab while still keeping the PeerInfoWithTagEditor popup visible:

2

  • Right-clicking to another avatar instead > will show a duplicate "Create new offer like this..." context menu without closing the previous one:

3

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

ACK - tested and works perfectly, just a small code suggestion

})
.show());
setOnMouseClicked(e -> {
if (e.getButton().equals(MouseButton.PRIMARY)) {
Copy link

Choose a reason for hiding this comment

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

Suggested change
if (e.getButton().equals(MouseButton.PRIMARY)) {
if (e.isPrimaryButtonDown()) {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you @xyzmaker123. isPrimaryButtonDown doesn't work correctly in this case and Peer Info window won't show up with a simple left click. It will only show up if left button is pressed the exact moment you release an other mouse button on the avatar

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

ACK

Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

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

utACK - based on #6309 (review)

@ripcurlx ripcurlx added this to the v1.9.5 milestone Aug 17, 2022
@ripcurlx ripcurlx merged commit 01ba30a into bisq-network:master Aug 17, 2022
@ghost ghost mentioned this pull request Sep 5, 2022
@Android-X13 Android-X13 deleted the peer-info-window branch September 20, 2022 12:53
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.

2 participants