-
Notifications
You must be signed in to change notification settings - Fork 12.2k
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
Bug: 6.7.0 "ReferenceError: Can't find variable: X" Desktop Safari #20424
Comments
I'm also having this problem. No icons showing at all for Safari users, on a wide range of versions (13-17 at least). Rollbar is reporting that this is also on iOS. Started on the 18th. I can't see evidence of a release of Font Awesome that might have brought this on - what's going on? [Update] Ah no your website says 11th November, but Github says 18th November, so it's release 6.7.0 for sure. |
@johnnorris @8bitmatt can you send an email (ask for Rob) to hello@fontawesome.com with your Kit tokens? I need to see how those are configured because I have a hunch what's going on here. |
Experienced the same problem. Pinned our version to 6.6.0 in the fontawesome console for now. Let me know if there's anything we can share to help debugging @robmadole 👍 |
Same issue here. But whole page is just blank (probably just implementation), with reported error in console. Pinned to v6.6.0 for, will see if it resolves it. EDIT: Pinning to 6.6.0 fixed it. So definitely something with v6.7.0 or later. |
Update for those interested. It looks like the minifier we are using for some of the JavaScript files went haywire/has a bug. I was able to fix a customer's Kit by switching to the unminified version. I don't have a root cause fix yet but we're working on it. |
Bug description
Seems to affect all versions of Safari before 18
pro.min.js
a=X[e][t];
Reproducible test case
Should be able to use any kit with 6.7.0 and see the error
Screenshots
No response
Font Awesome version
v6.7.0
Serving
Kit
Implementation
CSS
Browser and Operating System
Web bug report checklist
The text was updated successfully, but these errors were encountered: