Closed (duplicate)
Project:
Mime Mail
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
13 Nov 2008 at 21:39 UTC
Updated:
18 Feb 2009 at 01:20 UTC
We have had several users contact us to let us know our newsletters cannot be displayed on iPhones. It seems as though something may not be properly escaped, because the emails begin with a /******* MAIL STYLING *******/ section, which displays a list of the CSS for the message to the user.
Comments
Comment #1
virtualdrupal commentedI can confirm the issue in D6
for emails like the drupal contact form, with mime enabled, the iphone attaches TONS of styling code.
Comment #2
simeNo complete answers yet, but here's what is happening as far as I've worked out.
There are 2 aspects:
1. iphone won't handle base64 encoding, so it's using the plain text version
2. mimemail is inserting the site's style sheets into the plain text version.
I'll post back if I get further.
Comment #3
simeI think the problem is not with the iPhone anymore, well not exactly. With a small about of css the iphone displays the message fine.
But with the huge amount of css, not only the iPhone fails, but so does gmail (if gmail is set to only read the text version). So I'm guessing that when mimemail inserts the full stylesheet (which is like 50k charcters in our case) something is encoding badly and is not escaped properly. (??)
I'm updating to 6 dev, which is what we're using, but I guess the problem spans both.
Comment #4
virtualdrupal commentedI paid someone to fix this issue, but they don't have time to create a patch, here's a copy of the mimemail.module, can someone please create a patch.. if this doesn't work I'll include the mimemail.inc also, but I don't think any changes were made to it.
http://www.drupalbin.com/4542
This fixes the Iphone AND hotmail for me
Comment #5
virtualdrupal commentedHi, looks like I had it backwords maybe.. here is a copy of the mimemail.inc
http://www.drupalbin.com/4546
Comment #6
simemikenotmike is this a modification of the D6 version?
Comment #7
simeDuplicate of #310305: Disabling base64 encoding - hotmail compatibility where there is a patch.
Comment #8
virtualdrupal commentedYes, this was for D6