Fix service based variables.

Jas-SinghFSU · Dec 21, 2024 · 61f5c6e · 61f5c6e
1 parent 4928871
commit 61f5c6e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/src/components/bar/modules/media/index.tsx b/src/components/bar/modules/media/index.tsx
@@ -15,6 +15,8 @@ const { truncation, truncation_size, show_label, show_active_only, rightClick, m
     options.bar.media;
 
 const Media = (): BarBoxChild => {
+    activePlayer.set(mprisService.get_players()[0]);
+
     const isVis = Variable(!show_active_only.get());
 
     show_active_only.subscribe(() => {

diff --git a/src/components/menus/dashboard/shortcuts/buttons/RecordingButton.tsx b/src/components/menus/dashboard/shortcuts/buttons/RecordingButton.tsx
@@ -4,15 +4,17 @@ import Menu from 'src/components/shared/Menu';
 import MenuItem from 'src/components/shared/MenuItem';
 import { hyprlandService } from 'src/lib/constants/services';
 import { isRecording } from '../helpers';
+import AstalHyprland from 'gi://AstalHyprland?version=0.1';
 
-const monitorList = Variable(hyprlandService?.monitors || []);
+const MonitorListDropdown = (): JSX.Element => {
+    const monitorList: Variable<AstalHyprland.Monitor[]> = Variable([]);
 
-hyprlandService.connect('monitor-added', () => monitorList.set(hyprlandService.monitors));
-hyprlandService.connect('monitor-removed', () => monitorList.set(hyprlandService.monitors));
+    const monitorBinding = Variable.derive([bind(hyprlandService, 'monitors')], () =>
+        monitorList.set(hyprlandService.monitors),
+    );
 
-const MonitorListDropdown = (): JSX.Element => {
     return (
-        <Menu className={'dropdown recording'} halign={Gtk.Align.FILL} hexpand>
+        <Menu className={'dropdown recording'} halign={Gtk.Align.FILL} onDestroy={() => monitorBinding.drop()} hexpand>
             {bind(monitorList).as((monitors) => {
                 return monitors.map((monitor) => (
                     <MenuItem