1. в pop, возможно, лишняя операция, постдекремент можно сразу отдать
2. вынос относительно простого расчета в переменную, которая потом нигде не используется, да еще и летит на выхлоп - читабельность ради читабельности
3. одного стиля нет - где-то лишний пробел, где-то лишняя строка, а где-то в одних и тех же ситуациях (for) используете разный подход (где присвоение, где инкремент)
4. комментирование скудное - общего описания нет

в общем и целом, я бы оценила на твердое «хорошо»

имхо, ничего личного, меня все-равно поправят, если не права)

Есть вот такой инпут - input(type="range", min="0", max="100" value="0", id="i1-range"), как мне через js изменить значение value?

getElementById('i1-range').value = 25
помоему так

блен

function f1 () {
   date00 = new Date();
   let min = (date00.getMinutes())
   date0 = new Date();
   let hours = 60 * (date0.getHours()) + min
}

f1();

у меня вот такая функция и надо поменять value на hours

эт к сожалению не работает

Ошибку можно увидеть?

Uncaught ReferenceError: getElementById is not defined
   at script.js:10

document.getElement......

document.getElement is not a function

Голосовые запрещены

1. не вижу постдекремента, или о чем ты 🙊
2. он не может удалить ласт элемент и вернуть его  🙊
3. no comments 🙈
4. no comments 🙈

Друзья, я без понятия что мне делать

добавить document.

что тебе делать, верни внутри функции переменную hours
и потом напиши
i1-range.value = f1()

возвращай hours из фукции и все

брось на кодепен

а я и не сомневалась даже, что из всего чата, да еще и после полученного спама в лс, вы обязательно зацепитесь 🤷🏼‍♀

1. имя метода поправила
2. имелось ввиду действо в push
так что, имхо без изменений

поп возвращает удаленный элемент а не измененный массив
он не может в ретерне сразу и сокращать массив и вернуть ласт элемент. не забывай там мутирование