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

window close event does not get triggered anymore #913

Open
4 tasks done
bastimeyer opened this issue Oct 15, 2022 · 1 comment
Open
4 tasks done

window close event does not get triggered anymore #913

bastimeyer opened this issue Oct 15, 2022 · 1 comment

Comments

@bastimeyer
Copy link
Member

Checklist

Streamlink Twitch GUI version

v2.2.0

Streamlink version

No response

Operating system, environment and configuration details

.

Description

Since the upgrade of NW.js from 0.64.1 to 0.68.1 in 3d641ad (part of the v2.2.0 release), the close event on the main application window doesn't get triggered anymore, and neither seem SIGTERM/SIGINT signal callbacks on the global process instance (not used in the application code). This means that the confirmation dialog won't be shown when trying to close the application while streams are still running.

// listen for the close event and show the dialog instead of strictly shutting down
nwWindow.on( "close", function() {
if ( location.pathname !== "/index.html" ) {
nwWindow.close( true );
return;
}
try {
setVisibility( true );
setFocused( true );
nwjs.close();
} catch ( e ) {
quit();
}
});

I've already had a quick look at the issue, but this needs more debugging with a proper reproduction, so the issue can be reported upstream.

Debug log

No response

@bastimeyer
Copy link
Member Author

nwjs/nw.js#7981

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant