notification

Community feedback

• 

  Eddy•80

  @eddy-tPosted almost 2 years ago

  Good job, however it was not useful to bootstrap this project, it is easier and better to use Flexbox and CSS Grid to really learn how to use it and not need to bootstrap all the time.

  Moreover I notice some little mistakes:

  • The badge with the number of notifications doesn't change to 0 when you click on the "Mark all as read" link (plus there is a hover status on the badge that shouldn't be there)

  • You forgot to add for each unread notification the little red dot on the right

  • When you click on the link "Mark all as read" the background of the unread notifications changes correctly but the number of notifications on the badge stays at 3 (it should normally be 0)

  Check out my version of this challenge (coded entirely in HTML/CSS without bootstrap but with Flexbox and CSS Grid and with a tiny bit of JS for all the state changes when clicking on the "Mark all as read" link

  0