From: Bob Stayton <bobs@sagehill.net>
Date: Tue, 22 Jan 2013 19:48:46 +0000 (+0000)
Subject: Allow img elements to retain width and height attributes instead of styles.
X-Git-Tag: release/1.79.1~6^2~244
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=02e1aeaec9b8edc77fd31dda096539734b34fd58;p=docbook-dsssl

Allow img elements to retain width and height attributes instead of styles.
---

diff --git a/xsl/xhtml5/html5-element-mods.xsl b/xsl/xhtml5/html5-element-mods.xsl
index a362d9b0b..61d6e4cd5 100644
--- a/xsl/xhtml5/html5-element-mods.xsl
+++ b/xsl/xhtml5/html5-element-mods.xsl
@@ -336,17 +336,20 @@
 <!-- HTML5: converts obsolete HTML attributes to CSS styles -->
 <xsl:template match="*" mode="convert.to.style">
 
+  <xsl:variable name="element" select="local-name(.)"/>
+
   <xsl:variable name="style.from.atts">
     <xsl:for-each select="@*">
 
       <xsl:choose>
-        <xsl:when test="local-name() = 'width'">
+        <!-- width and height attributes are ok for img element -->
+        <xsl:when test="local-name() = 'width' and $element != 'img'">
           <xsl:text>width: </xsl:text>
           <xsl:value-of select="."/>
           <xsl:text>; </xsl:text>
         </xsl:when>
 
-        <xsl:when test="local-name() = 'height'">
+        <xsl:when test="local-name() = 'height' and $element != 'img'">
           <xsl:text>height </xsl:text>
           <xsl:value-of select="."/>
           <xsl:text>; </xsl:text>
@@ -401,16 +404,22 @@
         <xsl:value-of select="$style"/>
       </xsl:attribute>
     </xsl:if>
-    <!-- Also skip disallowed summary attributes -->
-    <xsl:copy-of select="@*[local-name(.) != 'width' and
-                            local-name(.) != 'height' and
-                            local-name(.) != 'summary' and
-                            local-name(.) != 'border' and
-                            local-name(.) != 'cellspacing' and
-                            local-name(.) != 'cellpadding' and
-                            local-name(.) != 'style' and
-                            local-name(.) != 'align' and
-                            local-name(.) != 'valign']"/>
+    <!-- skip converted atts, and also skip disallowed summary attribute -->
+    <xsl:for-each select="@*">
+      <xsl:choose>
+        <xsl:when test="local-name(.) = 'width' and $element != 'img'"/>
+        <xsl:when test="local-name(.) = 'height' and $element != 'img'"/>
+        <xsl:when test="local-name(.) = 'summary'"/>
+        <xsl:when test="local-name(.) = 'border'"/>
+        <xsl:when test="local-name(.) = 'cellspacing'"/>
+        <xsl:when test="local-name(.) = 'cellpadding'"/>
+        <xsl:when test="local-name(.) = 'style'"/>
+        <xsl:when test="local-name(.) = 'align'"/>
+        <xsl:otherwise>
+          <xsl:copy-of select="."/>
+        </xsl:otherwise>
+      </xsl:choose>
+    </xsl:for-each>
     <xsl:apply-templates mode="convert.to.style"/>
   </xsl:element>
 </xsl:template>