Pergunta

Tentando adicionar um manipulador onclick aos meus guias, e não consigo obter o direito seleção DOM. Vocês podem ajudar?

  <div id="tabstrip">
    <ul>
      <li id="a" class="selected"><a href="#">A</a></li>
      <li id="b"><a href="#">B</a></li>
      <li id="b"><a href="#">C</a></li>
    </ul>
  </div>

function initTabStrip()
{
  var lis = document.getElementById('tabstrip').getElementsByTagName('li');
  for (var i=0;i<items.length;i++)
  {
    var as = items[i].getElementsByTagName('a');
    for (var j=0;j<as.length;j++)
    {
      as[j].onclick=function(){changeTab(items[i].id);return false}
    }
  }
}
Foi útil?

Solução

Parece que o seu fechamento é errado. Tentar

as[j].onclick = function(items, i)
{
    return function()
    {
        changeTab(items[i].id);
        return false;
    };
}(items, i);

Se ele funciona, então a questão é um joguete de jQuery Closures, Loops e Eventos

Outras dicas

Eu concordo em RoBorg e sugerir a leitura sobre JavaScript escopos. Ele irá explicar muito.

A primeira linha do seu JS, você cria var lis mas então você iterate em items. É isso mesmo que você quer?

função completa, com as mudanças sugeridas por Røberg:

function initTabStrip()
{
  var tabstrip = document.getElementById('tabstrip');
  var items = tabstrip.getElementsByTagName('li');
  for (var i=0;i<items.length;i++)
  {
    var as = items[i].getElementsByTagName('a');
    for (var j=0;j<as.length;j++)
    {
      as[j].onclick = function(items, i)
      {
        return function()
        {
          changeTab(items[i].id);
          return false;
        };
      }(items, i);
    }
  }

}

Licenciado em: CC-BY-SA com atribuição
Não afiliado a StackOverflow
scroll top