liblxqt icon indicating copy to clipboard operation
liblxqt copied to clipboard

Published 20 hours ago • verified publisher icon lxqt

Add a timeout to the reboot/poweroff ... confirmation dialog

Open kreijack opened this issue 2 years ago • 6 comments

Add a timeout to the confirmation dialog for

  • poweroff
  • shutdown
  • logout
  • hibernate
  • suspend If the timeout expires, it is considered like the user pressed Ok.

The timeout is set on the basis of the timeout_confirmation settings:

  • -1 -> not timeout at all
  • 0 -> the confirmation dialog disappear instantly
  • 1... -> the dialog closes itself after the timeout set

This patch is inspired to the current behavior in mate-desktop, where the confirmation dialogs have a timeout. Also this patch correct a bug where the method MessageBox::warning ignores title and text

kreijack avatar Dec 10 '22 11:12 kreijack

Gave a look (finally, sorry) at this, it's working fine.

screen_area_sab_08:21:15_

This would need a PR to include in lxqt-config-session for timeout_confirmation= if leave_confirmation=true

@gfgit @tsujan should we include this in LXQt 2.0.0 ? I think it would be nice, as it can easily be disabled and most DEs have it.

stefonarch avatar Feb 10 '24 07:02 stefonarch

Btw I think this message could be changed as it's not true anymore, with many applications which have crash recovery nowadays. "Unsaved work could be lost..."

stefonarch avatar Feb 10 '24 09:02 stefonarch

Btw I think this message could be changed as it's not true anymore, with many applications which have crash recovery nowadays. "Unsaved work could be lost..."

Well it's worse to hope app recover and then they don't do it properly. At least user will double check. To be honest other DE do not have this warning.

Anyway I like this change, only I don't have much time to review it now unfortunately.

gfgit avatar Feb 10 '24 10:02 gfgit

@tsujan should we include this in LXQt 2.0.0 ?

I don't like a timeout.

tsujan avatar Feb 10 '24 11:02 tsujan

I think this message could be changed as it's not true anymore, with many applications which have crash recovery nowadays.

It isn't only about crashes. An editor may or may not save changes on being terminated without prompt. Actually, some devs (like me) may believe that such a saving isn't good because there's no guarantee that the user wants to save a change to a document that he considers good, and saving to files with strange names might be confusing.

tsujan avatar Feb 10 '24 12:02 tsujan

@tsujan should we include this in LXQt 2.0.0 ?

I don't like a timeout.

Others do I guess. I don't use it either but it would not hurt. Default could still be 0.

stefonarch avatar Feb 10 '24 12:02 stefonarch