Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

HTML, CSS (Flexbox), GSAP

Hyun Lee 140

@Sagameister

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I tried to incorporate GSAP which I am trying to learn. I would love your feedback. :-) Cheers! Hyun

Community feedback

@boedegoat

Posted

Great work Sagameister !

To fix your accessibility issue, you can change <div class="page-wrapper"> into <main class="page-wrapper">. Also don't forget to include alt attribute to <img> tag.

Overall pretty good, the animation on first load also very nice. That's all from me.

Happy coding

Marked as helpful

0

Hyun Lee 140

@Sagameister

Posted

@boedegoat , Thanks for your feedback! I will apply your accessibility suggestion. Have a great day! :-)

0

@KaiPereira

Posted

Incredible job Sagameister!

If you want to disable the Pinterest icons appearing when you hover over the image just use <meta name="pinterest" content="nopin" /> in your HTML header.

But aside from that it looks incredible, keep going!

0

Hyun Lee 140

@Sagameister

Posted

@KaiPereira, thank you!!

0

Please log in to post a comment

Log in with GitHub
Discord logo

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