Skip to main content
adicionou 5 caracteres ao conteúdo
Fonte Link
Sam
  • 80,8mil
  • 22
  • 73
  • 127

Ao meu ver não precisa atribuir eventos de click aos links <a>, basta apenas inserir o atributo target="_blank" que já irá abrir em uma nova guia.

Como você não sabe quanto tempo o Ajax irá levar para processar, uma forma é criar um setInterval que ficará rodando até encontrar um link externo definido no seu seletor. Assim que o if no setInterval for atendido, significa que o Ajax foi processado e os links foram inseridos, então basta adicionar o atributo target a todos os links de uma só vez, sem a necessidade de laço .each.

Veja como fica:

$(document).ready(function(){

   var niceJavascriptRoutine = setInterval(function(){

      var as = $("a[href^='http']:not([href*='zaha.in'])");
      
      if(as.length){
         as.attr('target', '_blank')
         .addClass('externalLink');
         clearInterval(niceJavascriptRoutine);
      }
   }, 1000);

});

O setInterval vai ficar rodando a cada segundo, e quando houver algum elemento na página que foi definido na variável as, irá aplicar o target a todos os elementos e parar o setInterval.

Agora, o ideal mesmo era buscar os links dentro de algum contêiner aonde o Ajax os inseriu, caso contrário o seletor a[href^='http'] irá buscar na página toda, e se houver algum link externo que não foi inserido pelo Ajax, o setInterval irá parar logo no início.

Deveria ser algo assim:

var as = $("#div a[href^='http']:not([href*='zaha.in'])");
            ↑↑↑↑
   especificar a div alvo do Ajax

Ao meu ver não precisa atribuir eventos de click aos links <a>, basta apenas inserir o atributo target="_blank" que já irá abrir em uma nova guia.

Como você não sabe quanto tempo o Ajax irá levar para processar, uma forma é criar um setInterval que ficará rodando até encontrar um link externo definido no seu seletor. Assim que o if no setInterval for atendido, significa que o Ajax foi processado e os links foram inseridos, então basta adicionar o atributo target a todos os links de uma só vez, sem a necessidade de laço .each.

Veja como fica:

$(document).ready(function(){

   var niceJavascriptRoutine = setInterval(function(){

      var as = $("a[href^='http']:not([href*='zaha.in'])");
      
      if(as.length){
         as.attr('target', '_blank')
         .addClass('externalLink');
         clearInterval(niceJavascriptRoutine);
      }
   }, 1000);

});

O setInterval vai ficar rodando a segundo, e quando houver algum elemento na página que foi definido na variável as, irá aplicar o target a todos os elementos e parar o setInterval.

Agora, o ideal mesmo era buscar os links dentro de algum contêiner aonde o Ajax os inseriu, caso contrário o seletor a[href^='http'] irá buscar na página toda, e se houver algum link externo que não foi inserido pelo Ajax, o setInterval irá parar logo no início.

Deveria ser algo assim:

var as = $("#div a[href^='http']:not([href*='zaha.in'])");
            ↑↑↑↑
   especificar a div alvo do Ajax

Ao meu ver não precisa atribuir eventos de click aos links <a>, basta apenas inserir o atributo target="_blank" que já irá abrir em uma nova guia.

Como você não sabe quanto tempo o Ajax irá levar para processar, uma forma é criar um setInterval que ficará rodando até encontrar um link externo definido no seu seletor. Assim que o if no setInterval for atendido, significa que o Ajax foi processado e os links foram inseridos, então basta adicionar o atributo target a todos os links de uma só vez, sem a necessidade de laço .each.

Veja como fica:

$(document).ready(function(){

   var niceJavascriptRoutine = setInterval(function(){

      var as = $("a[href^='http']:not([href*='zaha.in'])");
      
      if(as.length){
         as.attr('target', '_blank')
         .addClass('externalLink');
         clearInterval(niceJavascriptRoutine);
      }
   }, 1000);

});

O setInterval vai ficar rodando a cada segundo, e quando houver algum elemento na página que foi definido na variável as, irá aplicar o target a todos os elementos e parar o setInterval.

Agora, o ideal mesmo era buscar os links dentro de algum contêiner aonde o Ajax os inseriu, caso contrário o seletor a[href^='http'] irá buscar na página toda, e se houver algum link externo que não foi inserido pelo Ajax, o setInterval irá parar logo no início.

Deveria ser algo assim:

var as = $("#div a[href^='http']:not([href*='zaha.in'])");
            ↑↑↑↑
   especificar a div alvo do Ajax
Fonte Link
Sam
  • 80,8mil
  • 22
  • 73
  • 127

Ao meu ver não precisa atribuir eventos de click aos links <a>, basta apenas inserir o atributo target="_blank" que já irá abrir em uma nova guia.

Como você não sabe quanto tempo o Ajax irá levar para processar, uma forma é criar um setInterval que ficará rodando até encontrar um link externo definido no seu seletor. Assim que o if no setInterval for atendido, significa que o Ajax foi processado e os links foram inseridos, então basta adicionar o atributo target a todos os links de uma só vez, sem a necessidade de laço .each.

Veja como fica:

$(document).ready(function(){

   var niceJavascriptRoutine = setInterval(function(){

      var as = $("a[href^='http']:not([href*='zaha.in'])");
      
      if(as.length){
         as.attr('target', '_blank')
         .addClass('externalLink');
         clearInterval(niceJavascriptRoutine);
      }
   }, 1000);

});

O setInterval vai ficar rodando a segundo, e quando houver algum elemento na página que foi definido na variável as, irá aplicar o target a todos os elementos e parar o setInterval.

Agora, o ideal mesmo era buscar os links dentro de algum contêiner aonde o Ajax os inseriu, caso contrário o seletor a[href^='http'] irá buscar na página toda, e se houver algum link externo que não foi inserido pelo Ajax, o setInterval irá parar logo no início.

Deveria ser algo assim:

var as = $("#div a[href^='http']:not([href*='zaha.in'])");
            ↑↑↑↑
   especificar a div alvo do Ajax