lms icon indicating copy to clipboard operation
lms copied to clipboard

(feature request) Web player: star icon

Open whinette opened this issue 7 months ago • 3 comments

Hello,

I'm using more and more the web player. One of the feature I missed the most is the possibility to quick star (or unstar) a track.

I've made a quick mockup:

Image

In queue list and track lists, having a new icon that reflect the star status of the track (same as in track burger menu star/unstar). Clicking on the icon would change this status.

Would it be possible? :)

whinette avatar Jun 03 '25 07:06 whinette

Hello! Everything is possible :) Will make some test, but I am afraid we would have to hide it on mobile. Would it be an issue?

epoupon avatar Jun 03 '25 08:06 epoupon

It would not!

whinette avatar Jun 03 '25 08:06 whinette

I wonder how this would play together with the 5-star rating supported by the subsonic API (https://github.com/epoupon/lms/commit/29d0610eb2c990818c2f894ea1dab54bd5cc8c94). I really would like to see those ratings in the webui!

cquike avatar Jun 26 '25 23:06 cquike

searched for a rating issue and found this. Came here to add a +1 for 5 star rating of album/song in the open subsonic api. I'm working on a fork of a client and one the requests over there was to add this functionality on the app, which I don't think I can test on LMS - they are using navidrome.

eddyizm avatar Aug 05 '25 20:08 eddyizm

The 5 star rating should be already available in subsonic API!

epoupon avatar Aug 05 '25 21:08 epoupon

ah so the endpoint is there even if it is not reflected in the UI. Does an update get stored in a db in that case? eg album1 0 starts > album1 3 stars ?

eddyizm avatar Aug 05 '25 21:08 eddyizm

User ratings are stored per artist, track, album, and per user. Iirc, in the subsonic API you can list albums sorted by "highest", meaning highest rated first.

epoupon avatar Aug 06 '25 05:08 epoupon

so i was playing with the rating api tonight and was not able to get it to work successfully. using curl to for a simple POC on my end. I get this odd situation where it is saying the ID param is missing but even in the logs, i see the param is clearly being sent across. My first thought was it was the id string but I tried a a track and a release, without any success. Clearly I am doing something wrong yes?

curl -X 'GET' 'http://localhost:5082/rest/setRating?u=lms_dev&p=*REDACTED*&v=1.16.0&c=development&f=json&id=30411&rating=3'

[2025-Aug-07 04:02:53.870] 1 - [error] - 548494891976 [API_SUBSONIC] Error while processing request '/setRating', params = [{c=development}, {f=json}, {id=30411}, {p=*REDACTED*}, {rating=3}, {u=lms_dev}, {v=1.16.0}, ], code = 10, msg = 'Required parameter 'id' is missing.'

eddyizm avatar Aug 07 '25 04:08 eddyizm

The message is a bit misleading, I guess this is because the id itself must be valid (tr-xxx for a track, as reported by other endpoints)

epoupon avatar Aug 07 '25 04:08 epoupon

The message is a bit misleading, I guess this is because the id itself must be valid (tr-xxx for a track, as reported by other endpoints)

I didn't see this anywhere the api docs? Is this specific to lms? I opened up the db as well.

I will try again tomorrow. Thanks for your response!

eddyizm avatar Aug 07 '25 05:08 eddyizm

These IDs are opaque, you are not supposed to guess them: only use IDs that the server provides. This is true for all subsonic servers (some server may even use UUIDs)

epoupon avatar Aug 07 '25 06:08 epoupon

@epoupon yeah that makes sense, I jumped the gun with the api, went from ping to rating instead of getting the album or track list first for that necessary info. lol. Sorry mate!

eddyizm avatar Aug 07 '25 14:08 eddyizm

First attempt: Image

Still not sure about colors (and changing colors between on/off states does not work well here since it is within a button, same as the play button)

epoupon avatar Aug 09 '25 11:08 epoupon