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

Large playlists causing crashes #247

Closed
WardPearce opened this issue Jul 21, 2023 · 3 comments · Fixed by #258
Closed

Large playlists causing crashes #247

WardPearce opened this issue Jul 21, 2023 · 3 comments · Fixed by #258

Comments

@WardPearce
Copy link

Large playlists (150+ videos) tend to cause Clipious to lag or crash due to excessive memory use. This tends to happen on lower end devices like Smart TVs, probably due to all the videos being loaded all at once.

@lamarios
Copy link
Owner

From my test it is a tv only problem, will look into it.

@WardPearce
Copy link
Author

From my test it is a tv only problem, will look into it.

Yea most likely, probably as simple as adding a "Load more" button or smartly loading more once a user scrolls X amount of items down. Not sure if this is implemented on the mobile front, but if not I'd imagine low end mobile phones might have a similar issue.

lamarios added a commit that referenced this issue Aug 1, 2023
add handling of remote media buttons (play/pause/skip etc...)
add exponentially increasing fast forward / rewind if the remote is pressed rapidly or held for long

fix #247
fix #257
@lamarios
Copy link
Owner

lamarios commented Aug 2, 2023

the issue was that on TV it is using a non scrollable grid which loads all the widgets, while on mobile it uses a scrollable list which loads widgets when needed.

I did this to so there can be more screen space on tv to show the videos. I found a not so elegant alternative, I'll see if i can think of something else before i release this end of the week.

lamarios added a commit that referenced this issue Aug 2, 2023
add handling of remote media buttons (play/pause/skip etc...)
add exponentially increasing fast forward / rewind if the remote is pressed rapidly or held for long

fix #247
fix #257
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 a pull request may close this issue.

2 participants