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

Grid Four card feature

@r-jkrishna

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 feedbacks are welcomed 😊

Community feedback

HYDROCODER 555

@HYDROCODER

Posted

Hey there! Great job on this one. I would suggest the following if you don't mind:

  1. You can put a max-width on your content so that it doesn't stretch at larger screen sizes.

  2. You have put the code for the svg of the logos in the HTML itself. You can just use an image tag to link to the svg from you HTML.

  3. You don't have to put the width of 100vw in your body. Body tag by default has a width of 100%. Hope it helps :).

0

@r-jkrishna

Posted

@HYDROCODER

Thank you for the tips

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