Design comparison
SolutionDesign
Community feedback
- @KorneyChervonenkoPosted 6 months ago
Hello friend, 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 in your case "ul" and after click event just find out which notification was clicked exactly
- unread notifications counter doesn't work
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