[RFC] Get rid of JavaScript in HTML code

[Arty]

Ticket: http://tracker.phpbb.com/browse/PHPBB3-11795
PR: https://github.com/phpbb/phpbb3/pull/1666

Found several bugs while working in this issue

[hanakin]

if we are finally going to do this can we possibly consolidate all the js files into a sub folder to clean things up a little templates/js/... or in the root of the prosilver folder even better, might be its own rfc after all is said and done, but worth mentioning here

[Arty]

Definitely. There are too many small js files that could be consolidated in one big file and triggered via data attributes on document load.

[JoshyPHP]

I should correct myself. When I said "Twig can escape new lines in attributes" what I really meant was "Twig can escape attributes". Does Twig escape new lines? I've checked with the W3 Validator and it seems that new lines are valid in attribute values. As long as quotes get escaped, you're in the clear.

[NORDLANDVOLK]

data-attributes is valid in HTML5 as a new furure.
So with other words Twig dont see the new data-attributes?
I think is a good idea to have js file in one big file just like a library.

Definitely. There are too many small js files that could be consolidated in one big file and triggered via data attributes on document load.

+1.