-
Notifications
You must be signed in to change notification settings - Fork 314
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
requestIdleCallback not available in Safari #124
Comments
Ah dang. I had no idea 😬 Safari really is the new IE 🙃 Yeah, a PR would be welcome. Let's just do this: const rIC = requestIdleCallback ?? setTimeout
rIC(() => {
// ... etc.
}) |
Yeah, I had to double check to make sure it's not supported. 😂 I can create a PR, but checking the polyfill linked in MDN, it seems like we should add a timeout of 1 milliseconds (see |
Sounds good 👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce
Expected Behavior
It should mount React
Actual Behavior
It doesn't mount React, get an error in console:
Unfortunately
rIC
is not supported by Safari, according to caniuse.com. Either its existence should be checked or it should be polyfilled. (Or it can be removed altogether.) I can provide the PR depending on the decision.The text was updated successfully, but these errors were encountered: