RegisterWindowClosingHandler not working on linux (Debian)

[neurodot]

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]

We will look into it as soon as we can find some time. Thanks for the report.

[neurodot]

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]

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.