brl/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

mpris: Use a scope specific message instead of a global one

Browse Source 
Since the mpris implementation of the notification tray supports showing
multiple notification (one for each player), it doesn't make sense to
have only one global property to store the message, since that only
allows referencing one message at a time.

Instead, handle the MediaMessages completely inside the scope of
`_addPlayer()`, this should allow showing more than one message again,
which broke with commit 4dc44304df [1].

[1] https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/791

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/833
This commit is contained in:
Jonas Dreßler 2019-11-17 16:24:52 +01:00 committed by Florian Müllner
parent 55362aed3d
commit d5e8f8cdf7
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
js/ui/mpris.js
View File

@ -256,18 +256,20 @@ class MediaSection extends MessageList.MessageListSection {
return;
let player = new MprisPlayer(busName);
let message = null;
player.connect('closed',
() => {
this._players.delete(busName);
});
player.connect('show', () => {
this._message = new MediaMessage(player);
this.addMessage(this._message, true);
message = new MediaMessage(player);
this.addMessage(message, true);
});
player.connect('hide', () => {
this.removeMessage(this._message, true);
this._message = null;
this.removeMessage(message, true);
message = null;
});
this._players.set(busName, player);
}
@ -288,10 +290,7 @@ class MediaSection extends MessageList.MessageListSection {
if (!name.startsWith(MPRIS_PLAYER_PREFIX))
return;
if (newOwner && !oldOwner) {
if (this._message)
this.removeMessage(this._message);
if (newOwner && !oldOwner)
this._addPlayer(name);
}
}
});