Skip to content

load polyfill after DOM has loaded - #26

Merged
jgerigmeyer merged 5 commits into
mainfrom
check-dom-ready
Sep 14, 2022
Merged

load polyfill after DOM has loaded#26
jgerigmeyer merged 5 commits into
mainfrom
check-dom-ready

Conversation

@oluoluoxenfree

Copy link
Copy Markdown
Contributor

Steps to test/reproduce

Please explain how to best reproduce the issue and/or test the changes locally (including the pages/URLs/views/states to review).

Show me

Provide screenshots/animated gifs/videos if necessary.

REMEMBER: Attach this PR to the Trello card

@netlify

netlify Bot commented Sep 13, 2022

Copy link
Copy Markdown

Deploy Preview for anchor-polyfill ready!

Name Link
🔨 Latest commit c50abcf
🔍 Latest deploy log https://app.netlify.com/sites/anchor-polyfill/deploys/63220946ed49ca0008d0ba6d
😎 Deploy Preview https://deploy-preview-26--anchor-polyfill.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Comment thread src/polyfill.ts Outdated
@oluoluoxenfree oluoluoxenfree changed the title iniitialk implementation load polyfill after DOM has loaded Sep 13, 2022
@jgerigmeyer

Copy link
Copy Markdown
Member

@oluoluoxenfree

Copy link
Copy Markdown
Contributor Author

I tried "load"/onload, and onreadystatechange/'readystatechange'.

i was calling polyfill at the bottom of the document to bypass the button and nothing was happening; I assume it might have something to do with the order in which things are called, but I'll try again to check.

@jgerigmeyer

Comment thread src/polyfill.ts Outdated
Comment thread src/polyfill.ts Outdated
Comment thread src/polyfill.ts Outdated
@jgerigmeyer
jgerigmeyer merged commit 9d8a345 into main Sep 14, 2022
@jgerigmeyer
jgerigmeyer deleted the check-dom-ready branch September 14, 2022 17:58
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.

2 participants