-
Notifications
You must be signed in to change notification settings - Fork 11.9k
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
Cannot read properties of null (reading 'addEventListener') #11677
Labels
Comments
@chartjs sorry for the ping but I want to resolve this. I have asked about it about a month ago, I have provided a solution, and would like your input so I can contribute my fix. |
@etimberg / @LeeLenaleee seeing that you are recent contributors, I'm pinging you both about this issue (as per the recommendation from the support discord). |
@DAcodedBEAT the proposed changes make sense to me if you'd like to send a PR |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Expected behavior
addEventListener
andremoveEventListener
should only be callable on valid defined and non-null elements.Current behavior
From sentry:
Reproducible sample
.
Optional extra steps/info to reproduce
No response
Possible solution
Here is what I am proposing: DAcodedBEAT@8930d19
Context
I've seen a few related errors in other implementations according their Github issues
I made a new Github issue since Cannot read properties of null (reading 'addEventListener') #11295 (comment) has been ignored for 3 weeks
I didn't make the Pull Request since https://github.com/chartjs/Chart.js/blob/master/docs/developers/contributing.md advised against it but I made this issue since I was redirected from the discord chat to here (after the first inquiry was ignored)
chart.js version
4.4.1
Browser name and version
Chrome >=120 and Edge >= 120 according to my project's Sentry, but probably many others
Link to your project
No response
The text was updated successfully, but these errors were encountered: