- This topic has 11 replies, 4 voices, and was last updated 1 year ago by David.
April 4, 2020 at 11:13 am #1226589Anil
GPP not populating logo size. Please guide.April 4, 2020 at 1:12 pm #1226710LeoStaffCustomer Support
Can you try setting a logo width option?
Let me know 🙂April 4, 2020 at 7:44 pm #1226890Anil
I did, but it is not generating image dimensions…April 5, 2020 at 3:53 am #1227115DavidStaffCustomer SupportApril 5, 2020 at 4:44 am #1227155Anil
Hi David, it is not resolved, disabled cache.April 5, 2020 at 6:28 am #1227217DavidStaffCustomer SupportApril 5, 2020 at 10:00 am #1227562Anil
It is 40×40.
I believe function for logo output not populating the image dimension…April 5, 2020 at 11:24 am #1227650DavidStaffCustomer Support
aah ok – Tom provides a filter to add them here:
Unfortunately you will need to add the sizes to the function.April 5, 2020 at 9:31 pm #1227994Anil
Thanks David, it works.
I feel it should be automatically done by the function there, we should not provide the dimension every time we change the size of logo.
Can we add dimension automatically in above by reading the size of logo?April 6, 2020 at 5:51 am #1228351DavidStaffCustomer Support
As the logo is resized using CSS it shouldn’t really matter what sizes you use.
Especially if it is just to get rid of the GTMetrix notice.
BTW: GP will add width=100 and height=100 to the img if you also add a Retina Image – this is to get round a legacy bug in Firefox. That may be your simplest fix.April 8, 2020 at 7:27 am #1231198Jesse
Does this need to be done differently for sticky nav?
Is there a way to get this into the theme to handle it by default, its little things like this that can be the difference between 99% and 100% in gtmetrix 😉April 8, 2020 at 8:41 am #1231453DavidStaffCustomer Support
You can use the Navigation Logo here:
Or just add a Retina logo to the Site Identity and it will automatically add ‘some’ dimensions.
- You must be logged in to reply to this topic.