mjmlio / mjml

MJML: the only framework that makes responsive-email easy
https://mjml.io
MIT License
17.1k stars 961 forks source link

Outlook web client doesn't render columns #2876

Closed dpd73 closed 4 months ago

dpd73 commented 5 months ago

Describe the bug Outlook web client doesn't render columns on Mac Litmus.com shows the issue... However, this appears to be an issue for Mac and not Windows outlook web client?!

To Reproduce Steps to reproduce the behavior:

  1. Create a single file, basic layout MJML (this has two simple columns)
  2. Send it
  3. Observe columns stacked, not side by side

Expected behavior Columns appear side by side

MJML environment (please complete the following information):

Email sending environment(for rendering issues):

Affected email clients (for rendering issues):

Screenshots Screenshot 2024-06-26 at 17 42 00

Additional context This seems to be an intermittent outlook.com thing? Could well be nothing to do with MJML but a change at Microsoft? Putting this here in case any other unfortunate soul encounters the issue or someone else has further insight.

I did see intermittent expected rendering at litmus Screenshot 2024-06-26 at 16 46 15

dpd73 commented 5 months ago

Can someone confirm that this is likely due to lack of media query support in the web app? https://mjml.io/compatibility/mj-column (Seems odd that it sometimes gets honoured as per above last screen grab)

iRyusa commented 5 months ago

Outlook did update their engine removing support for media queries for now. There's a fix we should implement soon. Last time it was a temporary issue that was resolved from Microsoft, so let's wait a bit before applying the fix.

On Wed, Jun 26, 2024 at 7:34 PM David Pead @.***> wrote:

Can someone confirm that this is likely due to lack of media query support in the web app? https://mjml.io/compatibility/mj-column (Seems odd that is sometimes gets honoured as per above last screen grab)

— Reply to this email directly, view it on GitHub https://github.com/mjmlio/mjml/issues/2876#issuecomment-2192272138, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAELHTMWINXR33AECXXFBRDZJL3SFAVCNFSM6AAAAABJ6I2RYSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJSGI3TEMJTHA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

dpd73 commented 5 months ago

Adding the <style> elements to the <body> fixes the issue, as suggested elsewhere.

iRyusa commented 5 months ago

Reopening for now as it's still the case

iRyusa commented 4 months ago

Closing again (🤡 sorry)

Seems like Microsoft reverted the change 👍 no need to add a fix to MJML for that then 🎉