Почему не работает код?

Question

[CompiledBrain]

<button  id="hideFrame" data-val="true">Hide frame</button>

bFrame=doc.getElementById("hideFrame");

  bFrame.onclick=function () {
  var v=this.dataset.val
  ,d=v?"none":"block",
  frames=document.getElementsByTagName("iframe");
  console.log(d,v);
  for (var i = 0,l=frames.length; i < l; i++) {
    frames[i].style.display=d;
  }
  this.dataset.val=!v;
  }

Закрывать то он закрывает а открывать нет
помогите

Comments

[CODE GEEK]

поменяй на

<button  id="hideFrame" data-val="true">Hide frame</button>

[CompiledBrain]

Леонид Резвицкий: да там была ошибка я исправил
новий код:

<button  id="hideFrame" data-val="true">Hide frame</button>

но и дальше

Закрывать то он закрывает а открывать нет

Answer 1

[twobomb]

JS

document.querySelector("#hideFrame").addEventListener("click",function(){
	document.querySelectorAll("iframe").forEach(function(el){
  	el.style.display =  window.getComputedStyle(el).display == "block"?"none":"block"; 
  });
},false);

CSS

iframe{
      display:block;
    }

Comments

[CompiledBrain]

спасибо
также хочется знать по какой причине мой код не работает чтоб я больше не делал таких ошибок

[twobomb]

CompiledBrain:
doc - Неизвестная переменная
А не работает у вас потому-что значения которые вы получаете из атрибута data-val являются текстовыми, а не булевыми. Поэтому вы сравнивает не true == true, а true == "true"... Как вариант сделать так.

<button  id="hideFrame" data-val="0">Hide frame</button>
<iframe src="google.ru" frameborder="0"></iframe>

bFrame=document.getElementById("hideFrame");

  bFrame.onclick=function () {
  var v=this.dataset.val,
  d=v == "0"?"none":"block",
  frames=document.getElementsByTagName("iframe");
  for (var i = 0,l=frames.length; i < l; i++) {
    frames[i].style.display=d;
  }
  this.dataset.val= 1-v;
  }