fac-13 / jikt-todo

To do list by Jenath, Ivana, Katia, Tom
https://fac-13.github.io/jikt-todo/
2 stars 1 forks source link

Easier way to handle markasdone button #79

Closed shannonjensen closed 6 years ago

shannonjensen commented 6 years ago

You currently have a workaround for your mark as done button where an event called multiple times because the node click event is being called as well. Why not just get rid of mark as done button click handler completely and let the node click event handle it instead of doing your workaround?

tdoran commented 6 years ago

Haha, that makes much more sense! We also found another way using event.stopPropagation() inside the button event listener's function - but your suggestions is more intuitive. 🙏

tdoran commented 6 years ago

Fixed in #80