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

@Barissev

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

madhaus 1,040

@festsnusa

Posted

Hi there! Congratulations for completing this challenge.

Just a few bugs I've noticed while testing your page:

  • your paragraph's font in .topcontainer is different. perhaps, you should apply font-family to your body;
  • i've tested mobile screen on Chrome and everything looks okay. but when i switched to Opera, elements in .title collide w/each other;
  • instead of px you should use rem.

To make your code look more concise, I recommend you to use Pug & SCSS. So, it's possible to minimize your code using mixins and loops: https://pugjs.org/language/mixins.html

Hope you find my feedback useful. Happy coding!

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