Skip to content
Submitted almost 2 years ago

First time RWD, training only HTML and CSS

HipsterShaggy•170
@Jakub-Gryczka
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

The difficult part of the project was correct image size 😅. I couldn't make this one div wider than others, but I think I handled it. Not how I would like to, but it's ok in my opinion. That project was also my first time using responsive design. I am unsure about this image, if someone would explain me how to make it bigger without modyfing .wrapper padding it would be nice. Also if something isn't best practice you can tell me what, because I am just starting 😀. Thanks for feedback!

Community feedback

  • Noam Weiss•30
    @noamweisssPosted almost 2 years ago

    Hi, I like your solution! I would use a more semantic HTML (main tag, section tags etc.) and I love your positioning and use of CSS variables. Good Job!

    0
    HipsterShaggy•170
    @Jakub-GryczkaPosted almost 2 years ago

    @noamweisss Thanks!

    1

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