Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Four card feature section

Hesham Ali 190

@HeshamAliAhma

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Alex 3,130

@Alex-Archer-I

Posted

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

@HeshamAliAhma

Posted

thank you for helping me@Alex-Archer-I

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord