Manage landing page using JS Vanilla, SASS, BEM and Swiper JS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @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 the alt 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 helpful

  0

  ---

  

  Yaiza•410

  @Yaiza16Posted over 3 years ago

  Hello, @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