Manage landing page using JS Vanilla, SASS, BEM and Swiper JS
Design comparison
Solution retrospective
Feel free to give any feedback. It will be appreciated 😊
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hello there, Yaiza! 👋
This looks pretty cool! 😀 Nice work on this challenge! 🙌 I really like the animation you added to the heading in the hero section and the other extra details you included (such as the sticky header navigation) are a nice touch, too! 👍
I would just like to suggest you take a look at your solution report and try to clear up some of the errors that are there in order to improve the accessibility of your solution. Remember to add
alt
attributes to any<img>
tags you use (but to leave thealt
text as an empty string if you don't want it to be read by screen readers) and to wrap any list item elements you use in a list tag. 😉Keep coding (and happy coding, too)! 😁
Marked as helpful0@Yaiza16Posted over 3 years agoHello, @ApplePieGiraffe !! Thank you so much for you comment and your advice!! :)
I always forgot about checking the accessibility issues before uploading the solution. I'll modify my code as soon as I can in order to improve it.
Thank you:)
1
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