Single price grid component (HTML + CSS)

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • You are overusing the section element. A simple div will do.

  • There are only three headings in this component; “Join our community”, “Monthly Subscription”, and “Why us”. Everything else would be wrapped in a paragraph element.

  • The button was created with the incorrect element. When users click on the button they should directed to a different part of your site; the anchor element will allow this to happen.

  More Info:📚

  MDN The Anchor element

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding!🎄🎁

  Marked as helpful

  1

  ---

  

  Ahmed Arafa•120

  @nerdy-guyPosted almost 2 years ago

  @vcarames Thanks for your feedback. Regarding <section> aren't they better than div for semantic and accessibility? Should I replace all of them with div?

  0

  

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

  @nerdy-guy

  The section element really has no semantic meaning. It is the same as a div. the only difference is that the section element is used to define groups of related content.

  So for this challenge you "could" wrap the entire component in a section element since they are all related.

  Marked as helpful

  1
• 

  Harsh Kumar•1,390

  @thisisharsh7Posted almost 2 years ago

  Hi Ahmed Arafa, well done! Your solution looks perfect.

  Your web page is well designed and it is responsive at different widths. You use flexbox and grid in your solution which is the base for any responsive layout. Your codes are clear and easy to be read by anyone. Your work is excellent.

  Marked as helpful

  1

  ---

  

  Ahmed Arafa•120

  @nerdy-guyPosted almost 2 years ago

  @thisisharsh7 Thank you so much! Harsh Kumar.

  0