Skip to content
Submitted 8 months ago

CODIGO-QR

daniel22s•40
@daniel22s
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?

Que estoy logrando memorizar cosas de css que no conocia

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

los div para centrar y hacerlo responsive

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

a

Community feedback

  • Ian Parker•200
    @i-prkrPosted 8 months ago

    The solution is very close! Using flex boxes may have made the centring of the component and sizing easier rather than absolute, try updating the wrapping div's with display: flex, and for example setting the image width as 100% and setting some padding around it to centre it in the top of the card.

    The responsiveness is great and works really well, and the mobile version looks great too. Your structure in css is clear and great use of px and em. The HTML could be tidied a little with large gaps but indented correctly and easily readable. The image is missing an 'alt' for accessibility (I also forgot to add that).

    Nice work!!

    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