Вы пишите onclick="add1". Это не работает, потому что при нажатии на кнопку вычисляется выражение в кавычках. А результатом выражения "add1" является функция. Обратите внимание, не
вызов функции, а просто ссылка на функцию.
Следует писать так: onclick="add1()"
А вообще хочу заметить, что такой способ создания обработчика слегка морально устарел. По сути он приравнивается к использованию функции eval, а это скверно, не стильно и не модно.
Более правильный вариант убрать атрибут onclick, а в скрипте добавить обработчик примерно так:
var button = document.querySelector("button"); //как-то получаем указатель
button.addEventListener("click", add1); //добавляем сам обработчик