Responsive landing page using scss

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted about 2 years ago

  Hello Shena, You have done a good work! 😁

  Some little tips to improve your code:

  • add main tag and wrap the section of page for improve the Accessibility
  • use article tag instead of a simple div to the container of bottom card of the page (lastest articles) for improve the Accessibility
  • add descriptive text in the alt attribute of the images
  • remove all unnecessary code, the less you write the better as well as being clearer: for example the div container of image
  • add transition on the element with hover effect
  • instead of using px use relative units of measurement like rem -> read here

  Keep learning how to code with your amazing solutions to challenges.

  Hope this help 😉 and Happy coding!

  Marked as helpful

  0

  ---

  

  Shena•150

  @wshenaPosted about 2 years ago

  @denielden hello, thanks for your advice. I've only been learning frontend for 2 months and don't understand about accessibility. I got new knowledge because of your advice. Thank you very much

  1

  

  Travolgi 🍕•31,400

  @denieldenPosted about 2 years ago

  @wshena you are welcome and keep it up :)

  0