Skip to content
Submitted over 3 years ago

HTML, CSS (Flexbox), GSAP

gsap
Hyun Lee•140
@Sagameister
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

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

Code

Please log in to post a comment

Log in

Community feedback

  • Bhremada Fevreano•1,395
    @boedegoat

    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
  • Kai Pereira•240
    @KaiPereira

    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!

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