Skip to content

Commit

Permalink
xapp status: Make sure the recording icon size is set, disconnect
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
mtwebster committed Jun 28, 2024
1 parent 2bfdef2 commit 8d97e56
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ class RecorderIcon {

this._recordListenerId = Main.screenRecorder.connect("recording", () => this._recordingStateChanged());
this._recordingStateChanged();

this.refresh();
}

_recordingStateChanged() {
Expand Down Expand Up @@ -97,6 +99,7 @@ class RecorderIcon {
Main.screenRecorder.disconnect(this._recordListenerId);
this._recordListenerId = 0;
}
this.actor.destroy();
}
}

Expand Down Expand Up @@ -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;
Expand Down

0 comments on commit 8d97e56

Please sign in to comment.