Skip to content
Submitted over 3 years ago

Sass-HTML-Responsive

Sky•120
@sky-1991-sia
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Stats preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

almost everything is ok, I solved the class names issue, but I need some tips and tricks for better coding this challenge and more useful responsive design, please check my code and give advise me to refactor it in best way thank you

Community feedback

  • Yashasvi Singh•890
    @aUnicornDevPosted over 3 years ago

    You have used the image in an image tag. I would recommend using it as a background-image because it gives a lot more flexibility in terms of responsiveness. Helps with the overstretching and squeezing of the image.

    Like you have set a max-height for the card, set a max width as well... don't wanna extend the card to the full viewport width. can have a max width at 1200px for 1920px screen. It Looks good .. I've tried it.

    Keep Coding😊

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

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