Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Product preview card component challenge

@dev-pedroalmeida

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@WesSno

Posted

Replace the <h5> with <h1> to make the page/content accessible. Every html must have <h1> to make it accessible. Also, never skip a level of heading. The heading must always begin with <h1> tag. Wrap the sub-heading of <h1> in <h2> tag and that of the <h2> tag in <h3> tag. This continues until <h6>.

HOPE AM HELPFUL.

Marked as helpful

1

@dev-pedroalmeida

Posted

@WesSno Thanks a lot for the tips, i didn't consider the accessibility when making this. Also should the old price <h5> be replaced by a <p> tag or something else?

0

@WesSno

Posted

@dev-pedroalmeida You can replace both the old price and new price with <p> tags

Marked as helpful

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord