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

"No items found" message during slow loading of large schedules #119

Open
mcdemarco opened this issue Aug 18, 2022 · 4 comments
Open

"No items found" message during slow loading of large schedules #119

mcdemarco opened this issue Aug 18, 2022 · 4 comments

Comments

@mcdemarco
Copy link
Contributor

In development, I frequently reload and see the "no items" message while the schedule is loading. You can also see this message whenever freshly loading the page for a large schedule like Chicon's.

"No items found" should ideally be a "Loading" message instead, unless loading the schedule has failed or rendering is complete.

I was reminded about this issue by Peter and some other people who had long delays loading the Chicon schedule on slower machines than mine. They checked the load times and the delay was actually mostly in rendering the large schedule rather than downloading it.

For the render time issue itself, it might be helpful to start with a filter that selects the first day of the con before con and the current day during con, not unlike Konopas's approach, in lieu of implementing more complicated paging or scrolling behavior.

@lostcarpark
Copy link
Owner

Working on adding a loading screen (and maybe a spinner).

I've also been working on optimisation, and have found several areas I can significantly improve the load time.

Also looking at the render time. I think I can slightly improve that, but reducing the number of items displayed seems to be the easiest fix. Default the day filter to the current/first day seems attractive. However, that would mean that unless you remove the day filter, searches or other filters would only be selecting from that day.

My quick fix solution is to add a "maximum number of items to display" drop-down. If I default it to 100, it makes the site pretty responsive. People can show more items if they need to.

@mcdemarco
Copy link
Contributor Author

I like the idea of the item count filter, but it’s already confused someone who expected paging. Would it be possible to copy just the item count filter to the bottom of the page?

@LVerhulst4321
Copy link
Collaborator

Oh good idea! I'll look into doing that.

@lostcarpark
Copy link
Owner

I also added a "Load More" button to the bottom of the page, which adds the next batch of items to the page.
In future I'd like to change to use an "infinite scroll" display. However, currently the program items are nested inside timeslots, which are nested inside days, which doesn't work well with infinite scroll.

I think the issue as originally described could be considered resolved at this point.

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

No branches or pull requests

3 participants