Repeated mj-tag's inside mj-attributes get ignored

[rinopo]

Hi,

Repeated mj-tags inside the mj-attributes seem to be ignored except for the last ones.

Reproduction Steps:

Use following MJML sources:

https://mjml.io/try-it-live/HkEN2o8Az

<mjml>
  <mj-head>
    <mj-attributes>
      <mj-all color="blue" />
      <mj-all font-size="40px" />
    </mj-attributes>
  </mj-head>
  <mj-body>
    <mj-section>
      <mj-column>
        <mj-text>
          Hello World!
        </mj-text>
      </mj-column>
    </mj-section>
  </mj-body>
</mjml>

or

<mjml>
  <mj-head>
    <mj-attributes>
      <mj-text color="blue" />
      <mj-text font-size="40px" />
    </mj-attributes>
  </mj-head>
  <mj-body>
    <mj-section>
      <mj-column>
        <mj-text>
          Hello World!
        </mj-text>
      </mj-column>
    </mj-section>
  </mj-body>
</mjml>

Expected behavior:

Both color and font-size attributes are applied to Hello World!.

Observed behavior:

Only the last attribute (here font-size) is applied to Hello World!.

MJML version:

The "try-it-live" version (4.0.5?).

[rinopo]

You could work around this like:

    <mj-attributes>
      <mj-text color="blue" font-size="40px" />
    </mj-attributes>

But I thought I'd file it as an issue since:

• Repeated mj-tags inside mj-attributes used to work back in MJML3.
• It is not clearly documented (allowed or not allowed).
• I prefer using repeated mj-tags because I find it more readable, especially when you want to specify more than a couple of attributes.

Thank you so much for your consideration.

[iRyusa]

Thanks for finding this @rinopo

We'll try to fix it for 4.1 release. cc @kmcb777

[kmcb777]

Hi @rinopo we released a new beta for mjml 4.1, which you can install with npm install mjml@next, could you check if it resolves the problem you had ?