Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: refactor 'click' event listeners #540

Closed
wants to merge 7 commits into from

Conversation

Dhoni77
Copy link
Contributor

@Dhoni77 Dhoni77 commented Nov 2, 2023

Fixes Issue

My PR closes #539

πŸ‘¨β€πŸ’» Changes proposed(What did you do ?)

  • Refactor the entire 'click' event listeners and use event delegation to improve performance.
  • Added a global store to access state in other files.

βœ”οΈ Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • This PR does not contain plagiarized content.
  • The title and description of the PR is clear and explains the approach.

Note to reviewers

πŸ“· Screenshots

Copy link

vercel bot commented Nov 2, 2023

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
code-magic ❌ Failed (Inspect) Nov 2, 2023 3:28pm

@Dun-sin
Copy link
Owner

Dun-sin commented Nov 2, 2023

Your pull request is rejected, reason πŸ‘‰πŸΎis hereπŸ‘ˆπŸΎ

Thanks for wanting to contribute, I hope you continue to do so properly.

@Dun-sin Dun-sin closed this Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[REFACTOR] Too may event handlers
2 participants