Review of "Advanced Media Controller" version 6

Details Page Preview

Media controls for the GNOME Shell panel. Works with any MPRIS player (Spotify, VLC, Rhythmbox, browsers) and lets you switch between several players from tabs in the popup. * Play, pause, skip, shuffle and repeat * Track and artist in the panel with a scrolling label * Optional vinyl record effect on the album art, set per app (double-click the art) * Time-synced lyrics fetched from lrclib.net (single-click the art) * Seekable progress slider with selectable themes * Panel label font, colour and element order settings

Extension Homepage
https://codeberg.org/sanjai-shaarugesh/Advanced-media-controller

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental)

Shexli has not produced a result yet.

All Versions

Version Status
6.6 (44) Active
6.6 (43) Active
6.6 (42) Rejected
6.6 (41) Rejected
6.6 (40) Active
6.6 (39) Rejected
6.6 (38) Rejected
6.6 (37) Rejected
6.6 (36) Rejected
6.6 (35) Rejected
6.6 (34) Rejected
6.6 (33) Rejected
6.6 (32) Rejected
6.5 (31) Active
6.5 (30) Rejected
6.5 (29) Rejected
6.5 (28) Rejected
6.5 (27) Rejected
6.5 (26) Rejected
6.5 (25) Rejected
6.5 (24) Rejected
6.0 (23) Rejected
6.0 (22) Rejected
6.0 (21) Rejected
20 Active
19 Rejected
18 Rejected
17 Active
16 Rejected
15 Active
14 Active
13 Active
12 Active
11 Rejected
10 Rejected
9 Rejected
8 Active
7 Active
6 Rejected
5 Rejected
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

sanjai posted a review
fixed the issues and removed unnecessary files
JustPerfection rejected
Thanks! 1. #4 from previous review hasn't been addressed here. You should mention about clipboard usage in the description. 2. Regarding #12 from previous review, there is no timeout connected in that class (line 290-298 `utils/indicator/IndicatorEventHandlers.js`). 3. Timeout should be disconnected on destroy: - line 251 `utils/mpris/MprisPlayer.js` - line 58 `utils/indicator/IndicatorPlayerHandlers.js` 4. Better to call `this._panelUI.destroy()` instead since you may have more in the `destroy()` in the future updates (line 168 `utils/indicator.js`).