Missing alt tag text on Site Logo image Google PageSpeed Insights flag

This topic was automatically closed 365 days after the last reply. New replies are no longer allowed.
9 years ago
Hi support, running some optimization tests on PageSpeed Insights and other engines, we are getting a lower valuation due to this warning (referred to the Logo image plugin)

image has no alt text!
This image is not following part of the Google guidelines. Add alt text.
image has no dimensions (height/width)!

I can't find a way to correct this issue on VS2013
any help solving this will be appreciated
Thank you
================================================================================
PS. We posted it first in Nop-Template forum, see it here

Its Support answer:
Hello,

By default from nopCommerce the logo image comes with empty alt attribute.
<img title="" alt="" src="@Url.Content(logoPath)">

I admit that we haven`t saw that and this is really big fallback from nopCommerce.
You can suggest this to nopCommerce, as we are going to do, so they may fix it.
We are starting to fix it in all our themes.
Thanks for reporting this issue.

Best Regards !
9 years ago
Agree about alt/title. Thanks a lot. I've jsut created a work item

But "image has no dimensions (height/width)" is quite weird suggestion. Nobody follows this suggestion nowadays
9 years ago
Right Andrei, but Google desk minions think that is cool to nitpick to assert their gendarme power on us :-) just kidding!
9 years ago
Done. Please see changeset c2c6a24c3036
9 years ago
a.m. wrote:
Done. Please see changeset c2c6a24c3036


Hi Andrei,

I think it should be localized.

string storeName = EngineContext.Current.Resolve<IStoreContext>().CurrentStore.GetLocalized(x => x.Name);


p.s: There are other places in 3.4 where CurrentStore.Name is used directly rather than with a localized value i.e ProductTemplate.xxx.cshtml files.

Thanks
9 years ago
Hi,

Thanks a lot. Fixed.
This topic was automatically closed 365 days after the last reply. New replies are no longer allowed.