strange graphic placement when link wraps line

Could you try replacing the extlink.css with this code and let me know if it works on IE6 and IE7? It seems to improve (though not make perfect) the icon placement in IE6 for multiple lines:

The only difference is the "zoom" property, which seems to trigger IE's "hasLayout" property. I don't have an IE7 install I can test on at the moment.

My patch seems to cause odd behavior when a really long link is in the middle of a paragraph (shot attached). If you can come up with a CSS hack to make IE render the link properly I'll be glad to commit it.

Is there any status update on this?

Nice module indeed.

Yes, it is a very nice module, but we I too have the problem with displaying links that wrap onto the second line - only in IE (7), not in Firefox (2) or Opera (9.2).

Can the placement of the icon before the link fix this problem? I would prefer this option in general. That would fix the (optical) problem with ®, ™, and 's after/outside the link. I can post this as a feature request, too.

My solution was to append an (admittedly hackish) empty span to the end of the link and apply the graphic to that. Seems to resolve the positioning issues in IE6+. I have attached a patch against 5.x-1.6 for anyone interested.

Interesting! This looks like it might be a good approach! Marking code needs review to remind me to take a look. Any interested users, please also test and see if this looks like a valid approach.

This version of the patch works a little bit better, and should break less installations for those people upgrading.

- The "ext" class is still kept on the link itself, but it's also added to the span that is appended. The CSS is just specifically targeted to the span.ext class.
- The span is only appended if the link is being displayed inline. This keeps the external icon from showing up on a new line after links that might be in the menu ULs or on links that have image replacement in place.

Thoughtful additions. I'll be using #8 on a new project in the next few days. Thanks.

thanks , it works for me too

Subscribing to see when a new version with this fixed is made available. I have no clue how to patch modules unless downloading a replacement version.

Can I get this patched version as a full module zip?

I've gone ahead and committed the patch in #8, so you don't have to worry about patching it now. You can download the fully patched, latest version from the releases page. Go to http://drupal.org/project/extlink, click "View all releases", and find the 5.x-1.x-dev version (or 6.x version if you're using Drupal 6).

I won't roll out a new release until a whitelist is implemented, as this new approach makes exempting certain links more difficult than it was previously.

Thanks. Will test today and report back.

Editing to report that unfortunately I am still getting the same behavior in IE7. Still will continue to watch this topic for a future solution. Thanks again.

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

This still doesn't work on IE 7, FYI. Maybe there's nothing else you can do about it but if that's the case the module documentation should mention that it does not work properly on IE7.

@rgracia: I'm using IE7, and it does work for me.

Does anybody have solution for zoom:1 not validating for CSS? Is their any other option other than using zoom

.extlink-wrapper{
position: relative;
width: 12px;
margin: 0 12px 0 0;
}

ADD THE EXTRA SPAN AND NOW IT VALIDATES!!!!