princesinghrajput-Fixed-Animate all toggle button is receiving to many event listeners

princesinghrajput commented 7 months ago

What does this PR do?

This PR addresses the issue #41 by optimizing event listener management for the "Animate all" toggle button. It ensures that event listeners are added only once and removed when no longer needed, preventing accumulation and improving performance.

Why are we doing this?

Currently, the "Animate all" toggle button accumulates event listeners, leading to performance issues. By optimizing event listener management, we can enhance the user experience and maintain code efficiency.

Test Case(s):

[x] Tested the behavior of the "Animate all" toggle button locally to ensure it functions correctly.
[x] Verified that event listeners are added and removed appropriately for the button.
[x] Confirmed that performance improvements are achieved, especially in scenarios with a large number of elements.

Test Result(s):

All test cases passed successfully, demonstrating that the "Animate all" toggle button now functions correctly with optimized event listener management. Performance improvements were noticeable, particularly when handling a large number of elements.

Checklist

[x] Tested locally
[ ] Added new dependencies (if applicable)
[ ] Updated the documentation (if applicable)
[ ] Added a test (if applicable)

vercel[bot] commented 7 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
tailwind-animations	❌ Failed (Inspect)			Feb 11, 2024 8:45am

midudev commented 7 months ago

Thanks for the PR! I think somebody already fixed that in a previous PR!

midudev / tailwind-animations

princesinghrajput-Fixed-Animate all toggle button is receiving to many event listeners #45