Question

嘿伙计们，不应该这样吗？

//Add class if target blank
    $('.post .entry a').each(function() {
        if ( $(this).attr('target') == '_blank') ) {
            $(this).addClass('web');
        };
    });

这有什么不妥吗？

Answer 1

请尝试

//Add class if target blank
$('.post .entry a[target="_blank"]').addClass('web');

Answer 2

你有一个额外的)。只需删除它就可以了：）

$('.post .entry a').each(function() {
    if ( $(this).attr('target') == '_blank') ) {
---------------------------------------------^
        $(this).addClass('web');
    };
});

Answer 3

没关系。但这应该在document.ready中：

$(document).ready(function(){
   $('.post .entry a').each(function() {
        if ( $(this).attr('target') == '_blank') ) {
            $(this).addClass('web');
        };
    });
});

希望这会有所帮助。干杯

Answer 4

你在if语句中有额外的)。把它拿走就可以了：

$('.post .entry a').each(function() {
    if ( $(this).attr('target') == '_blank' ) {
        $(this).addClass('web');
    };
});

更简单的方法是：

$('.post .entry a[target="_blank"]').each(function() {
    $(this).addClass('web');
});

也使用较少的代码。

实际上@ ariel的回答说明了一种更好的方法。

Answer 5

我发现以下成功：

$('.post .entry a[href="_blank"]').addClass('web');

jquery：如果链接有目标空白，则添加类？

5 个答案: