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: Mark as Watched button in Feed #1581

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

crhallberg
Copy link

@crhallberg crhallberg commented Oct 18, 2022

This PR adds a button next to the Listen and Add to Playlist buttons on feed.

  • The button appears when Hide Watched is active.
  • The PR also separates Hide Watched from Store Watch History, for users that want to only manually mark videos in their Feed.
  • Right now this PR doesn't offer functionality to unmark a video as watched.

Addresses at least part of #1512.

src/App.vue Outdated Show resolved Hide resolved
src/components/PreferencesPage.vue Show resolved Hide resolved
src/components/VideoItem.vue Outdated Show resolved Hide resolved
src/components/VideoItem.vue Outdated Show resolved Hide resolved
@Bnyro
Copy link
Member

Bnyro commented Oct 18, 2022

Wouldn't it be better to use an icon, e.g. an eye for the button instead of using plain text?

@FireMasterK
Copy link
Member

I feel we also need a new three-dot menu for playlist actions and this since otherwise, the UI is going to get cluttered.

@crhallberg
Copy link
Author

crhallberg commented Oct 20, 2022

Thanks for the feedback! Gonna work on this a bit more soon.

Wouldn't it be better to use an icon, e.g. an eye for the button instead of using plain text?

Not sure what you mean, this was styled with a Font Awesome eye icon to match the listen and add to playlist buttons.

@Bnyro
Copy link
Member

Bnyro commented Oct 20, 2022

Not sure what you mean, this was styled with a Font Awesome eye icon to match the listen and add to playlist buttons.

Oh, sorry, my bad, I just saw you added translation strings for it and just wondered if an icon wouldn't be better in that case, didn't see you already use one ;)

@crhallberg
Copy link
Author

Ah! The translation strings were just for titles!

@crhallberg
Copy link
Author

Resolved the merge conflict. Looking forward to a new round of feedback.

Copy link
Member

@Bnyro Bnyro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've quickly fixed some issues caused by merging which prevented it from working for me, looks good to me now!

@crhallberg
Copy link
Author

Apologies for the long hiatus! I have resolved the conflicts of this PR, hoping we can finish it off!

Comment on lines +217 to +223
if (this.item.watched) {
let request = store.delete(videoId);
request.onsuccess = function () {
instance.$emit("update:watched", [instance.item.url]);
};
return;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we delete the item, then updateWatched won't really update the watched property if someone marks a video as unwatched. So, the video will still show up as watched until they reload the page.

@UltraBlackLinux
Copy link

Coming from invidious I only miss this one feature strongly. There is no other way to clean up your feed without this button and it has really helped me to get rid of FOMO, so please consider merging this PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants