Во-первых, если подключаете скрипт не перед закрытием body
, то поместите его туда.
Во-вторых, document.getElementsByClassName
возвращает HTMLCollection
- массив, который будет автоматически изменять свое содержимое при изменениях на странице. Поэтому, либо берете из него первый элемент, либо используете document.querySelector
.
Также, неплохо было бы проверять элементы, прежде чем добавлять им события; вместо onevent использовать element.addEventListener('event', handler);
; вместо ==
использовать ===
; вместо style.backgroundColor
использовать style.setProperty('background-color', value);
.