Notification page with HTML CSS & JS

Community feedback

• 

  Korney Chervonenko•990

  @KorneyChervonenkoPosted 6 months ago

  Hello friend, It seems to me that you have added event listener to every unread notification but what if you have hundreds or thousands of those? Your JS would be inefficient. It’s better to add only one event listener to some container of notification elements and after click event just find out which notification was clicked exactly

  Marked as helpful

  0

  ---

  

  Ali Reza•470

  @AliRezaCodingPosted 6 months ago

  @KorneyChervonenko , Thanks for sharing your opinion, you mentioned good point, but since the size of challenge was small and number of notifications was limit I decided to attach an event listener for each of them, but already the way you mentioned ( attaching one event listener to it's container ) is the easiest way and I'll try to change that.

  0