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 Responsive

Facundo Rubioβ€’ 250

@Facu3071

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


I appreciate it so much any feedback

Community feedback

Samson Shamβ€’ 140

@samsonsham

Posted

Hello Facundo πŸ‘‹ Congratulations for completing this challenge πŸŽ‰ I see you make good use of grid box and calculate the row and column nicelyπŸ‘

Some of my suggestions below:

  • I see you have imported the font family Poppins but I couldn't see you are using it. font-family: 'Poppins', sans-serif; So you might not be intentionally not using the given font, right?
  • I see you put the first part of heading into <span>. It should be part of the heading so is semantically better to put it into h1 as well and add necessary <strong> and line break. And <span> is an inline element so be careful your using might cause some unwanted inline effect.
  • Incorrect icon in Calculator card.

Hope this helps! Have a nice day!

Marked as helpful

0

Facundo Rubioβ€’ 250

@Facu3071

Posted

@samsonsham Thanks you so much for you help. Good vibes Samson!

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