fix(*): Remove all self closing tags in AngularJS templates

[christopherthielen]

Self closing tags are invalid HTML except for a few specific tags, i.e., <br/>, <input/>, etc. In some cases markup following a self closed tag will be embedded inside the self closed tag. i.e.,

<i class="fa fa-spin"/> 
<h1>hello</h1>

is parsed as

<h1>hello</h1>
</i>

AngularJS code relies on the browser html parser, and so we shouldn't use self-closed tags in AngularJS code.

Reference: angular/angular.js#1953 (comment)

This issue manifest as oddly placed icons when I changed typescript module type from commonjs to esnext in #6445 (and I have no idea why that would alter the behavior)