Skip to content
Submitted 7 months ago

QR Code Component

PraneethChitturi•20
@PraneethChitturi
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

What are you most proud of, and what would you do differently next time?

I am proud of building it from scratch in HTML and CSS

What challenges did you encounter, and how did you overcome them?

I am frequent user of React, so its pretty easy to build apps in that using, not going much into detail of HTML & CSS. So in this challenge, just using HTML and CSS was challenging.

What specific areas of your project would you like help with?

I am good!

Community feedback

  • Afacanc38•20
    @alperenisaPosted 7 months ago

    You should give the container a meaning class name. If you don't make this as the complexity of the code increases, you'll have trouble with maintaining code. Check it out: https://cssguidelin.es/ https://getbem.com/introduction/

    Also:

    @media (min-width:500px){
    .container{
        width:400px;
    }
}

    Instead using an media query, you can just put .container to max-width: 400px;, it is easier than, and just works.

    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