Notifications page using React framework

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted about 2 years ago

  Hello Katja, You have done a good work! 😁

  Some little tips to improve your code:

  • add main tag and wrap the card for improve the Accessibility
  • use p, strong, h1, h2, h3 tags instead of a simple div for the texts inside notification
  • remove all unnecessary code, the less you write the better as well as being clearer: for example the picture container of image... picture tag is good to change image by resolution -> read here

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

  Hope this help 😉 and Happy coding!

  Marked as helpful

  1

  ---

  

  Katja Danilova•340

  @katjadanilovaPosted almost 2 years ago

  @denielden Thanks, Deniel, for your comments. I finally had time to get back to this small project, and implemented some of the changes you recommended, and improved accessibility - indeed, a very important step!

  1

  

  Travolgi 🍕•31,400

  @denieldenPosted almost 2 years ago

  @katjadanilova You are welcome and keep it up :)

  0