I have completed this course.
When I type in a list of numbers and press Calculate it works and gives me all the correct information. However the console shows me:
TypeError: action is null
Why is this?
Learn Advanced Array Methods by Building a Statistics Calculator - Step 50
You`ve hit an error somewhere on the backend of this page, Inside the inspector will show the list.
We are listening for a submit event on the document (for the tests I assume) and the form is triggering that event. If you add event.stopPropagation() to the calculate function it should fix it.
I opened an issue for it.
Thank you. I also discovered that if I add action="" as an attribute for the form, that also stops it. Is there a potential problem if I use action=""?
This is what I got on my editor screen (using Firefox btw): when I typed in your solution (although it stops the TypeError: action is null):
Note that event is crossed out - don’t know what that means
Adding the empty action attribute is probably a better solution, at least for this issue with the challenge. It shouldn’t cause any issues. It might be less confusing for the camper and we can add that to the HTML silently without it being too obvious and confusing.
It is because the event is the global event object and that is discouraged usage (it is a deprecation warning). It should work just fine, but passing the event as an argument and adding the parameter to the function would be better. I just showed that because it was easier.
As an aside. Deprecation warnings in CSS/HTML/JS are not really deprecation warnings seeing as nothing is ever removed because of backward compatibility. If it is added to the browser it is (almost) never removed again. Talk about high pressure when adding new functionality. You get it right or you get it wrong…forever.
Thank you for your reply - I’ve made a note to myself about using action="" in future
To be valid HTML the action attribute shouldn’t use an empty value.
It isn’t a huge deal, but if you want to prevent a form submission (and subsequent page refresh) you should use preventDefault. To prevent the event from propagating you can use stopPropagation.
This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.
