Как заставить JavaScript ‘getElementById’ работать с переключателем HTML CSS

#javascript #html #css

#javascript #HTML #css

Вопрос:

Мне нужно активировать мой HTML / CSS ‘Toggle Switch’ с помощью JavaScript.

Я бы хотел, чтобы DIV с текстом был скрыт по умолчанию, и когда ползунок (переключатель) перемещается влево, он «запускает» DIV для «отображения» с использованием JavaScript.

Я понимаю, что я на правильном пути, но с моими действиями что-то не совсем так…

 function toggleDiv() {
  var triggeredDiv = document.querySelector('.triggeredDiv');
  if (document.getElementById('flipswitch').checked) {
    triggeredDiv.classList.remove('shown');
  } else {
    triggeredDiv.classList.add('shown');
  }
}

document.getElementById('flipswitch').addEventListener("change", toggleDiv);  
 .flipswitch {
  position: relative;
  width: 200px;
  -webkit-user-select: none;
  -moz-user-select: none;
  -ms-user-select: none;
}

.flipswitch input[type=checkbox] {
  display: none;
}

.flipswitch-label {
  display: block;
  overflow: hidden;
  cursor: pointer;
  border: 2px solid #999999;
  border-radius: 50px;
}

.flipswitch-inner {
  width: 200%;
  margin-left: -100%;
  -webkit-transition: margin 0.3s ease-in 0s;
  -moz-transition: margin 0.3s ease-in 0s;
  -ms-transition: margin 0.3s ease-in 0s;
  -o-transition: margin 0.3s ease-in 0s;
  transition: margin 0.3s ease-in 0s;
}

.flipswitch-inner:before,
.flipswitch-inner:after {
  float: left;
  width: 50%;
  height: 60px;
  padding: 0;
  line-height: 60px;
  font-size: 18px;
  color: white;
  font-family: Trebuchet, Arial, sans-serif;
  font-weight: bold;
  -moz-box-sizing: border-box;
  -webkit-box-sizing: border-box;
  box-sizing: border-box;
}

.flipswitch-inner:before {
  content: "MONTHLY";
  padding-left: 12px;
  background-color: #FFFFFF;
  color: #888888;
  font-family: 'Montserrat', sans-serif;
  font-weight: 400;
}

.flipswitch-inner:after {
  content: "BY COUNTRY";
  padding-right: 12px;
  background-color: #EBEBEB;
  color: #888888;
  text-align: right;
  font-family: 'Montserrat', sans-serif;
  font-weight: 400;
}

.flipswitch-switch {
  width: 45px;
  margin: 7.5px;
  background: #FFFFFF;
  border: 2px solid #999999;
  border-radius: 50px;
  position: absolute;
  top: 0;
  bottom: 0;
  right: 139px;
  -webkit-transition: all 0.3s ease-in 0s;
  -moz-transition: all 0.3s ease-in 0s;
  -ms-transition: all 0.3s ease-in 0s;
  -o-transition: all 0.3s ease-in 0s;
  transition: all 0.3s ease-in 0s;
}

.flipswitch-cb:checked .flipswitch-label .flipswitch-inner {
  margin-left: 0;
}

.flipswitch-cb:checked .flipswitch-label .flipswitch-switch {
  right: 0;
}

.triggeredDiv {
  display: none;
}

.triggeredDiv.shown {
  display: block;
}  
 <div class="flipswitch">
  <input type="checkbox" name="flipswitch" class="flipswitch-cb" id="fs" checked>
  <label class="flipswitch-label" for="fs">
    <div class="flipswitch-inner"></div>
    <div class="flipswitch-switch"></div>
  </label>
</div>

<div class="triggeredDiv">
  Show Text
</div>  

Комментарии:

1. Идентификатор переключателя — «fs», а не «flipswitch»

2. К вашему сведению, ваше if утверждение можно исключить, используя toggle с аргументом switch. triggeredDiv.classList.toggle('shown', !document.getElementById('flipswitch').checked); Когда параметр переключателя равен true , класс будет добавлен, в противном случае будет удален.

Ответ №1:

Проблема здесь в том, что вы пытаетесь ссылаться на переключатель с помощью идентификатора flipswitch, в то время как вы присвоили переключателю идентификатор fs. Ссылки в javascript просто должны быть изменены, чтобы быть:

 document.getElementById('fs')
  

вместо

 document.getElementById('flipswitch')
  

