Four card feature section

Community feedback

• 

  Alex•3,130

  @Alex-Archer-IPosted 4 months ago

  Hi!

  Your work is quite neat, congrats with finishing it =)

  I can give a couple of suggestions.

  First of all, a few semantic tips. There should be only one h1 element on the page. And it should be inside main tag, not header. Top level header is for page navigation, site logo and stuff. Here all content should be inside main (and you actually doesn't need a wrapper for it).

  And I want to say, that if you used grid that you didn't have to wrap two middle elements in additional wrapper. It's cool if you wanted to practice flex more, I just suggest you a bit more convenient way =)

  Hope that helps. Keep doing cool stuff =)

  Marked as helpful

  1

  ---

  

  Hesham Ali•190

  @HeshamAliAhmaPosted 4 months ago

  thank you for helping me@Alex-Archer-I

  1