window.onload = function () {
var firstH1 = document.getElementsByTagName('h1')[0];
firstH1.id = "heading1";
firstH1.onclick = function() {
var currentValue = this.nextSibling.style.display;
this.nextSibling.style.display = (currentValue == "none") ? "block" : "none";
}
}
Обратите внимание на несколько вещей: я упростил выборку вашего элемента, потому что нет смысла извлекать элемент, назначать ему идентификатор, а затем использовать этот идентификатор, чтобы снова найти тот же элемент.
Я также переключил порядок блоков / нет, потому что, если стиль не отображается, он будет пустым - и ваш первый щелчок назначит ему блок - и он не исчезнет. Таким образом, он делает.