The js code checks if an has a line display. If so, it wraps a span around it with the class ext. If not, the ext class is added to the tag itself. However, extlink.css only contains a icon background for span.ext and span.mailto, not for a.ext and a.mailto.

I have some links in my site with display:block (some CCK Links in my sidebar for example). On these links, the icon does not appear.

Comments

BarisW’s picture

BarisW commented

Apologies, forgot to add <code> blocks...

The js code checks if an <a> has a display:inline display. If so, it wraps a span around it with the class ext. If not, the ext class is added to the <a> tag itself. However, extlink.css only contains a background-image for span.ext and span.mailto, not for a.ext and a.mailto

I have some links on my site with display:block (some CCK Links in my sidebar for example). On these links, the icon does not appear.

quicksketch’s picture

quicksketch
he/him
commented

This is intentional, since often times links are display: block because they have a background image or are part of the primary navigation or tabs.

RaiMan’s picture

RaiMan commented
Title: Doesn't work with a display:block links » Doesn't work with a display:block links / in menu items

Don't know wether same problem:
in menu items: ext link is recognized (opens in new window) but icon is not attached.

quicksketch’s picture

quicksketch
he/him
commented
Category: bug » support

in menu items: ext link is recognized (opens in new window) but icon is not attached.

Yes this is probably because your theme uses display: block for displaying menu items in your navigation.

As I noted above, this is by design.

RaiMan’s picture

RaiMan commented
Status: Active » Closed (works as designed)

Thanks.

I just started with drupal and for my intentions your module is a must.

Using standard Garland, because my priority is functionality in the moment.

I've reset status to 'by design', but it would be great to get an answer, wether there will be a chance to get a solution for that or not.