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

Responsive Four Card Feature using Flexbox and CSS Grids

Shahin NJ 1,190

@SJ-Nosrat

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


Hi, Thanks for checking my projects.

Kindly provide any constructive feedback on the responsive-ness of the website: from 375px (mobile) right up to 1920px (widescreen).

Did I use CSS Grids and Flexbox correctly? Since, I just learned the technologies.

Also, does my HTMl need any restructing? Or refactoring of my CSS code?

Your help is highly appreciated! Thanks in advance!

Community feedback

P

@MarkoNikolajevic

Posted

Your project looks good and the code is clean and clear. You could replace the hard coded svg using and <img> tag instead. For example <img src='./images/icon-karma.svg' alt='karma icon' />

1

Shahin NJ 1,190

@SJ-Nosrat

Posted

@MarkoNikolajevic Thanks for the quick feedback!

Appreciate the tip on adding the svg's!

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