Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

FEM-notifications-page using HTML, CSS & JS

danemaas 520

@danemaas

Desktop design screenshot for the Notifications page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


I added a little feature to this challenge: to mark the specific notification as read upon clicking the notification. And the unread counter will also update upon marking the notification as unread. I did this to just challenge my self in my JS and I'm happy with the output. Any suggestion on how to make my JS code more efficient is very much welcome. Thank you!

Community feedback

@kabir-afk

Posted

heyy, good job on completing the challenge👍, on reviewing your code ,I found redeclaration of some variables to be really uncalled for . . . I'm guessing you tried declaring them globally but weren't able to access them whenever and wherever needed due to this keyword . . .not saying that your code is wrong in any way but it really looks repetitive

I faced a similar issue recently and I've come to conclusion that either you can expand your knowledge on this keyword , or use e.target to access whatever is being clicked or interacted with . . ., hope this helps . . . happy coding!!!

Marked as helpful

0

danemaas 520

@danemaas

Posted

@kabir-afk hello. I will review my code and revise it with this idea in mind. thank you very much for the feedback. cheers!! 😁👍

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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