From a1cc62bb9f7da9d12302ea86589ae70ca4f6edf2 Mon Sep 17 00:00:00 2001
From: Jeroen Hellingman <jeroen@bohol.ph>
Date: Sat, 21 Sep 2024 16:21:14 +0200
Subject: [PATCH] Handle footnotes smarter with aligned divisions

If only one side has footnotes, don't use two columns for those notes.
---
 modules/divisions.xsl  | 53 +++++++++++++++++++++++++++++++-----------
 style/aligned-text.css |  1 +
 2 files changed, 40 insertions(+), 14 deletions(-)

diff --git a/modules/divisions.xsl b/modules/divisions.xsl
index b88a0d3a..afce4073 100644
--- a/modules/divisions.xsl
+++ b/modules/divisions.xsl
@@ -816,20 +816,45 @@
 
         <!-- Include footnotes if at div1 level (for both sides) -->
         <xsl:if test="$a/../div1">
-            <xsl:if test="$a//note[f:is-footnote(.)] or $b//note[f:is-footnote(.)]">
-                <tr>
-                    <td class="first">
-                        <xsl:call-template name="insert-footnotes">
-                            <xsl:with-param name="div" select="$a"/>
-                        </xsl:call-template>
-                    </td>
-                    <td class="second">
-                        <xsl:call-template name="insert-footnotes">
-                            <xsl:with-param name="div" select="$b"/>
-                        </xsl:call-template>
-                    </td>
-                </tr>
-            </xsl:if>
+            <xsl:choose>
+                <!-- Only footnotes on left-hand side -->
+                <xsl:when test="$a//note[f:is-footnote(.)] and not($b//note[f:is-footnote(.)])">
+                    <tr>
+                        <td colspan="2">
+                            <xsl:call-template name="insert-footnotes">
+                                <xsl:with-param name="div" select="$a"/>
+                            </xsl:call-template>
+                        </td>
+                    </tr>
+                </xsl:when>
+
+                <!-- Only footnotes on right-hand side -->
+                <xsl:when test="not($a//note[f:is-footnote(.)]) and $b//note[f:is-footnote(.)]">
+                    <tr>
+                        <td colspan="2">
+                            <xsl:call-template name="insert-footnotes">
+                                <xsl:with-param name="div" select="$b"/>
+                            </xsl:call-template>
+                        </td>
+                    </tr>
+                </xsl:when>
+
+                <!-- Footnotes on either side -->
+                <xsl:when test="$a//note[f:is-footnote(.)] or $b//note[f:is-footnote(.)]">
+                    <tr>
+                        <td class="first">
+                            <xsl:call-template name="insert-footnotes">
+                                <xsl:with-param name="div" select="$a"/>
+                            </xsl:call-template>
+                        </td>
+                        <td class="second">
+                            <xsl:call-template name="insert-footnotes">
+                                <xsl:with-param name="div" select="$b"/>
+                            </xsl:call-template>
+                        </td>
+                    </tr>
+                </xsl:when>
+            </xsl:choose>
         </xsl:if>
     </xsl:template>
 
diff --git a/style/aligned-text.css b/style/aligned-text.css
index c30ca132..76a93c83 100644
--- a/style/aligned-text.css
+++ b/style/aligned-text.css
@@ -6,6 +6,7 @@
 
 table.alignedText, table.alignedVerse {
     border-collapse: collapse;
+    width: 100%;
 }
 
 table.alignedText td.first, table.alignedText td.second {