Skip to content
Submitted over 3 years ago

Stats preview card

Aarish Fakih•10
@aar-ish
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

Happy to get a feedback

Community feedback

  • Ivan Villarosa•55
    @IMVillarosaPosted over 3 years ago

    Nice work man! for your mobile layout - you may can try using media queries and set it to max-width: 375px so that when it reaches the specific mobile width, it will then use the styling within that media query.

    Marked as helpful
    1
    Aarish Fakih•10
    @aar-ishPosted over 3 years ago

    @IMVillarosa Thanks for the feedback I’ll add that media query.

    0
  • dewslyse•3,025
    @dewslysePosted over 3 years ago

    Hello Aarish 👋! Congrats on your submission.. Looking really nice.

    • You may have forgotten about the mobile design. Other than that, everything looks fine.
    • Since you have a stylesheet, why not move the inline style on the html body tag there <body style="font-family: 'Inter' sans-serif">
    • You could also wrap your content in a main tag to address the accessibility issues
    Marked as helpful
    1
    Aarish Fakih•10
    @aar-ishPosted over 3 years ago

    @dewslyse Thanks a lot for the feedback I will surely do the changes.

    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