Skip to content
Submitted 4 months ago

html and CSS(flexbox) Qr Code Component

kapowlo•80
@kapowlo
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 not sure if giving html,body a height of 100% was better than simply setting body to a 100vh. Since i want my container to take up the full page.

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

I originally had a repository on GitHub called frontend-mentor-challenges, where I was gonna put all my Frontend mentor projects.But after looking a bit online this wasn't recommended.So i'll just stick to one repository per project which ,as a beginner, makes it easier to deploy with GitHub pages.

Community feedback

  • James Atkins•40
    @End-Us3rPosted 4 months ago

    Hey @kapowla, I just got started on this platform so they have me reviewing your code. Anyway, a couple of quick fixes will have you closer to what they want. I would decrease the width of the elements a bit, adjust your border radius for the corners, and make the spacing on the left/right and top of the image element equal to each other. Hope this helps, Good luck!

    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