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

Amanβ€’ 590

@AmanGupta1703

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 would like to hear your feedback! 😊

Community feedback

Elaineβ€’ 11,420

@elaineleung

Posted

Hi Aman, great work here! I think you've done really well with the responsiveness, and the placement of the cards in the tablet view was quite an interesting choice (which I think I like!). Some quick observations for you:

  1. The font weight for the paragraph under the main heading might be a bit too light, so you might want to check the reference again. It should be the same font weight as the text in the cards.

  2. In the desktop view, the "calculator" and "karma" cards in your solution seem to be in a different placement than the ones in the design.

  3. You're using flexbox on the body for centering, which is great; the only thing you're missing is the align-items: center property because right now, the flex-direction is row (by default if you don't declare it specifically), and so the vertical alignment in that case would be done via align-items

Well done on the whole! 😊

Marked as helpful

1

Amanβ€’ 590

@AmanGupta1703

Posted

@elaineleung Thank you! 😊

1
Divine Obetenβ€’ 2,435

@Deevyn9

Posted

Hi Aman, congrats on completing this project, the cards on the mobile view are packed together, there’s no separation. Add a margin bottom to each of the cards to solve this issue.

Happy coding 🎈

Marked as helpful

1

Amanβ€’ 590

@AmanGupta1703

Posted

@Deevyn9 Thank you!

1
Divine Obetenβ€’ 2,435

@Deevyn9

Posted

@AmanGupta1703 you’re welcome

1
Lucas πŸ‘Ύβ€’ 104,560

@correlucas

Posted

πŸ‘ΎHello Aman, Congratulations on completing this challenge!

I liked a lots that you've added multiple layouts for the card grid, this is amazing. But the solution itself its not responsive yet even if you're using media queries.

Replace the width of the card classes to allow it to shrink while scaling to fit better the text content inside:

@media (max-width: 786px)
.banner-box {
    margin: 0 0 1.5rem;
    max-width: 20rem;
}

✌️ I hope this helps you and happy coding!

1

Amanβ€’ 590

@AmanGupta1703

Posted

@correlucas Thank you! 😊

1
Lucas πŸ‘Ύβ€’ 104,560

@correlucas

Posted

@AmanGupta1703 You're welcome Aman =)

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