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 Using CSS, HTML, Grid

@otmanezahhari

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


If anybody would like to review my code and give me feedback that would be great!

Any other type of feedback is appreciated as well! Thank You!

Community feedback

P
ApplePieGiraffe 30,565

@ApplePieGiraffe

Posted

Hi, ZAHHARI OTMANE! 👋

Nice job on this challenge! 👏

The main thing I suggest you work on is making your site more responsive. Currently, things look good on purely the desktop and mobile layouts, but there is no styling to take care of what the page will look like on devices with dimensions between desktop and mobile devices (such as on tablets). It's important that you make your solution look good on all screen sizes so that as many people as possible can enjoy your work. 😉

Also, I think you can just use a single <h1> tag for the heading of the page (rather than two different heading tags, since the heading is really just a single heading, not two separate headings). You can easily style the two parts of the heading differently by wrapping one part in a <span> tag. 😉

Keep coding (and happy coding, too)! 😁

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