Skip to content
Submitted almost 3 years ago

Mobile first design

ccbdp•315
@ccbdp
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Any feedback is highly appreciated. Thank you!

Community feedback

  • Web Wizard•5,690
    @rsrclabPosted almost 3 years ago

    Hi, @ccbdp ~

    Congratulate on your solution to the challenge on FM platform. I have studied your work carefully and learned a lot from it.

    Here are some of the tips I like to provide.

    • On smaller devices, card goes over screen, and I think max-width: 100% can solve this issue.
    • Please try BEM for naming element classes. It will help you a lot on bigger projects.

    Cheers ~

    Marked as helpful
    0
    ccbdp•315
    @ccbdpPosted almost 3 years ago

    @tymren608 Thank you for the feedback! I'll definitely look into that to further improve my work!

    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