invidious icon indicating copy to clipboard operation
invidious copied to clipboard

[Feature request] Visible playlist assignment

Open Vertux opened this issue 1 year ago • 2 comments

I think my idea is a little related to #4080 , because it would help to avoid this mistake.

If you add a video to a playlist, the playlist entry gets a check mark:

Bildschirmfoto 2024-04-05 um 08 55 46

When you later view the same video, you will not see that you have already assigned it to a playlist. Under Playlist, you will only see the first playlist created (default).

Bildschirmfoto 2024-04-05 um 08 56 31

Normally the assigned playlist with the check mark should be shown here, as in the first screenshot.

Hint: If a video has not been assigned to a playlist, it may be better not to display a playlist (or e.g. "none") in the drop-down box at all, that way you could eliminate the need for a check mark and it would make the interface clearer and more intuitive.

Vertux avatar Apr 05 '24 07:04 Vertux

If you add a video to a playlist, the playlist entry gets a check mark.

When you later view the same video, you will not see that you have already assigned it to a playlist. Under Playlist, you will only see the first playlist created (default).

There is no check (either client or server side) to know if playlist "P" contains a video "V" on that page. The checkmark is added on the client side when the API request behind "add to playlist" succeeded. As soon as you reload the page, that checkmark disappears.

I'm worried that adding such a check would slow down page load significantly for users with many playlists/videos....

Hint: If a video has not been assigned to a playlist, it may be better not to display a playlist (or e.g. "none") in the drop-down box at all, that way you could eliminate the need for a check mark and it would make the interface clearer and more intuitive.

By default, it shows the first playlist in the list, no matter what. I guess you'd want a placeholder entry like that as the default?

image

SamantazFox avatar Apr 20 '24 13:04 SamantazFox

There is no check (either client or server side) to know if playlist "P" contains a video "V" on that page. The checkmark is added on the client side when the API request behind "add to playlist" succeeded. As soon as you reload the page, that checkmark disappears.

That is interesting, I was not aware, that the check mark is just a temporary thing. I agree with you, if the check takes too long, this comfort feature makes no sense. In this case, it would make more sense to check if the video you want to add is already in the selected playlist and then display an info text e.g. "This video is already assigned to the playlist XY"

I guess you'd want a placeholder entry like that as the default?

Right, that was my idea, I thought it would be more intuitive and less "confusing".

Vertux avatar Apr 20 '24 18:04 Vertux