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

Suggestion: Decorate videos with their status (unlisted, deleted, privated, currently live, shorts, members only) #662

Open
JeffreyMercado opened this issue Jan 14, 2023 · 5 comments
Labels
backlog Difficult / Very low priority enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@JeffreyMercado
Copy link

This is a great tool for being an archive of a channel's videos and finding other related videos. Occasionally videos will be removed, change status or other.

You'll kinda know that a video was deleted or otherwise made inaccessible if the thumbnail doesn't show. You'll kinda know that a video is members only if it has the topic membersonly shown in the thumbnail but I'm not sure how reliable of an indicator that is. But there should be something more explicit and reliable.

Can the status of a video be saved for a video? Given the known status of a video, make it filterable and have icons or labels indicating it in the UI.

I'd imagine refreshing an entire channel periodically would be infeasible, maybe on visit by a user when the data is loaded after some amount of time since the last check, a check is made with youtube (since it's already embedding, I'm assuming other APIs are being used) and updated if there's changes.

@JeffreyMercado JeffreyMercado changed the title Suggestion: Decorate videos with their status (unlisted, deleted, privated, shorts, members only) Suggestion: Decorate videos with their status (unlisted, deleted, privated, currently live, shorts, members only) Jan 14, 2023
@NeloBlivion
Copy link
Member

NeloBlivion commented Jan 14, 2023

So two parts here:

  • Adding new icons for all the statuses seems good on paper, but I feel like this could run the risk of cluttering the UI a bit too much with the amount of potential indicators this would introduce.
  • I can see the merit in being able to filter by statuses, and it is supported by the /videos endpoint so it's certainly feasible, but it might not fit well in the current v2 UI and instead could be saved for v3 where the design has a lot more chance to evolve (unless someone else has a better concept in their head than I).

Some elaboration on statuses, as it stands videos on Holodex have 5 statuses:

  • new being videos that have just been crawled and are queued for their data to be updated
  • upcoming for any future stream/premiere
  • live for any live stream/premiere
  • past for any videos, shorts, and past stream/premieres
  • missing for anything that isn't publicly available; these are both privated and deleted streams

While there is some gradual backcrawling, video statuses generally won't update some time after being uploaded/turned into an archive. As for your indicator suggestions:

  • Members Only and Shorts have existing topics that can be searched by, which double as indicators
  • Live is fairly obvious already
  • Deleted and Privated can typically be seen at a glance because of the thumbnail as you mentioned (unless it's cached, but this clears quickly); adding an indicator here wouldn't be useful
  • While Unlisted is a separate type received from YouTube, it's not present on Holodex; whether it's something that's necessary could be argued, but the way I see it there's not much reason to differentiate it from standard videos

TL;DR Status filtering/searching could work at some point, but adding indicators for every status seems extra.

@JeffreyMercado
Copy link
Author

JeffreyMercado commented Jan 14, 2023

But these "statuses" are already mutually exclusive so only one would be shown at a time, or if that's not the case, show a single icon based on some priority. I don't think every status needs an icon necessarily, just that there should be some visual distinction. Maybe deleted or privated videos titles would have more gray text than the usual style.

Here's some quick mockups:
imageimageimage

One nice thing about this is that we're freeing up the topic text for members videos and knowing which videos are no longer publicly visible.

Other statuses to consider either provided by youtube or synthetic, I'm not sure if there's any more other than age-restricted, perhaps "long" (over 2:30 hours or something).

@NeloBlivion
Copy link
Member

One nice thing about this is that we're freeing up the topic text for members videos

This is a very good point, but rather than creating an entire new status just for membersonly this issue will be solved in v3 when videos can have multiple topics.

I don't think every status needs an icon necessarily, just that there should be some visual distinction

But that's the exact problem I'm seeing here; these videos are already visually distinct enough on the UI; adding icons isn't functionally useful, because I reckon the first thing the user would see is the other indicators like the thumbnail being missing or the topic text.
Using the style of a recent addition one I can see potentially working is missing which is an easy frontend change, but the rest I don't really think so.
image

I'm not necessarily against the idea, but it needs far more exploring to add anything here that requires proper changes on the backend (e.g. new status types). Though again, and I suppose this is starting to sound like a cop out but I can't really do much about that, this idea could be explored more for v3. Also you're always welcome join the Discord where's there's generally more discussion, though of course you don't have to.

@sphinxrave
Copy link
Collaborator

Displaying a trash sign on something that's already missing seems a bit extraneous (are we calling this video trash?).

Maybe instead of the youtube default video thumbnail we can show a more interesting 'missing' / 'not available' thumbnail instead? Open for ideas

@sphinxrave sphinxrave added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers backlog Difficult / Very low priority labels Jan 20, 2023
@NeloBlivion
Copy link
Member

Maybe instead of the youtube default video thumbnail we can show a more interesting 'missing' / 'not available' thumbnail instead?

This is actually a really cute idea. IIRC we used to have an aqua 404 image or something when the site was down? We could edit that into a default missing thumbnail, and this is extra work but perhaps we could seek out org-specific variants of it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Difficult / Very low priority enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants