Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature request] Visible playlist assignment #4565

Open
Vertux opened this issue Apr 5, 2024 · 2 comments
Open

[Feature request] Visible playlist assignment #4565

Vertux opened this issue Apr 5, 2024 · 2 comments
Labels
feature-request Request of a new feature module:playlists module:watch Watch page-related (other than the player)

Comments

@Vertux
Copy link

Vertux commented Apr 5, 2024

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 Vertux added the feature-request Request of a new feature label Apr 5, 2024
@SamantazFox SamantazFox added module:playlists module:watch Watch page-related (other than the player) labels Apr 20, 2024
@SamantazFox
Copy link
Member

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

@Vertux
Copy link
Author

Vertux commented Apr 20, 2024

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".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request of a new feature module:playlists module:watch Watch page-related (other than the player)
Projects
None yet
Development

No branches or pull requests

2 participants