Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive QRBox with Flexbox

duongns-vn 210

@duongns-vn

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

@hannibal1631

Posted

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-vn

Posted

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

0

Please log in to post a comment

Log in with GitHub
Discord logo

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