Как сделать два события на одну кнопку?

Question

[MarkSexton]

Всем привет! Есть проблема....

По нажатию на кнопку "Отключить скролл" мы выключаем скролл на странице, а по нажатию на "Включить скролл" мы возвращаем скролл странице, все конечно логично, НО, как объединить данный функционал в одну кнопку/ссылку, а не в 2?

Прикладываю исходник

Спасибо за внимание

Comments

[Андрей]

Про оператор if слышали?

Answer 1

[Никита Егоров]

Зачем оборачивать код в функцию, а потом вызывать, если там обработчики события? Это не обязательно.
Меняете класс active, через toggle и смотрите, есть ли он.

Comments

[MarkSexton]

Огромное спасибо за наглядный пример!

[MarkSexton]

Еще тогда вопрос, если мы не вызываем функцию, то как данный код вставить в common JS? как оборачивается данный код?

[Никита Егоров]

MarkSexton, а, если так, то да, придется обернуть. Но в iife.Что бы потом не вызывать.

// scroll.js
module.exports = (function() { 'здесь код' })();

// app.js
(function() { require('./scroll.js') })()

[MarkSexton]

Никита Егоров, Еще раз спасибо!

Answer 2

[Владимир Проскурин]

Делаете переменную isScrollOn указывающую на текущее состояние скролла. При клике на одну кнопку, проверяете
если isScrollOn то выполняем код выключения скролла и присваиваем isScrollOn = false
иначе выполняем код включения скролла и присваиваем isScrollOn = true
Ну, или гуглите реализацию jQuery toggle click

Answer 3

[frees2]

for onclick class button label - на всё можем повесить событие, без JQUERY
Конечно будут проблемы в телефонах...

<label for="modal"   onclick="onclickVhod()" class="button">YouTube </label>

демо ( для тестирования) dulsky.eu/t/Lq6q3YJqLaQ