Single Price Grid Component Solution

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  👾Hello again Faruk, Congratulations on completing this challenge!

  Great solution, the component its beautiful and full responsive, amazing! Something to fix is the html markup, Instead of using div blocks that doesn't have any semantic meaning you can use <article> that's a better fit for this case and add one <h1> heading to show which is the main title of this page.

  This article from Freecodecamp explains the main HTML semantic TAGS: https://www.freecodecamp.org/news/semantic-html5-elements/

  ✌️ I hope this helps you and happy coding!

  1

  ---

  

  Faruk Ahmad•150

  @farukwebcode21Posted about 2 years ago

  @correlucas Thanks

  0
• 

  Ahmed Kamel•1,190

  @hmadamkPosted about 2 years ago

  • Well done acouple of notes though
  • make sure that all of your page is contained by a landmark examples of landmarks are

  header for the top section
  main for your main section
  section with aria-label to describe why you added this section
  footer for the bottom section
  

  for the main section it's required to make users with disabilities to be able to skip right into the content of your site

  0

  ---

  

  Faruk Ahmad•150

  @farukwebcode21Posted about 2 years ago

  @hmadamk thanks for your suggestion

  0