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

Fixed DOMContent loaded, Updated test to account for document.readySt… #18

Closed
wants to merge 2 commits into from

Conversation

gkweb
Copy link

@gkweb gkweb commented Dec 13, 2018

Fixes #10.

This will cover use case where scripts are included with async as in - <script src="some/bundle.js" async></script> OR loaded inline. - As there is now a check for document.readyState.

If there is ever a need to disable this loading immediately on initialisation - The options.onLoad flag can be set to false for manual control.

Sorry to stalk you @jshjohnson - Just building some street cred and I really your vanilla js packages! 👍

Copy link

@orditeck orditeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR only requires a minor version adjustment to be ready to be merged

@@ -1,4 +1,4 @@
/*! animate.js v1.4.0 | (c) 2018 Josh Johnson | https://github.com/jshjohnson/animate.js */
/*! animate.js v1.4.1 | (c) 2018 Josh Johnson | https://github.com/jshjohnson/animate.js */
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be 1.4.2 now

@orditeck
Copy link

orditeck commented Jul 8, 2019

This PR should also be rebased.

@jshjohnson
Copy link
Owner

I've rewritten this library in TypeScript and have included your fix :) thanks.

@jshjohnson jshjohnson closed this Oct 24, 2019
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 this pull request may close these issues.

DOMContentLoaded and asset loaders
3 participants