Skip to content
Submitted 8 months ago

qr-code

Daniel Wiśniewski•30
@TheWicha
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 don't know tbh :) Maybe i would use tailwind CSS

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

only with Figma, but i finally found out what fonts are there

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

in this project i dont know :)

Community feedback

  • Guilherme•40
    @guilhermepatriarcaPosted 8 months ago

    Great work :)

    These are my suggestions for improvement:

    • Use semantic HTML to replace div with article for example
    • Use alt in tag <img>
    Marked as helpful
    0
  • Alex Blue•70
    @fake-alex-bluePosted 8 months ago

    I like the approach you took to this solution. It's clear you've made some different stylistic choices, using Montserrat for the h2 text, for example - which I can't argue with personally; I think it looks good.

    The only small thing I'd mention, is that you've not added the box-shadow on the card, which is a shame; I reckon it elevates the design of the original; although to be honest it's so subtle that I completely missed it on my first attempt at the challenge.

    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