callouts with classes are now rendered correctly, according to https:…

…//foundation.zurb.com/emails/docs/callout.html
lorenzo · Sep 18, 2018 · 84a24a3 · 84a24a3
1 parent a182f26
commit 84a24a3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 14 deletions.
diff --git a/src/inky.xsl b/src/inky.xsl
@@ -164,20 +164,10 @@
     </xsl:template>
 
     <xsl:template match="//callout">
-        <table class="">
+        <table class="callout">
             <xsl:copy-of select="@*[name()!='class']"/>
             <tr>
-                <th class="callout-inner"><xsl:apply-templates /></th>
-                <th class="expander"></th>
-            </tr>
-        </table>
-    </xsl:template>
-
-    <xsl:template match="//callout">
-        <table class="{normalize-space(concat(@class, ' callout'))}">
-            <xsl:copy-of select="@*[name()!='class']"/>
-            <tr>
-                <th class="callout-inner"><xsl:apply-templates /></th>
+                <th class="{normalize-space(concat('callout-inner ', @class))}"><xsl:apply-templates /></th>
                 <th class="expander"></th>
             </tr>
         </table>

diff --git a/tests/ComponentsTest.php b/tests/ComponentsTest.php
@@ -319,9 +319,9 @@ public function testCreateCalloutWithClasses()
         <callout class="primary">Callout</callout>
 doc;
         $expected =<<<doc
-        <table class="primary callout">
+        <table class="callout">
             <tr>
-              <th class="callout-inner">Callout</th>
+              <th class="callout-inner primary">Callout</th>
               <th class="expander"></th>
             </tr>
         </table>