lxqt / lxqt-session

The LXQt session manager
https://lxqt.github.io
GNU Lesser General Public License v2.1
57 stars 52 forks source link

Prevent screen locking before suspend without X11 #468

Closed tsujan closed 1 year ago

tsujan commented 1 year ago

Because it would cause a crash under Wayland.

Fixes https://github.com/lxqt/lxqt-session/issues/457

tsujan commented 1 year ago

@stefonarch Would you please test it under Wayland and X11 — and especially under X11, with and without that option checked?

tsujan commented 1 year ago

... as I'm not using xscreensaver.

Sorry, I wanted to say "especially under Wayland" but wrote "especially under X11" — the code isn't changed for X11. Anyway, you tested it under both :)

I think we could add a lineedit in lxqt-config-session: "Screenlock command under wayland: " at some point when it's clear where and how to insert a wayland pane there.

Yes, of course.

tsujan commented 1 year ago

Superseded by https://github.com/lxqt/lxqt-session/pull/469