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
Icons Font Display Swap #10095
Comments
I second this. Not only for internal fonts: also for Google Fonts, TypeKit etc. |
Thirded! |
Fourth! |
add font-display: swap to Font Awesome; improves site score in Google Page Speed Insights (see also: https://web.dev/font-display/ and issue elementor#10095)
On the one hand Lighthouse tells us to use |
I think this issue is fixed in the latest release of font awesome refer the ticket FortAwesome/Font-Awesome#16077 Upgrading the font-awesome version to 5.13 should fix the issue. |
What version is Elementor on now? |
Can we also consider adding |
Also there is already a hook for this so we simply need to append Please this would be a super easy change to hook into your existing filter created for local fonts. |
It won't work the way you think display=swap won't work for font-awesome. Issue is because of font-awesome not with elementor. |
I saw that there were other comments relating to this being added for Google fonts too so that's why I added my comment. Maybe could I suggest you change the ticket title so that it's more clear that it relates to icons? Or just Font Awesome. Cause really it's not solved and the title implies more of a global (applies to all not covered) approach? |
There I have added Icons to the title, like visrey said, Font Awesome has already done the updates to make lighthouse happy, Elementor just needs to stay up to date, Font Awesome resolved this in March. |
This feature has been released in Elementor v3.1. Please feel free to update |
This issue still exists for Elementor icons. Running Lighthouse shows the following:
Confirmed that adding |
Can you please write the code exactly how it should be so I can add to my website. I am not very familiar with coding so your help is most appreciated. thank you |
need same |
Confirmed still an issue |
Indeed, still an issue. Hope this will be fixed soon |
I created #15014 to resolve this issue. |
I tried this directly modifying the css and .min.css files under wp-content/plugins/elementor/assets/lib/eicons/css/ however, it didn't work. I'm not entirely sure why, but my browser is requesting the .woff2 icons with query parameter ?5.10.0 instead of ?5.11.0, and it doesn't work even when adding the new urls. |
I'm having the same issue on my end. |
|
Nevermind it looks like it's also affecting desktop |
It affects desktop and mobile for my website.. all changes on the .css & min.css do not change anything.. so how could we tackle that issue? |
Confirmed still an issue. |
Still getting this error on PSI for elementor 3.4 |
Yes, also still getting this error. Seems to be no way to remove eicons. |
We are planning on releasing a new e-icons loading method that will eliminate this issue (inline SVG) Thanks |
Great news! Looking for it 😉 |
any updates on this? Thanks |
#15897 |
Same problem someone managed to fix it |
Hello! |
Yeah still not fixed. |
Issue looks to be resolved in the most recent version of Elementor 😎
…Sent from my iPhone
On Dec 9, 2021, at 1:12 AM, Jonathan ***@***.***> wrote:
Yeah still not fixed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
|
Does anybody else have problem with the cart icon after this update? |
Just noticed that as well…..
…
On Dec 18, 2021, at 10:30 AM, Salman Mohammadi ***@***.***> wrote:
Does anybody else have problem with the cart icon after this update?
Cart icon no longer show in my client website
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.
|
Hope the fix will be implemented soon. I have tried the Inline SVG method which is currently in alpha stage. That messed up my fonts so I had to revert. |
So, this was closed but not fixed? I mean I'm posting this on June 17, 2023 and the issue is still there ...any workaround here? |
Hello, /wp-content/plugins/elementor/assets/lib/eicons/fonts/eicons.woff2?5.23.0 still issue. |
Prerequisites
What problem is your feature request going to solve? Please describe.
PageSpeed Insights "Ensure text remains visible during webfont load" Fonts being loaded by Elementor such as the icons are causing a 1-second delay in load time.
I am not referring to custom fonts from Elementer Pro, I am referring to the bundled icons/fonts with Elementor, paths such as:
Describe the solution you'd like
Adding
font-display: swap;
Update: Font Awesome did this themselves, just update to the latest version 5.15
Describe alternatives you've considered
I have added the code in myself but they were removed after the update. I don't want to keep manually adding the font display.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: