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

First Challenge outside program

@XaviKluyskens

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


All feedback is more then welcome! Just started out.

Community feedback

P
Matt Studdert 13,611

@mattstuddert

Posted

Nice work on this challenge, Xavi! You've done a really good job getting close to the design. As you can see in the comparison above there are just a few small things like font-size and some spacings that need ironing out. But that's me being very picky! 🙂

Here are some thoughts after taking a look at your code:

  • You're duplicating a lot of your styles in your four declaration blocks for the boxes. A nice way to stop that duplication would be to take the repetitive styles and put them in a more generic class called something like .box. You can then add this class to each box to add those styles. This will remove a good few lines of code.
  • A great next step would be to try making the site responsive!

Keep up the great work. Let me know if you have any questions 🙂

0

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