-
Notifications
You must be signed in to change notification settings - Fork 121
Exit alert ticketbuyer running #1989
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
Exit alert ticketbuyer running #1989
Conversation
@@ -51,23 +56,13 @@ class App extends React.Component { | |||
this.props.showAboutModalMacOS(); | |||
} | |||
}); | |||
} | |||
|
|||
beforeWindowUnload(event) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the whole block as it seemed to have no use with the new exit logic. According to this thread, it's not recommended to override this event. It's also undocumented in electron.
Seems like there's an issue with powerSaveBlocker on macOS, I opened an bug report about it. |
# Conflicts: # app/containers/App.js # app/main.development.js # app/main_dev/templates.js
Ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on linux.
Nice touch on removing the beforeWindowReload()
event!
Resolves #1923
Tasks
prevent & alert when entering suspend mode