Product Preview Card using Flexbox and CSSGrid

Community feedback

• 

  Muhammadh•1,140

  @Mr-jawPosted almost 2 years ago

  Hello there 👋

  Congratulation on completing the challenge

  HTML 📄

  • avoid using <span> tag to display text. Since the <span> tag is mostly used to group inline elements. rather you could have used <h1> tag instead of <span class="category">PERFUME</span> and <h2> tag instead <h1 class="product-title">Gabrielle Essence Eau De Parfum</h1> to increase accessibility.

  • rather than using <p> tag for <p class="special-price">$149.99</p> and <p class="regular-price">$169.99</p>. you may consider using <em> or <strong> or <small> tags for short text, Since<p> is used to descriptive text

  These are issues related with lack of Semantic HTML and accessibility

  I hope this was helpful 😊

  KEEP GRINDING

  Marked as helpful

  1

  ---

  

  Angel Estevez•100

  @AEPEX25Posted almost 2 years ago

  @Mr-jaw Thanks for the feedback, I’ll take note about those suggestions.

  0