"mailto" links over images displayed broken

Subscribing

not tested so I will not create a patch:

In spamspan.js (line 34), replace:

var _anchorText = $("span." +  Drupal.settings.spamspan.t, this).text().replace(/^ \((.*)\)$/, "$1");

with:

var _anchorText = $("span." +  Drupal.settings.spamspan.t, this).html().replace(/^ \((.*)\)$/, "$1");

Using .text() discards the tag, using .html() keeps it.

Of course, to work, you have to compress again the spamspan.js file into spamspan.compressed.js

Hope this help.

Fixed in 6--1.5. Thanks.

Reviving this bug. The fix applied did not work when there was no nested element within the <a> tag, so I have removed it.

The fix also overrides the image alignment when linking the image to an email.

I have created a patch which solves the problems above.
The code first checks if an image is present and then utilizes .html (keeping the img tag in place) otherwise it just uses .text

There is still the issue where the filter_xss() function in spamspan.module throws out any styling inserted by wysiwyg, such as: style="align:right;"

So for that I replaced (Line 224):
$output = filter_xss($output, $allowed_tags = array('em', 'strong', 'cite', 'b', 'i', 'code', 'span', 'img'));
with:
//$output = filter_xss($output, $allowed_tags = array('em', 'strong', 'cite', 'b', 'i', 'code', 'span', 'img'));

This disables the filter, but then also presents a security risk. I am not too sure how to get around this one yet.

Re-rolled the patch in #9 for version 7.x-1.1 beta 1

Thanks all for the patches. I have integrated the main idea of the patches in the patch below. In addition it deals with the fact that latest 7.x.-1.x dev code is first converting images to <!-- img $hash --> and then puts them back.

I don't know what happened, this was fixed in D6-1.5 but reverted in D6-1.6. When you compare the to version the 1.6 is reverted back to using the .text() method rather than the .html() method.

Can we maybe get this committed to the D6 branch and then get a new release.