Design comparison
SolutionDesign
Solution retrospective
I added button for marking as unread. In js with event delegation in order to exercise.
Community feedback
- @CatalinBaniciPosted almost 2 years ago
That's really nice actually, after seeing this I might consider reworking on mine.
But don't forget to add the hover effects, I can only see one on mark as read/unread buttons, and there should be more.
Also try to add a function to remove the classlist for each notification, not only from the Mark All As Read button, as realistically , when you click on a individual notification, it should go on a read state.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord