-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Uncaught ReferenceError: Modernizr is not defined #2614
Comments
hi @GeremyHanjra can you give more information on your setup where this error occurs? |
I have encountered this error too. Perhaps this could help: Version: 3.11.4
Attached generated modernizr.js (with npx modernizr): modernizr.zip Location: Modernizr.addTest('localizednumber', function() {...
A null check fixes the issue:
Also issue goes away if I remove "forms/inputnumber-l10n" from configuration. |
Thanks for the hint @arhanta I added your code fix to the v3 brach. Could you check it out and see if it fixes your issue? |
Looks good. I did wonder if there are any other places in code where such a check might be needed, and I see you found one more. Am I somehow using Modernizr improperly, that such a thing (body without parent) happens? |
Good question but I am not that deep into the code. Are you using modernizr just simply on your page or do you add it via iframe or anything special? |
Apologies for not answering this. I will take advantage of the necro and mention that we are just including |
@damian-cell your issue should be fixed (if you need the test) since v3.11.5 |
No description provided.
The text was updated successfully, but these errors were encountered: