With HTML, CSS and Flexbox

Community feedback

• 

  

  Matt Studdert•13,611

  @mattstuddert

  Posted over 5 years ago

  Hey Tobi, congrats on submitting your first solution! You've done a really good job. Your code isn't very messy at all! After having taken a look I've got a few pointers that will hopefully help:

  • For your . heading class, I would recommend not using a section here. A section is typically used to group a larger collection of themed content. For example, a testimonials section on a page. For this smaller grouping of content, a div would be absolutely fine.
  • You've got h1 and h3 headings but no h2. Be sure to avoid skipping heading levels unnecessarily, as this can cause accessibility issues.
  • You don't need the div wrapping the anchor tag. You could just style the anchor tag directly.
  • In the Why Us area, I'd use a ul with li elements to create a list instead of having separate p tags for each item.

  You've done a really good job. Let me know if you have any questions 🙂

  22
• 

  Saurav Chamoli•370

  @sauravchamoli17

  Posted over 5 years ago

  Well done!on your project.The improvements you can make in this project are: 1. the per month text should be centered with related to the $29 text.Add a box shadow and hover state to your sign up button.Add a box shadow and border radius to your main container. Move the why us subtext in an unordered list instead of an paragraph and remove padding from them.

  Keep up the good work!

  21