-
Notifications
You must be signed in to change notification settings - Fork 61
Show video length #43
Comments
Hey ! This is a GREAT idea so I started hacking away at it and I think I got a pretty solid implementation at #44 You can see the demo here: https://5b0d69eadd6a540f7d24c206--peaceful-banach-108bd4.netlify.com/ What do you think ? |
looks great. in fact its so impt a bing style filter (<20min, 20-60min,
60+++) might be helpful for this exact use case. the TED website does this
too
…On Tue, May 29, 2018 at 10:58 AM Sara Vieira ***@***.***> wrote:
Hey !
This is a GREAT idea so I started hacking away at it and I think I got a
pretty solid implementation at #44
<#44>
You can see the demo here:
https://5b0d69eadd6a540f7d24c206--peaceful-banach-108bd4.netlify.com/
What do you think ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#43 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AiT1gvF3nGk0lCvJKsnoILfTS_HVpOoKks5t3WH-gaJpZM4URf_p>
.
|
It's very helpful ! Once this is merged I will work on that filter and also an order by duration :) |
Hey @SaraVieira, looks great so far, although I think the bold on Do we need the state Also, if we're going with full |
Both of your ideas may be better because I have no idea how to make things plurar in moment :( I'll be flying to Amsterdam today morning but I'll get to this in the afternoon :) |
We tend to sync together to watch frontend related conference talks over lunch, so being able to see how long each video is without having to click to play each video would be helpful to know which videos we can watch given only a certain about of time left at lunch.
The text was updated successfully, but these errors were encountered: