Co tak na pierwszy rzut oka zauważyłem:
1. Skrypty JS można dołączyć na końcu body
2. onclick vs https://developer.mozilla.org/pl/docs/Web/API/Element/addEventListener
3. type="text/css" nie trzeba dodawać w
<link rel="stylesheet" type="text/css" href="style.css" />
4. W JS warto sobie deklarować na początku skryptu odwołania do elementów HTML, a nie robić to za każdym razem od nowa.
5. Co do skryptu to jeszcze: https://scotch.io/tutorials/understanding-scope-in-javascript
6. Z tego co pamiętam to textContent jest szybsze niż innerHTML