Skip to content
Submitted over 2 years ago

HTML, CSS (Flexbox), GSAP

Hyun Lee•140
@Sagameister
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

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

Community feedback

  • Bhremada Fevreano•1,395
    @boedegoatPosted over 2 years ago

    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
    @SagameisterPosted over 2 years ago

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

    0
  • Kai Pereira•240
    @KaiPereiraPosted over 2 years ago

    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
    @SagameisterPosted over 2 years ago

    @KaiPereira, thank you!!

    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