Ответ №2:

Вместо использования document.getElementById вы должны использовать что-то для получения элементов с помощью class , поскольку это то, что определено в вашей разметке.

Кроме того, ваш флажок ввода не переключается checked , он просто переключает состояние triggered div.

Мы можем заставить это работать, скорректировав код:

 function toggleDiv() {
   this.element || ( this.element = document.querySelector('.triggeredDiv') );

   this.element.classList.toggle("shown");
  }
  

С комментариями для лучшего понимания:

 function toggleDiv() {
  /*
   if we don't have a reference in `toggleDiv.element` 
   we use `document.querySelector` to retrieve
   and save the reference to the element `.triggeredDiv`
   this ensures that we only go through the DOM once to retrieve the element
   no matter how many times the function is called
   which is more performant. 
  */
  this.element || (this.element = document.querySelector('.triggeredDiv'));

  /*
   after we have the element, we simply toggle the `shown` class
   using the `classList.toggle` method.
  */
   this.element.classList.toggle("shown");
}

  document.querySelector('.flipswitch').addEventListener("change", toggleDiv);
  

 function toggleDiv() {
   this.element || ( this.element = document.querySelector('.triggeredDiv') );
   
   this.element.classList.toggle("shown");
  }
  
  document.querySelector('.flipswitch').addEventListener("change", toggleDiv);  
 .flipswitch {
  position: relative;
  width: 200px;
  -webkit-user-select: none;
  -moz-user-select: none;
  -ms-user-select: none;
}

.flipswitch input[type=checkbox] {
  display: none;
}

.flipswitch-label {
  display: block;
  overflow: hidden;
  cursor: pointer;
  border: 2px solid #999999;
  border-radius: 50px;
}

.flipswitch-inner {
  width: 200%;
  margin-left: -100%;
  -webkit-transition: margin 0.3s ease-in 0s;
  -moz-transition: margin 0.3s ease-in 0s;
  -ms-transition: margin 0.3s ease-in 0s;
  -o-transition: margin 0.3s ease-in 0s;
  transition: margin 0.3s ease-in 0s;
}

.flipswitch-inner:before,
.flipswitch-inner:after {
  float: left;
  width: 50%;
  height: 60px;
  padding: 0;
  line-height: 60px;
  font-size: 18px;
  color: white;
  font-family: Trebuchet, Arial, sans-serif;
  font-weight: bold;
  -moz-box-sizing: border-box;
  -webkit-box-sizing: border-box;
  box-sizing: border-box;
}

.flipswitch-inner:before {
  content: "MONTHLY";
  padding-left: 12px;
  background-color: #FFFFFF;
  color: #888888;
  font-family: 'Montserrat', sans-serif;
  font-weight: 400;
}

.flipswitch-inner:after {
  content: "BY COUNTRY";
  padding-right: 12px;
  background-color: #EBEBEB;
  color: #888888;
  text-align: right;
  font-family: 'Montserrat', sans-serif;
  font-weight: 400;
}

.flipswitch-switch {
  width: 45px;
  margin: 7.5px;
  background: #FFFFFF;
  border: 2px solid #999999;
  border-radius: 50px;
  position: absolute;
  top: 0;
  bottom: 0;
  right: 139px;
  -webkit-transition: all 0.3s ease-in 0s;
  -moz-transition: all 0.3s ease-in 0s;
  -ms-transition: all 0.3s ease-in 0s;
  -o-transition: all 0.3s ease-in 0s;
  transition: all 0.3s ease-in 0s;
}

.flipswitch-cb:checked .flipswitch-label .flipswitch-inner {
  margin-left: 0;
}

.flipswitch-cb:checked .flipswitch-label .flipswitch-switch {
  right: 0;
}

.triggeredDiv {
  display: none;
}

.triggeredDiv.shown {
  display: block;
}  
 <div class="flipswitch">
  <input type="checkbox" name="flipswitch" class="flipswitch-cb" id="fs" checked>
  <label class="flipswitch-label" for="fs">
    <div class="flipswitch-inner"></div>
    <div class="flipswitch-switch"></div>
  </label>
</div>

<div class="triggeredDiv">
  Show Text
</div>  

Ответ №3:

Я думаю, что если вы просто измените два экземпляра document.getElementById('flipswitch') на document.getElementById('fs') в своей функции, ваш код будет работать нормально.

Пример JSFiddle:https://jsfiddle.net/oq8wL2v4