From 8d97e5689f7bea670fb8882d54f48f91e33f04bf Mon Sep 17 00:00:00 2001 From: Michael Webster Date: Fri, 28 Jun 2024 14:40:54 -0400 Subject: [PATCH] xapp status: Make sure the recording icon size is set, disconnect the recording signal when the applet is removed. The indicator had the correct size when Cinnamon started because it would receive an icon-theme-changed signal. If it got reloaded, that signal wasn't sent, so refresh() was never called on the indicator box. --- .../cinnamon/applets/xapp-status@cinnamon.org/applet.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/files/usr/share/cinnamon/applets/xapp-status@cinnamon.org/applet.js b/files/usr/share/cinnamon/applets/xapp-status@cinnamon.org/applet.js index c2da75955c..417d17b19e 100644 --- a/files/usr/share/cinnamon/applets/xapp-status@cinnamon.org/applet.js +++ b/files/usr/share/cinnamon/applets/xapp-status@cinnamon.org/applet.js @@ -39,6 +39,8 @@ class RecorderIcon { this._recordListenerId = Main.screenRecorder.connect("recording", () => this._recordingStateChanged()); this._recordingStateChanged(); + + this.refresh(); } _recordingStateChanged() { @@ -97,6 +99,7 @@ class RecorderIcon { Main.screenRecorder.disconnect(this._recordListenerId); this._recordListenerId = 0; } + this.actor.destroy(); } } @@ -620,7 +623,7 @@ class CinnamonXAppStatusApplet extends Applet.Applet { delete this.ignoredProxies[key]; }; - this._recording_indicator.actor.destroy(); + this._recording_indicator.destroy(); this._recording_indicator = null; this.monitor = null;