Skip to content

callouts with classes are now rendered correctly - #8

Merged
lorenzo merged 1 commit into
lorenzo:masterfrom
Basster:callout-fix
Sep 18, 2018
Merged

callouts with classes are now rendered correctly#8
lorenzo merged 1 commit into
lorenzo:masterfrom
Basster:callout-fix

Conversation

@Basster

@Basster Basster commented Sep 18, 2018

Copy link
Copy Markdown

The current version of the callouts with classes is erroneous:

<table class="primary callout">
  <tr>
    <th class="callout-inner">
      <p>foobar</p>
    </th>
    <th class="expander"></th>
  </tr>
</table>

According to https://foundation.zurb.com/emails/docs/callout.html, callouts with classes should produce the following html:

<callout class="primary">
  <p>foobar</p>
</callout>

produces

<table class="callout">
  <tr>
    <th class="callout-inner primary">
      <p>foobar</p>
    </th>
    <th class="expander"></th>
  </tr>
</table>

This PR fixes this...

Nevertheless callout template was duplicated on the inky.xsl.

@lorenzo

lorenzo commented Sep 18, 2018

Copy link
Copy Markdown
Owner

Thanks!

@lorenzo
lorenzo merged commit 5694315 into lorenzo:master Sep 18, 2018
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

Successfully merging this pull request may close these issues.

2 participants