shell/applets/MprisApplet.qml

@@ -306,14 +306,20 @@ Column {
     // Player switcher
     Item {
         width: root.width
-        height: root.players.length > 1 ? _playerFlow.implicitHeight + 6 : 0
+        height: root.players.length > 1 ? 28 : 0
         visible: root.players.length > 1
 
-        Flow {
+        Flickable {
-            id: _playerFlow
+            id: _switcher
-            anchors.horizontalCenter: parent.horizontalCenter
+            anchors.centerIn: parent
-            anchors.verticalCenter: parent.verticalCenter
+            width: Math.min(_playerRow.implicitWidth, parent.width - 16)
-            width: parent.width - 16
+            height: 22
+            contentWidth: _playerRow.implicitWidth
+            clip: true
+
+            Row {
+                id: _playerRow
+                height: 22
                 spacing: 6
 
                 Repeater {
@@ -331,6 +337,7 @@ Column {
                         color: _active ? S.Theme.base02 : (pHover.hovered ? S.Theme.base02 : "transparent")
                         border.color: _active ? root.accentColor : S.Theme.base03
                         border.width: _active ? 1 : 0
+                        anchors.verticalCenter: parent.verticalCenter
 
                         Text {
                             id: _pLabel
@@ -356,4 +363,5 @@ Column {
                 }
             }
         }
+    }
 }

shell/services/MprisService.qml

@@ -1,7 +1,6 @@
 pragma Singleton
 
 import QtQuick
-import QtQml.Models
 import Quickshell.Services.Mpris
 
 QtObject {
@@ -14,6 +13,7 @@ QtObject {
 
     property string _selectedIdentity: ""
     property var _lastPlayedTime: ({})
+    property var _watchers: []
 
     onPlayerChanged: {
         if (player)
@@ -33,19 +33,34 @@ QtObject {
         }
     }
 
-    // Watch all raw players for state changes via Instantiator -
+    // Watch raw Mpris.players for add/remove - this does NOT fire on state changes
-    // delegates persist across state changes since the model only
+    property Connections _mprisConn: Connections {
-    // changes on player add/remove, not property changes
+        target: Mpris
-    property Instantiator _stateWatchers: Instantiator {
+        function onPlayersChanged() {
-        model: Mpris.players
+            root._reconnectWatchers();
-        delegate: QtObject {
-            required property MprisPlayer modelData
-            property Connections _conn: Connections {
-                target: modelData
-                function onPlaybackStateChanged() {
-                    root._onPlaybackStateChanged(modelData);
         }
     }
+
+    function _reconnectWatchers() {
+        for (const w of _watchers)
+            w.destroy();
+
+        const raw = Mpris.players.values ?? [];
+        const newWatchers = [];
+        for (const p of raw) {
+            const conn = _watcherComp.createObject(root, {
+                target: p
+            });
+            newWatchers.push(conn);
+        }
+        _watchers = newWatchers;
+    }
+
+    property Component _watcherComp: Component {
+        Connections {
+            function onPlaybackStateChanged() {
+                root._onPlaybackStateChanged(target);
+            }
         }
     }
 
@@ -69,6 +84,7 @@ QtObject {
                 }
             }
         } else if (p.playbackState === MprisPlaybackState.Paused && p === player) {
+            // Current player paused - switch to a playing one if available
             _switchToPlaying();
         }
     }
@@ -123,4 +139,6 @@ QtObject {
         if (players[idx])
             _selectedIdentity = players[idx].identity ?? "";
     }
+
+    Component.onCompleted: _reconnectWatchers()
 }