[JENKINS-73946] Remove inline event handlers in History/index.jelly

[yaroslavafenkin]

https://issues.jenkins.io/browse/JENKINS-73946

Testing done

Going to the history page and changing the values in the dropdowns is enough to trigger the affected code.

https://www.loom.com/share/37c4c276c3a14d2bab38168eb40f7af1?sid=310bc533-b971-4969-9a53-2098f7b3cffc

https://www.loom.com/share/170fb279203a4b38add532a1584cc87e?sid=45a8a87a-e919-4357-a836-0d87bf5dc1d9

Submitter checklist

• Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
• Ensure that the pull request title represents the desired changelog entry
• Please describe what you did
• Link to relevant issues in GitHub or Jira
• Link to relevant pull requests, esp. upstream and downstream changes
• Ensure you have provided tests - that demonstrates feature works or fixes the issue

[basil]

The application/json script tag was only acting as a data holder for the script and was not even triggering a violation

Why not keep it as-is, then, to minimize code change (i.e., risk) and the size of the diff (i.e. for reviewability)?

[yaroslavafenkin]

The application/json script tag was only acting as a data holder for the script and was not even triggering a violation

Why not keep it as-is, then, to minimize code change (i.e., risk) and the size of the diff (i.e. for reviewability)?

I thought it wasn't even a valid thing to do, but I've looked it up and there seems to be nothing wrong with it. I'll revert that part.

[basil]

Thanks! Nice and simple.