WidgetExt::hide_on_delete useless in current form

Question

sophie-h opened this issue 4 years ago · comments

fn hide_on_delete(&self) -> bool

should be connectable to

fn connect_delete_event<F: Fn(&Self, &Event) -> Inhibit + 'static>(
    &self,
    f: F
) -> SignalHandlerId

which doesn't work out that well.

Sebastian Dröge · Answer 1 · Mon May 25 2020 02:51:27 GMT+0800 (China Standard Time)

Yes it should return an Inhibit, needs a manual implementation for that. Do you want to create a PR?
As workaround for now you can do

window.connect_delete_event(|window| Inhibit(window.hide_on_delete(window)))

Sophie Herold · Answer 2 · Mon May 25 2020 06:02:21 GMT+0800 (China Standard Time)

Actually, |window, _| :)

I will have a look at the manual implementations tomorrow. Doesn't sound that complicated

Sebastian Dröge · Answer 3 · Mon May 25 2020 15:03:54 GMT+0800 (China Standard Time)

Ok :) Let me know if you have questions