Skip to content
Submitted over 3 years ago

Mobile-first approach using flexbox

JJ•25
@j-j-martin
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

Hey there!

This was my first challenge on here, i had a lot of fun.

I didn't really structure my css code in a way that makes sense I guess. Next time i will be using BEM, but even then, is there a right way to structure the code inside of a declaration block? As in sorting declarations alphabetically or anything?

Any feedback is greatly appreciated!

Community feedback

  • Account deleted

    Hello j-j-martin , 👋

    Congratulations on completing your first challenge👏

    Your solution looks almost perfect.

    I have also completed this challenge, you can checkout my solution.

    Great work here 😊

    Have fun coding 💻

    1
    JJ•25
    @j-j-martinPosted over 3 years ago

    @Nik-web12 Hey Nikhil, thanks a lot! I'll check it out as soon as I'm home. 😊

    0
  • Kunal•105
    @kunals741Posted over 3 years ago

    Hey Martin, I checked your solution, you have done a good job. My suggestion is that you should use relative values to set margin/padding etc like rem instead of fixed value(px).

    GOOD JOB!

    1
    JJ•25
    @j-j-martinPosted over 3 years ago

    @kunals741 Ok thanks a lot! I'll try it next time :)

    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