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
Comment #1
bryanhirsch commentedComment #2
quicksketchThe adding to both the span and a tags is intentional.
Comment #3
bryanhirsch commentedWhat's the benefit of adding this class to the
<a>tags?Comment #4
ksenzeeFor one thing, that's the mechanism by which the module keeps from processing the link every time Drupal.behaviors is run.
Comment #4.0
ksenzeeUpdated issue summary.