Если пользователь нажимает снова, функция просто вызывается снова.Итак, после первого щелчка больше нет изменений.
Лучшим способом было бы назначить стили темного режима классу, скажем «btn - dark-mode», а затем использовать js для переключения этого класса:
function dark_mode() {
document.querySelector('#dark-mode-toggle').classList.toggle('dark-mode'); // ie9+ only
}
.btn--dark-mode {
background-color: #2C2F33;
}
<button id="dark-mode-toggle" class="btn btn-primary" onclick="dark_mode()">Dark Mode ON</button>
Это будет применять класс или снимать его в зависимости от того, существует ли он уже.