Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MJML replace css break-all by break-word #1728

Closed
antoninbeaufort opened this issue Oct 9, 2019 · 1 comment
Closed

MJML replace css break-all by break-word #1728

antoninbeaufort opened this issue Oct 9, 2019 · 1 comment

Comments

@antoninbeaufort
Copy link

Describe the bug
MJML replace css break-all by break-word, which don't work on Windows 10 Mail and Outlook,
how to force break-all on ms ?

To Reproduce
Steps to reproduce the behavior:

  1. Create a file with this MJML code:
    <mj-style inline="inline"> .break { -ms-word-break: break-all; } </mj-style>
    and
    class="break" on very long link
  2. Send mail and check it on Outlook, open mail in IE to inspect code and

Expected behavior
The link should not be outside of 600px section.

MJML environment (please complete the following information):

  • OS: [e.g. MacOS]
  • MJML Version [e.g. 4.2.0]
  • MJML tool used [e.g MJML App]

Email sending environment(for rendering issues):

  • Platform used to send the email [e.g Putsmail]

Affected email clients (for rendering issues):

  • Email Client : Outlook,
  • OS: Windows 10 1908 (build 12026.20108)
  • Browser : IE 11

Screenshots
bug-mjml

@kmcb777
Copy link
Collaborator

kmcb777 commented Oct 9, 2019

Hi @antoninbeaufort thanks for reporting this,
There is a default value of word-break: break-word already added inline by mjml on this element.
After examination it turns out that the lib we use for inlining css doesn't replace the existing value.
We'll fix that, for now as a workaround, if you add word-break: break-all !important; in your inline mj-style, you'll get the result you want (it won't preserve the 'important' tag)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants