Remove no needed NodeList item counting and for loop after querySelectorAll()

With my javascript eyes on this seems like a good conversion, leaning on tests that active-link.js breakage would show.

Are there not other places where this fix can be applied? E.g. announcements.js and states.js in the misc folder?

Are there not other places where this fix can be applied? E.g. announcements.js and states.js in the misc folder? There appear to be several instances in those files where the same sub-optimal syntax is used. Should they be included in this issue or in a follow-up?

nod_ → closed merge request !12457

Let's do follow-ups, I'd rather get that in regularly than wait 6 months for a big MR.

Committed 93279d3 and pushed to 11.x. Thanks!