extlink is adding the extClass (which defaults to "ext") to both <a> tags and <span> tags for external links. As far as I can tell, this is more than necessary. All we need if for this class to be applied to tags.

I have a use case where I would like to be able to change the external link icon by doing this: .ext { background: url('example.png'); } rather than having to do this: span.ext { background: url('example.png'); }

Seems like I should be able to do it.

Attached is a little patch, applied on top of version 7.x-1.12 to fix this.

Comments

bryanhirsch’s picture

quicksketch’s picture

Status: Active » Closed (works as designed)

The adding to both the span and a tags is intentional.

bryanhirsch’s picture

What's the benefit of adding this class to the <a> tags?

ksenzee’s picture

For one thing, that's the mechanism by which the module keeps from processing the link every time Drupal.behaviors is run.

ksenzee’s picture

Issue summary: View changes

Updated issue summary.