Tagged: logo
Hey guys,
There is some hard coded image size tags related to the logo that need to be looked at. You can see it on any site, including your demo site for the 2017 theme. Here is the code in question:
<img height="100" width="300" src="https://kriesi.at/themes/enfold-2017/wp-content/uploads/sites/66/2017/02/logo.png" alt="Enfold 2017" />
`
The problem is, logo’s aren’t all 100×300. So the wrong image sizes are hardcoded in no matter what image size is selected. 80×80 logo selected, but 100×300 being output in code for no reason.
See an example on my site in private area below.
Can you please look at correcting this so that the correct image size values are output?
Thanks so much
Hey THP,
Thanks for contacting us!
According to the comment it is basically just for better page speed ranking :)
I will ask our devs if we can add a filter to it :)
Best regards,
Yigit
Hi Yigit,
Sorry, for some reason I never got notified of your reply, so I just saw it now.
Yes it is important to have the width and height attributes in there, but they shouldn’t be arbitrary ones. They should be coming from the logo size selected, surely.
This helps with things like content layout shift, which will be a ranking factor in a few months time.
Thanks
Hi,
Thanks for the heads up! I have checked with our devs and they agreed to add a filter for it :)
Cheers!
Yigit
Hey, I saw that the new srcset for logo isn’t working due to this issue, but I think your next release includes a fix, right?
If so, this can be closed.
Hi,
Yes, the fix will be included in upcoming version – https://kriesi.at/support/topic/enfold-upcoming-fixes/
fixed: responsive images for logo not working
Closing this one. Please feel free to start a new thread if you have any other questions or issues :)
Best regards,
Yigit