alampros / react-confetti

Confetti without the cleanup.
http://alampros.github.io/react-confetti/
MIT License
1.51k stars 99 forks source link

Confetti now become the target of pointer events #63

Closed yanlin-duan closed 5 years ago

yanlin-duan commented 5 years ago

In react-confetti 2.4.1 we apply pointer-events: none to the confetti (https://github.com/alampros/react-confetti/blob/v2.4.1/src/react-confetti.js#L105) but we don't do it anymore since 2.5.0 (https://github.com/alampros/react-confetti/blob/v2.5.0/src/ReactConfetti.tsx#L32). This has caused a production issue where things beneath the confetti is not clickable.

Is this an intentional change? Do we have a lot of use cases where we want the confetti to be the target of a pointer event?

alampros commented 5 years ago

Oh no! Sorry to hear this caused you some troubles. It was not an intentional change – probably an accidental commit when I was working on the mouse rain demo.

alampros commented 5 years ago

:tada: This issue has been resolved in version 4.0.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: