- It mixes program code (JavaScript) with presentation code (HTML/template language) in one file. - It's impossible to reuse the code elsewhere without copying it. - Browsers may choose to cache an external JavaScript file. They don't have that option with embedded JavaScript.
Fixed now.
Looks good.