Hi,

I was getting a warning from IE browsers about the page containing both secure and insecure elements every time I enabled the add this module and navigated to a https page. While an earlier patch fixed the inclusion of the image for https, it didn't address the inclusion of the stylesheet. The attached patch fixes this.

Cheers,
Stella

CommentFileSizeAuthor
#6 addthis.module.patch594 bytesstefanhoth
ssl.patch948 bytesstella

Comments

epiphanydigital’s picture

epiphanydigital commented

Same issue. Thanks for the patch!

rgibson75’s picture

rgibson75 commented

Had the same issue - the patch fixed the problem. Thanks!

mrfelton’s picture

mrfelton commented
Version: 6.x-2.8 » 6.x-2.9
Status: Needs review » Reviewed & tested by the community

Same problem here. Patch fixes. Please get this committed.

timwood’s picture

timwood
Primary language English
Location Rockville, Maryland
commented

Thanks for the patch. Applied, subscribing to this post so I can find out when it makes it into a released version.

vega’s picture

vega commented

Patch works for me. http://blog.httpwatch.com/2009/04/23/fixing-the-ie-8-warning-do-you-want... helped me figure out that this was the issue.

stefanhoth’s picture

stefanhoth commented
Status: Reviewed & tested by the community » Patch (to be ported)
StatusFileSize
new addthis.module.patch594 bytes

I stumbled upon the same issue.

I added a patch without repeating unneccessary lines and using the same test for SSL the module uses in other places.

stella’s picture

stella commented
Status: Patch (to be ported) » Reviewed & tested by the community
vesapalmu’s picture

vesapalmu commented
Status: Reviewed & tested by the community » Fixed

Thank you Stella. This will make it to 2.10 version and it's finally been commited.

Status: Fixed » Closed (fixed)

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