Skip to content
Submitted 8 months ago

QR code component page using flexbox

mkhantk•200
@mkhantk
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'm proud of doing most of the work done by myself. I'll try to improve to handle this kind of work single-handedly.

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

I did encounter my html body only taking half of the height and I got to google how to fix it. Only then, I was able to start with the flexbox.

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

I would like to ask seniors for feedbacks. My code is a mess as it seems to be. So... How can I make it better overall? I appreciate it, thanks.

Community feedback

  • Kendpr•290
    @RealKendprPosted 8 months ago

    I suggest to use a fluid size for your .wrapper. Instead of using a 300px fixed width you can try to use %.

    Marked as helpful
    0
  • Aryarishi Sikdar•270
    @ArryBlackPosted 8 months ago

    Hi @mkhantk, great job! The code seems good. You can try making the wrapper div into a main tag for more semantic markup.

    Marked as helpful
    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