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

[GUI] send widget, hide contacts menu when clear event is triggered. #2161

Merged
merged 1 commit into from
Feb 6, 2021

Conversation

furszy
Copy link

@furszy furszy commented Jan 26, 2021

Fixing crash reported in #2160 .

Inside the send screen the "clear all" event removes every row and there by every entry from the widget, as the contacts menu stores a local pointer to the entry where is being open and the data being pointed does't exist anymore, it was occasioning a crash there.
The solution is simple, close the menu contacts when the clear button is pressed.

random-zebra
random-zebra previously approved these changes Jan 27, 2021
Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

ACK 519fbf7

Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

re-utACK 64abc55

Copy link
Collaborator

@Fuzzbawls Fuzzbawls left a comment

Choose a reason for hiding this comment

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

ACK 64abc55

@furszy furszy merged commit 4bee55e into PIVX-Project:master Feb 6, 2021
@furszy furszy deleted the 2020_fix_clear_bugfix branch November 29, 2022 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants