Skip to content
Submitted over 3 years ago

Simple HTML + SCSS solution, used flexbox

Łukasz Kiełbasa•80
@arctindev
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

I would be happy for any feedback, trying to improve my css skills to start playing around with css-in-js in future

Community feedback

  • Riyana Gueco•495
    @rnguecoPosted over 3 years ago

    Hey Łukasz!

    Great job on this. 👍

    I have only one thing to point out: when viewing the site on mobile landscape, the image starts overflowing towards the top and gets cut off. This is because you've set your body's height to 100vh which means it will not expand to accommodate overflowing content. What you want instead is a min-height of 100vh.

    Cheers and happy coding! 🎉

    Marked as helpful
    1
    Łukasz Kiełbasa•80
    @arctindevPosted over 3 years ago

    @rngueco Ty very much for feedback :D noticed the mistake n solved it just as u said :D also added some 10vh margins on mobile viewport to improve card overall visibility :D Cheers!! Have a great day!

    0
  • Patrick•14,265
    @palgrammingPosted over 3 years ago

    Nice Job every thing looks really good ⭐⭐⭐⭐⭐

    1

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