alexgibson/tap.js

Maybe document .destroy()

Closed this issue · 3 comments

Having looked at the code, it seems to me that I should be calling myTap.destroy() when I no longer need the tap event ... not just el.removeEventListener('tap'). Do you agree?

If so, I think it would be worth documenting this function and its purpose.

Pull requests are welcome 👍

Cool, just wanted to make sure I wasn't missing the point. PR submitted.

Thanks again!