Skip to content

Commit 3677273

Browse files
kanghengliuPurian23
authored andcommitted
fix(player): combine trackid with text identity (#2808)
fixes #2807 (cherry picked from commit c445597)
1 parent 375afaa commit 3677273

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

quickshell/Services/TrackArtService.qml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -184,11 +184,13 @@ Singleton {
184184
const p = activePlayer;
185185
if (!p)
186186
return "";
187-
// Prefer the stable track id; title/artist/album fill in progressively (Chrome).
187+
// Combine trackid with the text identity. trackid alone dedups Chrome's multi-size art
188+
// re-publish, but some players (e.g. KDEconnect) expose a constant trackid across every
189+
// track, so keying on it alone wedged the dedup and new-song art never loaded. Folding
190+
// in title/artist/album busts the lock on a real track change while same-track art
191+
// churn (stable text) stays deduped.
188192
const tid = p.metadata && p.metadata["mpris:trackid"] ? p.metadata["mpris:trackid"].toString() : "";
189-
if (tid !== "")
190-
return tid;
191-
return (p.trackTitle || "") + "" + (p.trackArtist || "") + "" + (p.trackAlbum || "");
193+
return tid + " " + (p.trackTitle || "") + " " + (p.trackArtist || "") + " " + (p.trackAlbum || "");
192194
}
193195

194196
function artReadyFor(player) {

0 commit comments

Comments
 (0)