2

Написал вот такой код. Не работает. Подскажите что исправить.

var video = document.getElementById("video");
var volume = document.getElementById("volume");

volume.onclick = function() {
  if (video.hasAttribute('muted')) {
    video.removeAttribute('muted');
  } else {
    video.addAttribute('muted');
  }
};
<video id="video" autoplay loop muted>
  <source src="img/video.mp4" type="video/webm" />
</video>
<button id="volume">Звук</button>

2
  • можно увидеть html?
    – Areshka
    6 мар 2018 в 3:34
  • @AuRoom Добавил 6 мар 2018 в 3:45

2 ответа 2

4

var video = document.getElementById("video");
var volume = document.getElementById("volume");

volume.addEventListener('click', function() {
    video.muted = (video.muted == false) ? true : false;
});

2
  • Спасибо! Буду применять в следующих проектах. 6 мар 2018 в 5:36
  • Всегда пожалуйста...)
    – Air
    6 мар 2018 в 5:38
0

Атрибут muted является логическим атрибутом. Достаточно сделать так:

volume.addEventListener('click', function () {
    video.muted = !video.muted;         
})
6
  • Повторный клик почему-то не работает. 6 мар 2018 в 5:04
  • volume.click = function () { if (video.muted = true) { video.muted = false; } else { video.muted = true; } } 6 мар 2018 в 5:06
  • Поправил ответ, попробуйте снова 6 мар 2018 в 5:09
  • скопировал ваш код, но выключение звука не работает, тоесть то что по else идёт. 6 мар 2018 в 5:20
  • а не проще сделать в if конструкцию video.muted = !video.muted, тогда ветка else не понадобится 6 мар 2018 в 5:26

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.