Closed mrlubos closed 2 years ago
Are you passing minify: true
?
Difference between 2.0.7 and 2.0.8 is that passing minify: true
actually does something in 2.0.8, previously it was deprecated in mjml
. If you are passing minify: true
then that should explain why it is breaking for you between 2.0.7 and 2.0.8.
@daliusd Hi Dalius. Yes, I am passing minify: true
in render()
from mjml-react
. I have many questions here but the most important one is: shouldn't minifying NOT alter the functionality? I honestly don't care about this parameter and it will be no issue setting it to false
, but I'd expect my templates to work regardless of this flag's value.
That's what I would expect as well, but I guess minifiers are different and that's why I was reluctant adding anything at all into mjml-react.
I am closing this issue as everything works as expected for now.
Hey there! The latest version 2.0.8 seems to be breaking my templates.
Here's output from 2.0.7:
And here's the same code output from 2.0.8:
You can verify these don't produce the same visual result using the SendGrid UI editor for example, that's what I use. Also, please consider adding a changelog to this project so it's easier to identify which version might be breaking things! I had to go back to the latest known version and test each release to detect where the change occurred.