feature(storefront): BCTHEME-142 Unnecessary heading on product cards

[yurytut1993]

What?

We have product link wrapped in h4 tag which is unnecessary. It can be possible just to remove h4, but I decided to change h4 to span for saving style class which was setted on h4, and now it is setted on span.

Also I should mention that typography styles little bit changed after disabling h4. But we don't have heading already - so we don't need heading styles. I think it is correct - now product name link has similar typography styles with other links in cornerstone.

Tickets / Documentation

Ticket

Screenshots (if appropriate)

main page grid cards

compare page cards

brand page cards

[bigbot]

Autotagging @bigcommerce/storefront-team @davidchin

[rowleyaj]

@yurytut1993 will this change have any impact on SEO due to the removal of headers from the page and no relative importance applied to the product names?

I haven't done much research but dropping headers which wrap product titles seems pretty significant

old article (2017) https://www.velsof.com/blog/search-engine-optimization/how-to-organize-the-header-tag-structure-on-an-ecommerce-site