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

3-column preview card component

LiBee 390

@Li-Bee

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Feedback on my code please - is there anything i could improve on?

One question i did have was why i had to do overflow:hidden on the .card-section class to show the border-radius - can anyone explain this please?

Community feedback

Han 340

@hcxweb

Posted

Because it's the parent container and you set border-radius of 8px on it, which will allow the child content to overflow.

Marked as helpful

2

LiBee 390

@Li-Bee

Posted

@hcxweb thanks for the explanation

0
Ayush Nath 360

@Beats-Ayush

Posted

@Li-Bee Excellent work completing the challenge. A couple of places where you might want to improve your code

  • Put the screen-reader class outside of the main section so that it does not interfere with your content.
  • To increase responsiveness, use max-width to restrict the width of the card container.
  • border-radius simply gives a rounded edge to the border. It doesn't affect the div container.

Marked as helpful

1

LiBee 390

@Li-Bee

Posted

@Beats-Ayush thanks for the advice much appreciated.

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