shell icon indicating copy to clipboard operation
shell copied to clipboard

Re-apply window title on/off after screen unlocking

Open fhackenberger opened this issue 3 years ago • 3 comments

Before this change the window titles were still visible after unlocking the screen, even if the setting was to off.

fhackenberger avatar Mar 11 '22 11:03 fhackenberger

This adds methods to pop-shell's public DBus API. Did you mean to add these methods to the extension class instead?

mmstick avatar Mar 11 '22 11:03 mmstick

Before this change the window titles were still visible after unlocking the screen, even if the setting was to off.

I'm not able to recreate this issue on Pop!_OS 21.10, 20.04 LTS, or 22.04 LTS pre-release. I typically test this feature with Evolution, Thunderbird, or LibreOffice. I've tried manually locking, waiting for the screen to time out, and using a sleep command to launch a window while the screen's locked-- in all cases, the titlebar was still hidden when I unlocked the screen again. What OS and application are you seeing this behavior with?

jacobgkau avatar Mar 11 '22 21:03 jacobgkau

I'm not able to recreate this issue on Pop!_OS 21.10, 20.04 LTS, or 22.04 LTS pre-release.

I'm using Ubuntu 20.04 LTS with pop-shell compiled from git master and experience it with Thunderbird for example. Most gnome apps are unaffected these days as they haven't got separate window titles anymore. I lock my screen with Ctrl-Alt-L or suspend the laptop.

This adds methods to pop-shell's public DBus API. Did you mean to add these methods to the extension class instead?

I was unsure where a DBus signal subscription would fit best. I didn't add this method to the DBus interface, so it's not a publicly (over DBus) callable method and prefixed with on... to indicate that it's an event handler. Do you think it should rather be in the Ext class?

fhackenberger avatar Mar 14 '22 10:03 fhackenberger