google / xsecurelock

X11 screen lock utility with security in mind
Apache License 2.0
863 stars 65 forks source link

Rename XSECURELOCK_SAVER_STOP_ON_DPMS #157

Closed vincentbernat closed 12 months ago

vincentbernat commented 1 year ago

The correct name should be XSECURELOCK_SAVER_STOP_ON_BLANK. We also starts children when screensaver is on, but screen is not blanked, or when screensaver is one and screen is blanked, but this option is not enabled.

I can do a separate pull request for this one-line change if you prefer. Then, I get the same behaviour than #139 by setting XSECURELOCK_BLANK_TIMEOUT=-1.

anthraxx commented 12 months ago

@divVerent any updates, do you want to pull this in? I'm asking as I'd also like to request if a new release can be issued which contains this feature? It has been a rather long time now.

divVerent commented 12 months ago

Oh sorry, I missed this. Yes, this change is uncontroversial, getting it in.

divVerent commented 12 months ago

Thank you!