Notifications page main

Community feedback

• 

  Korney Chervonenko•960

  @KorneyChervonenkoPosted 6 months ago

  Hello friend,

  1. 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
  2. I think when user clicks on the unread notification remove mark must be removed and counter decreases by one. Am I wrong?

  0