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.

Sign up for GitHub

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

Jump to bottom

Memory leak due to event listener #1142

Closed

4 tasks done

Palakkgoyal opened this issue Aug 5, 2023 · 3 comments · Fixed by #1180

Closed

4 tasks done

Memory leak due to event listener #1142

Palakkgoyal opened this issue Aug 5, 2023 · 3 comments · Fixed by #1180

Labels

bug

Contributor

Palakkgoyal commented Aug 5, 2023 •

edited

Loading

Description

There is a event listener in the App.jsx file that is running and not getting cleared. It could cause memory leaking.

Steps to reproduce

Use the website for some time
In the DevTools => elements => event listeners => scroll
You will see there are so many scroll event listeners are triggered.

Screenshots

No response

Desktop (please complete the following information)

No response

Smartphone (please complete the following information)

No response

Additional information

No response

Record

I agree to follow this project's Code of Conduct
I have checked the existing issues
I'm a GSSoC'23 contributor
I want to work on this issue

Palakkgoyal added the bug label

github-actions bot commented Aug 5, 2023

Hello Palakkgoyal!
Thank you for raising this issue! 😊 Your contribution is valuable to us! 😊

Please make sure to follow our Contributing Guidelines. 💪🏻

Our reviewers shall carefully assess the issue and reach out to you soon! 😇
We appreciate your patience!

Palakkgoyal mentioned this issue

Fix event listener leaking #1180

Merged

narayan954 closed this as completed in #1180

Owner

narayan954 commented Aug 6, 2023

Thanks for fixing/finding!

Contributor Author

Palakkgoyal commented Aug 7, 2023

No worries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment