Skip to content
Submitted over 3 years ago

Responsive Web with HTML & SCSS (using flexbox)

raspow•30
@grtrspw
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

This is my first challange and English isn't my first languange, so i'm sorry for the grammar mistakes.

I think the transition to mobile display is kinda messy. Can i get some feedback for that?

Community feedback

  • Vanza Setia•27,775
    @vanzasetiaPosted over 3 years ago

    👋Hi raspow!

    I have some feedback on this solution:

    • You can increase the max-width value on your @media query. On screen that has more than 375px width, let's say 640px, it should not be on two columns yet.
    • Don't use id for styling, use class instead.
    • Use heading tags in chronological order, like from h1 to h6 also for the stat item you can use p instead of headings, since heading is used for titling section.

    That's it! Hopefully this is helpful!

    Marked as helpful
    0
    raspow•30
    @grtrspwPosted over 3 years ago

    @vanzasetia hey thanks for your feedback and i followed your advice! In my second challange i use class instead id

    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