Skip to content
Submitted 12 months ago

Responsive QR Code Component using Flexbox and CSS Grid

node
Undertesting21•10
@Undertesting21
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

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

I'm most proud of how the QR Code Component looks on different screen sizes. Using Flexbox and CSS Grid made it easy to create a responsive design that works well on both mobile and desktop devices. Next time, I would explore using CSS custom properties to manage colors and spacing more effectively for better maintainability.

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

One of the main challenges was ensuring the component stayed centered on all screen sizes. Initially, I struggled with alignment issues, but I overcame this by utilizing CSS Grid's centering properties. Additionally, setting up Git and pushing the project to GitHub was a bit tricky, but following detailed guides helped me get everything configured correctly.

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

I'd appreciate feedback on the responsiveness of the component. Specifically, I'm curious if there are more efficient ways to handle the layout adjustments for different screen sizes. Any tips on optimizing the CSS for better performance would also be highly valuable.

Code

Please log in to post a comment

Log in

Community feedback

  • P
    Daniel 🛸•44,790
    @danielmrz-dev

    Hello there!

    Congrats on completing the challenge! ✅

    Your solution is really impressive!

    I've got a couple of ideas (about how to use HTML better) that could make it even stronger:

    📌 First: Think about using <main> to wrap your main content instead of <div>.

    Imagine <div> and <span> in HTML as basic containers. They're good for holding stuff, but they don't tell us much about what's inside or its purpose on the webpage.

    📌 Second: Consider using <h1> for your main title instead of <h2>.

    It's more than just text size — it's about structuring your content effectively:

    • The <h1> to <h6> tags are used to define HTML headings.
    • <h1> is for the most important heading.
    • <h6> is for the least important heading.
    • Stick to just one <h1> per page – it should be the main title for the whole page.
    • And don't skip heading levels – start with <h1>, then use <h2>, and so on.

    These tweaks might not change how your page looks, but they'll make your HTML code clearer and help with SEO and accessibility.

    Hope that's helpful!

    Keep up the great work!

  • Guerinik Abderrahmane•460
    @AbderrahmaneGuerinik

    Hey, you did well in general but there is some details that you can improve :

    • add margin bottom and border radius to the image
    img {
   border-radius: 20px;
   margin-bottom: 1.5rem;  
  }

    For responsivity of the container div you can do this :

    .attribution {
  width: min(95%, 350px)
  }

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