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

Slow loading on Firefox 78.0.2 #46

Closed
Adamantcheese opened this issue Jul 27, 2020 · 7 comments
Closed

Slow loading on Firefox 78.0.2 #46

Adamantcheese opened this issue Jul 27, 2020 · 7 comments

Comments

@Adamantcheese
Copy link

While the widget does eventually load, it seems to take almost exactly 30 seconds to load on Firefox when in Google or Chrome it loads instantly. Not sure if there's anything you can do to remedy that or if it's a Firefox issue, which I suspect it is.

@rb28z2
Copy link
Owner

rb28z2 commented Jul 27, 2020

Sounds like something might be timing out. Are you able to provide console logs while the page is trying to load?

@Adamantcheese
Copy link
Author

Adamantcheese commented Jul 28, 2020

Here's everything that the console spits out; note that the "Brand New Animal" and "12/12" lines appear exactly as the graphics load in, which again, is almost exactly 30 seconds after that red error block. (Updated image with better info)
image

@rb28z2
Copy link
Owner

rb28z2 commented Jul 28, 2020

Hmm yeah I was able to reproduce this on FF as well. It seems to be hanging on the load of progressbar.min.js for some reason. I'm not sure why this is, so it may take some time for me to figure it out and push out a fix. I may be able to work around it by using a CDN to load that specific file over serving it locally but its something I'll have to explore.

Honestly this is going to be fairly low priority as I'm more interested in rewriting this whole project as its been long overdue.

@Adamantcheese
Copy link
Author

Adamantcheese commented Jul 28, 2020

I wonder if it would be as simple as moving the script declarations around so that progressbar.min.js comes before updater_functions.js in the HTML so that it's loaded before any updates occur?

@rb28z2
Copy link
Owner

rb28z2 commented Jul 28, 2020

Hmmm that's a good idea and certainly worth trying.

@rb28z2
Copy link
Owner

rb28z2 commented Jul 28, 2020

@Adamantcheese I've made the changes and updated the version that's running on the website. It seems to load correctly on FF for me now - please take a look and let me know if the issue persists.

@Adamantcheese
Copy link
Author

Working correctly now for me! Thanks for the update!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants