From: Norman Walsh <ndw@nwalsh.com>
Date: Sun, 12 May 2002 11:18:47 +0000 (+0000)
Subject: Improve support for table borders drawn with CSS (get the semantics right); add suppo... 
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=67c6bcf725cac78679a7df41ec3ee6a1534562d8;p=docbook-dsssl

Improve support for table borders drawn with CSS (get the semantics right); add support for entry-bgcolor PI
---

diff --git a/xsl/html/table.xsl b/xsl/html/table.xsl
index ffdc38a76..fbbbce9fb 100644
--- a/xsl/html/table.xsl
+++ b/xsl/html/table.xsl
@@ -122,21 +122,56 @@
     </xsl:if>
 
     <xsl:choose>
-      <xsl:when test="../@frame='none'">
-        <xsl:attribute name="border">0</xsl:attribute>
-      </xsl:when>
       <xsl:when test="$table.borders.with.css != 0">
         <xsl:attribute name="border">0</xsl:attribute>
         <xsl:choose>
-          <xsl:when test="../@frame='topbot' or ../@frame='top'">
+          <xsl:when test="../@frame='all'">
+            <xsl:attribute name="style">
+              <xsl:text>border-collapse: collapse;</xsl:text>
+              <xsl:call-template name="border">
+                <xsl:with-param name="side" select="'top'"/>
+              </xsl:call-template>
+              <xsl:call-template name="border">
+                <xsl:with-param name="side" select="'bottom'"/>
+              </xsl:call-template>
+              <xsl:call-template name="border">
+                <xsl:with-param name="side" select="'left'"/>
+              </xsl:call-template>
+              <xsl:call-template name="border">
+                <xsl:with-param name="side" select="'right'"/>
+              </xsl:call-template>
+            </xsl:attribute>
+          </xsl:when>
+          <xsl:when test="../@frame='topbot'">
+            <xsl:attribute name="style">
+              <xsl:text>border-collapse: collapse;</xsl:text>
+              <xsl:call-template name="border">
+                <xsl:with-param name="side" select="'top'"/>
+              </xsl:call-template>
+              <xsl:call-template name="border">
+                <xsl:with-param name="side" select="'bottom'"/>
+              </xsl:call-template>
+            </xsl:attribute>
+          </xsl:when>
+          <xsl:when test="../@frame='top'">
             <xsl:attribute name="style">
+              <xsl:text>border-collapse: collapse;</xsl:text>
               <xsl:call-template name="border">
                 <xsl:with-param name="side" select="'top'"/>
               </xsl:call-template>
             </xsl:attribute>
           </xsl:when>
+          <xsl:when test="../@frame='bottom'">
+            <xsl:attribute name="style">
+              <xsl:text>border-collapse: collapse;</xsl:text>
+              <xsl:call-template name="border">
+                <xsl:with-param name="side" select="'bottom'"/>
+              </xsl:call-template>
+            </xsl:attribute>
+          </xsl:when>
           <xsl:when test="../@frame='sides'">
             <xsl:attribute name="style">
+              <xsl:text>border-collapse: collapse;</xsl:text>
               <xsl:call-template name="border">
                 <xsl:with-param name="side" select="'left'"/>
               </xsl:call-template>
@@ -145,8 +180,16 @@
               </xsl:call-template>
             </xsl:attribute>
           </xsl:when>
+          <xsl:otherwise>
+            <xsl:attribute name="style">
+              <xsl:text>border-collapse: collapse;</xsl:text>
+            </xsl:attribute>
+          </xsl:otherwise>
         </xsl:choose>
       </xsl:when>
+      <xsl:when test="../@frame='none'">
+        <xsl:attribute name="border">0</xsl:attribute>
+      </xsl:when>
       <xsl:otherwise>
         <xsl:attribute name="border">1</xsl:attribute>
       </xsl:otherwise>
@@ -341,7 +384,22 @@
 <xsl:template match="row">
   <xsl:param name="spans"/>
 
+  <xsl:variable name="row-height">
+    <xsl:if test="processing-instruction('dbhtml')">
+      <xsl:call-template name="dbhtml-attribute">
+        <xsl:with-param name="pis" select="processing-instruction('dbhtml')"/>
+        <xsl:with-param name="attribute" select="'row-height'"/>
+      </xsl:call-template>
+    </xsl:if>
+  </xsl:variable>
+
   <tr>
+    <xsl:if test="$row-height != ''">
+      <xsl:attribute name="height">
+        <xsl:value-of select="$row-height"/>
+      </xsl:attribute>
+    </xsl:if>
+
     <xsl:if test="$table.borders.with.css != 0">
       <xsl:if test="@rowsep = 1">
         <xsl:attribute name="style">
@@ -501,7 +559,22 @@
     </xsl:when>
 
     <xsl:otherwise>
+      <xsl:variable name="entry-bgcolor">
+        <xsl:if test="processing-instruction('dbhtml')">
+          <xsl:call-template name="dbhtml-attribute">
+            <xsl:with-param name="pis" select="processing-instruction('dbhtml')"/>
+            <xsl:with-param name="attribute" select="'entry-bgcolor'"/>
+          </xsl:call-template>
+        </xsl:if>
+      </xsl:variable>
+
       <xsl:element name="{$cellgi}">
+        <xsl:if test="$entry-bgcolor != ''">
+          <xsl:attribute name="bgcolor">
+            <xsl:value-of select="$entry-bgcolor"/>
+          </xsl:attribute>
+        </xsl:if>
+
         <xsl:if test="$show.revisionflag and @revisionflag">
           <xsl:attribute name="class">
             <xsl:value-of select="@revisionflag"/>