From 6f4485a4539be52e85adec65858ecf6e2e7fb46a Mon Sep 17 00:00:00 2001
From: Peter Krempa <pkrempa@redhat.com>
Date: Thu, 29 Feb 2024 14:11:57 +0100
Subject: [PATCH] docs: newapi: Avoid table where every row has an cell with
 'colspan'

The HTML standard requires that a table column must include at least one
row which defines it exclusively, thus having a table where all rows
unite it via 'colspan' is illegal.

Modify the enum value generator to always output the description field
even when it's empty rather than uniting it, as in case when each value
doesn't have a description the generated document would violate the
standard.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Pavel Hrdina <phrdina@redhat.com>
---
 docs/newapi.xsl | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/newapi.xsl b/docs/newapi.xsl
index 539e0a4175..36bb41c877 100644
--- a/docs/newapi.xsl
+++ b/docs/newapi.xsl
@@ -317,9 +317,9 @@
           <tr>
             <td><a id="{@name}"><xsl:value-of select="@name"/></a></td>
             <td><xsl:text> = </xsl:text></td>
+            <td class="enumvalue"><xsl:call-template name="enumvalue"/></td>
             <xsl:choose>
               <xsl:when test="@info != ''">
-                <td class="enumvalue"><xsl:call-template name="enumvalue"/></td>
                 <td>
                   <div class="comment">
                     <xsl:call-template name="dumptext">
@@ -329,7 +329,7 @@
                 </td>
               </xsl:when>
               <xsl:otherwise>
-                <td colspan="2" class="enumvalue"><xsl:call-template name="enumvalue"/></td>
+                <td><xsl:comment> </xsl:comment></td>
               </xsl:otherwise>
             </xsl:choose>
           </tr>
-- 
2.47.2