HTML and CSS with Flexbox

Community feedback

• 

  Szymon Rojek•4,540

  @SzymonRojek

  Posted over 4 years ago

  Hi Sam,

  Well done :D

  I have checked mainly your HTML structure, a few tips from me:

  • instead of div container you can use the main tag;
  • I think the header doesn't need to be here. I'd recommend using the section tag and inside three divs (here divs are perfect for generic groupings of content);
  • you did great with h1 tag but also there is another way to use it: h1 and inside of it two spans (main-heading: Join our community and sub-heading: 30-day, hassle-free money back guarantee);
  • instead of <h3>$29 <span>per month</span></h3> I'd recommend to use the p and inside of it two spans for the price and per month;
  • instead of h4 put the p tag;
  • remember: move headings down the levels gradually;
  • well done with the link, but also you can add additional attribute title,
  • in the last box instead of p will be better to use ul > li;
  • text per month should be in the middle of the price;
  • something wrong with this effect on :hover (check it out).

  In the end, I'd recommend reading about semantic tags and headings => it is very easy to overuse them.

  Greetings :D

  22