JS works with wrong class name for external links.

js/elf.js contains:
$("a.external-link").each(function() {
but it should be:
$("a.elf-external").each(function() {

So, 'Use JavaScript to open external links in a new window' doesn't work due to this issue.

I'm going to provide a patch.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Konstantin Komelin’s picture

Attached the patch.
Please review.

Konstantin Komelin’s picture

Status: Active » Needs review
WorldFallz’s picture

Version: 7.x-3.x-dev » 6.x-3.x-dev
Issue summary: View changes
Status: Needs review » Patch (to be ported)

This has finally been fixed. Thanks for the patch. Need to do for d6 also.

WorldFallz’s picture

Status: Patch (to be ported) » Fixed

done

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.