Skip to content
Submitted over 4 years ago

four-card-feature-section Using CSS, HTML, Grid

ZAHHARI OTMANE•200
@otmanezahhari
A solution to the Four card feature section challenge
Preview (opens in new tab)Code (opens in new tab)

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!

Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • ApplePieGiraffe•30,525
    @ApplePieGiraffe

    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)! 😁

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