[5.1] Minor cleanup for component JS

[C-Lodder]

Summary of Changes

As Joomla no longer supports IE11, this PR mainly removes the need for converting a NodeList to an Array, as evergreen browser's implementation of forEach can iterater over a NodeList.

Also includes some other minor tweaks.

Happy to work on the remaining JS

Testing Instructions

Code review @dgrammatiko @Fedik

[dgrammatiko]

Left some comments about the redundant curly braces.

Great stuff Charlie.

Are you up to also remove the redundant DOMContentLoaded events on the files (almost all) with either defer attribute or type=module?

[C-Lodder]

@dgrammatiko

Are you up to also remove the redundant DOMContentLoaded events on the files (almost all) with either defer attribute or type=module?

I'll do that in a separate PR, as it will probably involve more than just a code review.

[dgrammatiko]

@C-Lodder can you fix the linter parts? Then I'll give it a test

[dgrammatiko]

I have tested this item ✅ successfully on 29f887d

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42778.}

[dgrammatiko]

@Fedik could you review this and give it a successful test?

[Fedik]

I have tested this item ✅ successfully on 29f887d

Looks good on review.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42778.}

[Fedik]

rtc

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42778.}

[LadySolveig]

Thank you @C-Lodder and also for review and testing @dgrammatiko @Fedik