extra column when you type html comment in <mj-group>

[xunuo]

Describe the bug
extra column when you type html comment in <mj-group>

To Reproduce
Steps to reproduce the behavior:

<mj-group>
  <!-- foreach($n){ -->
  <mj-column>
    ....
  </mj-column>
  <!-- } -->
  <mj-column>
    ....
  </mj-column>
</mj-group>

will build 4 columns, and for each width will be 16%

Expected behavior
only 2 column

MJML environment (please complete the following information):

• OS: MacOS
• MJML Version 4.2.0

[iRyusa]

Hi,

I think this is fixed in 4.3.X https://mjml.io/try-it-live/r1vAwVU1r ?

You should use mj-raw for that instead of using html comment like that:

<mj-group>
<mj-raw>foreach($n){ </mj-raw>
  <mj-column>
    ....
  </mj-column>
<mj-raw>}</mj-raw>
  <mj-column>
    ....
  </mj-column>
</mj-group>

[xunuo]

Hi,

I think this is fixed in 4.3.X https://mjml.io/try-it-live/r1vAwVU1r ?

You should use mj-raw for that instead of using html comment like that:

<mj-group>
<mj-raw>foreach($n){ </mj-raw>
  <mj-column>
    ....
  </mj-column>
<mj-raw>}</mj-raw>
  <mj-column>
    ....
  </mj-column>
</mj-group>

thanks for answer, but it's not working...

[iRyusa]

Oh you're right ! cc @kmcb777 maybe we can add this to 4.4, i think you just need to blacklist raw children ?

A temporary fix would be to define the width to 50% directly on both column inside mj-group !

[kmcb777]

@xunuo Hi, thanks for reporting this, there was indeed a problem 👍
It will be fixed in the next release

Raw elements were correctly ignored in mj-section but not in mj-group

[xunuo]

@xunuo Hi, thanks for reporting this, there was indeed a problem 👍
It will be fixed in the next release

Raw elements were correctly ignored in mj-section but not in mj-group

Thanks!!! look forward it!

[xunuo]

@kmcb777 hello, could we use the new version now, we have a project need it :)

[xunuo]

how to use the newest version :)

[iRyusa]

This has not been released yet, it should be available soon. As a temporary fix just fix the column width to 50% manually in the group

[kmcb777]

Hi @xunuo , new version has been released, you can check if this problem is fixed. Still in beta so you need to install with the tag mjml@next