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