RegisterWindowClosingHandler not working on linux (Debian)
Opened this issue · 3 comments
neurodot commented
I succesfully ported small winform application as a multiplatform desktop app running under Photino.
With one exception - RegisterWindowClosingHandler works on the windows, but not works on the linux.
When I click on the X (close icon on the toolbar), „closing event“ is not fired and when I close the child window (with X), whole application is closed. On the windows it works as expected.
MikeYeager commented
We will look into it as soon as we can find some time. Thanks for the report.
neurodot commented
Super.
IMHO the solution is rather simple. Something as:
g_signal_connect(G_OBJECT(_window), "delete-event", G_CALLBACK(on_delete_event), this);
and
gboolean on_delete_event(GtkWidget* widget, GdkEvent* event, gpointer self) {
t_wwNative* instance = ((t_wwNative*)self);
return instance->InvokeClose();
}
Thanks for your work :)
Kamil
From: Mike Yeager ***@***.***
Sent: Thursday, November 11, 2021 5:28 PM
To: tryphotino/photino.Native ***@***.***>
Cc: neurodot ***@***.***>; Author ***@***.***>
Subject: Re: [tryphotino/photino.Native] RegisterWindowClosingHandler not working on linux (Debian) (Issue #75)
We will look into it as soon as we can find some time. Thanks for the report.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#75 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/AFH74YVBHC547AQ6G7BREPDULPVI3ANCNFSM5HIEW46Q> .
Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub> . <https://github.com/notifications/beacon/AFH74YVTC3RAGIKDDJ2Z33DULPVI3A5CNFSM5HIEW462YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHGNLWSY.gif>
…--
Tato zpráva byla zkontrolována na viry programem Avast Antivirus.
https://www.avast.com/antivirus
ottodobretsberger commented
Thank you for that proposal. If you'd like to test out this suggestion and then wrap it in a pull request, we'd be happy to look at it and integrate this for our next build.