Skip to content
Submitted 5 months ago

Responsive QRBox with Flexbox

duongns-vn•210
@duongns-vn
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?

The effort to write concisely and easily to understand, ensuring class reuse and uniqueness

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

How to make it as neat as possible, but still ensure comprehensibility. Make class names unique

Community feedback

  • Rupak Mukherjee•1,070
    @hannibal1631Posted 5 months ago

    Great job mate!

    Well i looked at your code, and it looks neat to me. Although one thing i would suggest you is to keep the image div separate from the content div, cause you made one img div and put the contents inside that. Just make one container div(in your case the main), make two div image-box and image-content and you're good to go.

    Hope this helps and happy coding.

    1
    duongns-vn•210
    @duongns-vnPosted 5 months ago

    Dear @hannibal1631. Thank you for helpful advice! I will listen and improve.

    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