-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: Support for thumbnails. #2
Comments
Hey @chiraglulla I am planning to pick this up, and will be clubbing this issue with #63 because I feel thumbnails would go great with the Grid view UI wise. What do you think? Also, we have 3-4 options for the thumbnail.
What do you think? I also am considering a default thumbnail for existing videos + for videos where we couldn't extract the thumbnail. |
Regarding the clubbing of 2 issues, I would say go with atomic issues, close them and move forward. Let's achieve our discussed milestone step by step.
I also am considering a default thumbnail for existing videos + for videos where we couldn't extract the thumbnail. |
Alrighty, that makes sense. about thumbnails for the existing videos, yes that was the plan -> running a separate script to extract thumbnails for existing videos, but by the time we do that, they will have the |
Cool. |
Alrighty, I'll raise a draft PR right way. |
Not sure if it applies here. |
No description provided.
The text was updated successfully, but these errors were encountered: