if (!(max <= valueInp + step)) А почему бы не писать это в виде max > valueInp + step? Я бы предложил добавить строку debugger внутрь обработчика клика и посмотреть на значения всех переменных if (!(min >= valueInp - step)) { numberInp.value = (valueInp - step); } else { numberInp.value = min }; А эту штуку можно записать как numberInp.value = Math.max(min, valueInp - step)

Представляю вашему вниманию фреймворк управления контентом Revolver.     Работает на PHP, MySQL и ECMAScript 7. Унифицированный интерфейс на View Port единицах и специальный шаблонизатор позволяет быстро и эффективно создавать вебсайты любой сложности под высокие нагрузки с низким требованием к ресурсам хостинга(достаточно компьютера типа Raspbery и 10Mb выделенной оперативной памяти). Трехуровневое кэширование позволяет свободно развивать ваши сайты и сервисы не беспокоясь о высокой стоимости обслуживания сервера. Архитектура Kernel->Model->View прекрасно спланирована и дает перспективы развития проекта на долгие годы.  Revolver CMF великолепно защищен от любых атак и имеет встроенную капчу, которую не смог обойти ни кто. Страница проекта здесь: RevolveR CMF.

Ну да: перепутал. А что толку? Вот мы его и выяснили.   Ну да: перепутал. А что толку? Вот мы его и выяснили.   Ну да: перепутал. А что толку? Вот мы его и выяснили.