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]: Make "Remove Watched Videos" constant #5171

Open
3 tasks done
kommunarr opened this issue May 25, 2024 · 0 comments
Open
3 tasks done

[Feature Request]: Make "Remove Watched Videos" constant #5171

kommunarr opened this issue May 25, 2024 · 0 comments
Labels

Comments

@kommunarr
Copy link
Collaborator

Guidelines

  • I have searched the issue tracker for open and closed issues that are similar to the feature request I want to file, without success.
  • I have searched the documentation for information that matches the description of the feature request I want to file, without success.
  • This issue contains only one feature request.

Problem Description

I watch my videos in my Quick Bookmark playlist one at a time, removing the video I'm currently watching and onto the next one. I like to keep my playlists clean of the videos I've already watched, and this is currently one of the most efficient ways to do it.

You might think that the "Remove Watched Videos" feature is perfect for this, but the main problem is that you have to manually click it every single time you want that change to take effect. Counting the prompt, that's two clicks, and you'll have to do them on a regular basis to keep your playlist clean of watched videos. My existing solution on doing it manually on every video as I watch it is better at keeping it consistently "clean," but it requires manual conscientiousness in my flow to remember to check it, and I can't have autoplay enabled because I don't always have the reaction time to do the manual step at the end of a video.

It would be optimal if I could have a "set it and forget it" way of removing watched videos.

Proposed Solution

Add a "Remove Videos As They Are Watched" toggle button. When the toggle is set to false and then interacted with, the existing prompt appears (with language modified from Are you sure you want to remove all watched videos from this playlist? This cannot be undone. to Do you also want to remove all existing watched videos from this playlist? This cannot be undone.). The options are Yes and No (instead of the current "Cancel"). Same as before, if they select "Yes", their watched videos are removed. But regardless of their selection, the toggle is now enabled, and new videos they save will be removed as they watch them.

This new toggle button replaces the "Remove Watched Videos" icon button.

The check for whether to remove the video on watch will be evaluated only when the video is set from Not Watched to Watched, not Watched -> Watched. This means that you can choose to manually keep a video even while you're watching it through the Add to Playlist and Quick Bookmark buttons in watch-video-info, or videos you've already watched.

To check whether the current video is a member of a playlist with this setting enabled, and enact the change for all matching playlists, is O(p * max(pn)), where p is the number of playlists, and pn is the number of videos in a given playlist. This would result in colossal load times on video pages for users who have this setting enabled on multiple large user playlists whenever they Mark as Watched or open up any random video. As a result, there are two ways we can implement this without risking a sizable detriment to the user experience:

  1. Wait until the Vue 3 migration is complete (Vue3 Migration #4428), and we can use Maps and/or Sets to make this O(p * max(pn)) problem an O(p) one, or even better than that in the average case depending on the implementation.
  2. Something a bit more hacky like "only allow one playlist to have this setting enabled" as a rule (not a fan).

I strongly opt for option 1 and keeping this feature request on ice until then.

Alternatives Considered

The Quick Bookmark icon on watch-video-info will have to be disabled if this setting is active on the Quick Bookmark target. The playlists with this setting enabled in the add video to playlist prompt will show as disabled.

Issue Labels

ease of use improvement, improvement to existing feature

Additional Information

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: To assign
Development

No branches or pull requests

1 participant