I agree with everything but inline javascript

<button onclick="this.classList.add('bg-green')"> Click Me </button>

This is going to be problematic when it comes to debugging. I still think scripts should be in a file and loaded in the header.