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

Nullify the window when its closed #1022

Closed

Conversation

Jeremy1026
Copy link
Contributor

No description provided.

@tzarebczan
Copy link
Contributor

@Jeremy1026 can you be a little more descriptive in terms of what this fixes? I linked the issue to the PR (with the Zenhub plugin), but would be good to address that in your comments

@Jeremy1026
Copy link
Contributor Author

Sure. The window object currently remains instantiated even after closing it. In macOS clicking the dock icon brings forward all windows of that application. Since the window has been closed, but not removed from memory, macOS still thinks it exists. This fix removes references to the window when it is closed, so that when the dock icon is clicked it will recreate the application window.

This should carry over to Windows/Linux users as well, but needs to be tested on those OS builds to confirm. I don’t currently have those VMs set up to fully test them though.

@tzarebczan
Copy link
Contributor

@Jeremy1026 don't have issues bringing LBRY back up after closing to tray on Windows/Ubuntu.

@tzarebczan
Copy link
Contributor

This might be fixed by #1042 also, @Jeremy1026 can you confirm?

@neb-b
Copy link

neb-b commented Feb 27, 2018

Yeah just tested #1042 and confirmed it fixes #1021

@neb-b neb-b closed this Feb 27, 2018
@Jeremy1026
Copy link
Contributor Author

Looks good to me as well.

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.

4 participants