From: Bob Stayton <bobs@sagehill.net>
Date: Mon, 20 Jan 2014 19:54:56 +0000 (+0000)
Subject: Fix bug #1298 where @rowheader = 'firstcol' incorrectly assigns
X-Git-Tag: release/1.79.1~6^2~116
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=740ef5afcad0c9894e33c8b331c040327991d3fa;p=docbook-dsssl

Fix bug #1298 where @rowheader = 'firstcol' incorrectly assigns
<th> to cell that is first in a row but not the first column due
to row span above.
---

diff --git a/xsl/html/table.xsl b/xsl/html/table.xsl
index c8969924e..f3df9320c 100644
--- a/xsl/html/table.xsl
+++ b/xsl/html/table.xsl
@@ -697,20 +697,6 @@
 
   <xsl:param name="spans"/>
 
-  <xsl:variable name="cellgi">
-    <xsl:choose>
-      <xsl:when test="ancestor::thead">th</xsl:when>
-      <xsl:when test="ancestor::tfoot">th</xsl:when>
-      <xsl:when test="ancestor::tbody and 
-                      (ancestor::table[@rowheader = 'firstcol'] or
-                      ancestor::informaltable[@rowheader = 'firstcol']) and
-                      ancestor-or-self::entry[1][count(preceding-sibling::entry) = 0]">
-        <xsl:text>th</xsl:text>
-      </xsl:when>
-      <xsl:otherwise>td</xsl:otherwise>
-    </xsl:choose>
-  </xsl:variable>
-
   <xsl:variable name="empty.cell" select="count(node()) = 0"/>
 
   <xsl:variable name="named.colnum">
@@ -744,6 +730,20 @@
     </xsl:call-template>
   </xsl:variable>
 
+  <xsl:variable name="cellgi">
+    <xsl:choose>
+      <xsl:when test="ancestor::thead">th</xsl:when>
+      <xsl:when test="ancestor::tfoot">th</xsl:when>
+      <xsl:when test="ancestor::tbody and 
+                      (ancestor::table[@rowheader = 'firstcol'] or
+                      ancestor::informaltable[@rowheader = 'firstcol']) and
+                      $entry.colnum = 1">
+        <xsl:text>th</xsl:text>
+      </xsl:when>
+      <xsl:otherwise>td</xsl:otherwise>
+    </xsl:choose>
+  </xsl:variable>
+
   <xsl:variable name="rowsep">
     <xsl:choose>
       <!-- If this is the last row, rowsep never applies. -->