update some transition animation #320
Conversation
change full progress indication on video list by a check mark instead of a full progress bar sync view history on app start for logged in users change time needed to send view status of a video to invidious from 90 to 10% fix #314
|
Awesome work, yet again! Out of sheer curiosity on my part, is there a particular reason (or set of reasons) that the debounced call to the Invidious History API was increased from five seconds to 10 minutes? Seems like the increased time delay assumes that users are actively using the app for at least 10 minutes. Not saying that this assumption is reasonable or not; just pointing it out for discussion/understanding on my part. Thanks! |
|
I changed from a debounce to a throttle. Meaning that instead of waiting for it to have no calls for x minutes to trigger. It will trigger as soon as it can and prevent new triggers for x minutes. It's easier to do this than keep track of whether I already sent the view status for this video. |
|
Ahhh, nice!!! Thanks for the transparency and great work! |
|
Hey @lamarios , sorry to revisit this, but do you see value in switching from a percentage to time in seconds, with regard to the threshold needed to add a video to a user's watch history? I know this wouldn't affect a majority of users, but it would certainly help those who watch longer form videos (i.e., lectures or streams that go on for hours). For instance, a 2 hour stream implies that the user would need to be watching the video from the beginning for a contiguous 20 min. I suppose this is doable for most, but for longer videos, this may prove more challenging. One workaround is simply scrubbing to a point well last the 10% mark (so that the video gets added to the watch history) and then scrubbing back to approximately where you left off. I don't expect many users to do this, however. Anyhow, what are your thoughts on something like a 5-10 second threshold for adding a video to watch history, assuming the video is at least that long? Eager to hear your thoughts. |
change full progress indication on video list by a check mark instead of a full progress bar
sync view history from invidious on app start for logged in users
change time needed to send view status of a video to invidious from 90 to 10% fix #314