Design comparison
SolutionDesign
Solution retrospective
Any feedback is always appreciated. Thanks for taking a look
Community feedback
- @DavidMorgadePosted over 2 years ago
Hello and congrats on finishing this challenge! here is my approach that can maybe help you for future projects:
- Try to not add the event listener in to the html, instead of having
onclick
on your html, use your Javascript to select the button and attach de event there, adding it to the html can result in some buggy behaviour.
For the rest the project seems to be responsive and fully working! good job!
Hope my feedback helps you!
1@MURRAY122Posted over 2 years agoThanks for your feedback and suggestion, I've since updated my JS based on your suggestion. Thanks again
0 - Try to not add the event listener in to the html, instead of having
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