From: Keith Fahlgren <abdelazer@users.sourceforge.net>
Date: Tue, 27 May 2008 15:54:46 +0000 (+0000)
Subject: Work to make HTML and XHTML targets more valid
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=593db7980e413ae2ba3a9018b7d124dfbb64f643;p=docbook-dsssl

Work to make HTML and XHTML targets more valid
---

diff --git a/xsl/html/inline.xsl b/xsl/html/inline.xsl
index ab745b07e..f218c1799 100644
--- a/xsl/html/inline.xsl
+++ b/xsl/html/inline.xsl
@@ -738,7 +738,10 @@
 <xsl:template match="emphasis">
   <span>
     <xsl:choose>
-      <xsl:when test="@role and $emphasis.propagates.style != 0">
+      <!-- We don't want empty @class values, so do not propagate empty @roles -->
+      <xsl:when test="@role  and
+                      normalize-space(@role) != '' and
+                      $emphasis.propagates.style != 0">
         <xsl:apply-templates select="." mode="class.attribute">
           <xsl:with-param name="class" select="@role"/>
         </xsl:apply-templates>
@@ -799,7 +802,10 @@
     <xsl:if test="@lang or @xml:lang">
       <xsl:call-template name="language.attribute"/>
     </xsl:if>
-    <xsl:if test="@role and $phrase.propagates.style != 0">
+    <!-- We don't want empty @class values, so do not propagate empty @roles -->
+    <xsl:if test="@role and 
+                  normalize-space(@role) != '' and
+                  $phrase.propagates.style != 0">
       <xsl:apply-templates select="." mode="class.attribute">
         <xsl:with-param name="class" select="@role"/>
       </xsl:apply-templates>
diff --git a/xsl/html/synop.xsl b/xsl/html/synop.xsl
index caa3129bb..1ac6f88d5 100644
--- a/xsl/html/synop.xsl
+++ b/xsl/html/synop.xsl
@@ -150,7 +150,9 @@
   <xsl:variable name="snum">
     <xsl:apply-templates select="." mode="synopfragment.number"/>
   </xsl:variable>
-  <p>
+  <!-- You can't introduce another <p> here, because you're 
+       already in a <p> from cmdsynopsis-->
+  <span>
     <xsl:variable name="id">
       <xsl:call-template name="object.id"/>
     </xsl:variable>
@@ -161,7 +163,7 @@
     </a>
     <xsl:text> </xsl:text>
     <xsl:apply-templates/>
-  </p>
+  </span>
 </xsl:template>
 
 <xsl:template match="funcsynopsis">
@@ -365,9 +367,9 @@ paramdef      ::= (#PCDATA|type|replaceable|parameter|funcparams)*
     </xsl:for-each>
   </table>
   <xsl:if test="paramdef">
-    <blockquote class="paramdef-list">
+    <div class="paramdef-list">
       <xsl:apply-templates select="paramdef" mode="kr-funcsynopsis-mode"/>
-    </blockquote>
+    </div>
   </xsl:if>
   <div class="funcprototype-spacer">&#160;</div> <!-- hACk: blank div for vertical spacing -->
 </xsl:template>