Design comparison
Solution retrospective
I tried to incorporate GSAP which I am trying to learn. I would love your feedback. :-) Cheers! Hyun
Community feedback
- @boedegoatPosted almost 3 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 includealt
attribute to<img>
tag.Overall pretty good, the animation on first load also very nice. That's all from me.
Happy coding
Marked as helpful0@SagameisterPosted almost 3 years ago@boedegoat , Thanks for your feedback! I will apply your accessibility suggestion. Have a great day! :-)
0 - @KaiPereiraPosted almost 3 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
Please log in to post a comment
Log in with GitHubJoin 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