Daniel
@DaijekAll comments
- @danielpellegriniSubmitted about 2 years ago@DaijekPosted about 2 years ago
Okay so I think you did a good overall but for the changing the seen status, I don't think that it should be toggled. I believe that if you've seen a notification, you shouldn't be able to unsee it. Cheers
1 - @AJonastechSubmitted about 2 years ago
To implement the notification system I had to collect the div items in a node list object using .getElementByclassName. Then I added the click event listener to each of the items in the node list object, but had difficulty accessing the specific div item that was clicked. So I assigned unique ids to each of the div items, used the this.id method to determine which item was clicked and it eventually worked. Any idea on other ways I could have done this?
@DaijekPosted about 2 years agoI find this approach very intriguing because it is really different from mine (and it still works), although some of that is due to the fact that my solution is with react and yours is with vanilla javascript. I think you did a great job on this, keep up the great work. Cheers
1