From: Norman Walsh <ndw@nwalsh.com>
Date: Wed, 1 Jan 2003 20:26:23 +0000 (+0000)
Subject: CALS says the default for colsep and rowsep is 1.
X-Git-Tag: release/1.79.1~6^2~4980
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9727f59db9047a500647c5a3ae24cdb4dd694a47;p=docbook-dsssl

CALS says the default for colsep and rowsep is 1.
---

diff --git a/xsl/common/table.xsl b/xsl/common/table.xsl
index eb59715b3..416d346f7 100644
--- a/xsl/common/table.xsl
+++ b/xsl/common/table.xsl
@@ -314,22 +314,16 @@ or 0 (the empty string)</para>
   </xsl:variable>
 
   <xsl:variable name="default.value">
-    <!-- rowsep and colsep have defaults based ultimately on the frame setting -->
-    <!-- handle those here, for everything else, the default is the tgroup value -->
+    <!-- This section used to say that rowsep and colsep have defaults based -->
+    <!-- on the frame setting. Further reflection and closer examination of the -->
+    <!-- CALS spec reveals I was mistaken. The default is "1" for rowsep and colsep. -->
+    <!-- For everything else, the default is the tgroup value -->
     <xsl:choose>
       <xsl:when test="$tgroup.value != ''">
         <xsl:value-of select="$tgroup.value"/>
       </xsl:when>
-      <xsl:when test="$attribute = 'rowsep'">
-        <xsl:if test="$tgroup/parent::*/@frame = 'all'">
-          <xsl:value-of select="1"/>
-        </xsl:if>
-      </xsl:when>
-      <xsl:when test="$attribute = 'colsep'">
-        <xsl:if test="$tgroup/parent::*/@frame = 'all'">
-          <xsl:value-of select="1"/>
-        </xsl:if>
-      </xsl:when>
+      <xsl:when test="$attribute = 'rowsep'">1</xsl:when>
+      <xsl:when test="$attribute = 'colsep'">1</xsl:when>
       <xsl:otherwise><!-- empty --></xsl:otherwise>
     </xsl:choose>
   </xsl:variable>