Responsive & Interactive Notification Page using CSS grid and Flexbox

Community feedback

• 

  Marjan•320

  @MarjanZivkovic

  Posted about 2 years ago

  Hey KRISHNA! Great job on the design! I find it a little strange to completely remove the badge and the button from the DOM once you mark all messages as unread. If that was your goal, it's, fine. If not, there's an easy fix:

                   //....
                   let notificationCount = unreadNotifications.length - 1;
                   const onNotificationClick = (notification) => {
                             //...
                             if(notificationCount === 0){
                                    return;     
                              }
                              --notificationCount;
                   }
  //...
  }  ```                            
  
  You just stop your function once the counter hits 0. Also, you have to move decreasing your counter below the if check so it stops decreasing if it's less than 0.
  Keep going!
  

  01

  Marked as helpful