From a643098a26fcd82b352d8fa49ac22c1803daa448 Mon Sep 17 00:00:00 2001
From: Ankush Menat <ankush@frappe.io>
Date: Wed, 25 Oct 2023 15:47:49 +0530
Subject: [PATCH] fix: Footer should show up once at end if not repeating
 header/footer (#22902)

* fix: Footer should show up once at end if not repeating header/footer

* fix: only show page number if repeating header

[skip ci]
---
 frappe/templates/print_formats/standard.html | 26 ++++++++++----------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/frappe/templates/print_formats/standard.html b/frappe/templates/print_formats/standard.html
index f5ce1d3f5a..642e74415e 100644
--- a/frappe/templates/print_formats/standard.html
+++ b/frappe/templates/print_formats/standard.html
@@ -6,19 +6,6 @@
 		{{ add_header(loop.index, layout|len, doc, letter_head, no_letterhead, footer, print_settings, print_heading_template) }}
 	</div>
 
-	{% if print_settings.repeat_header_footer %}
-	<div id="footer-html" class="visible-pdf">
-		{% if not no_letterhead and footer %}
-		<div class="letter-head-footer">
-			{{ footer }}
-		</div>
-		{% endif %}
-		<p class="text-center small page-number visible-pdf">
-			{{ _("Page {0} of {1}").format('<span class="page"></span>', '<span class="topage"></span>') }}
-		</p>
-	</div>
-	{% endif %}
-
 	{% for section in page %}
 	<div class="row section-break" data-label="{{ section.label or '' | e }}">
 		{%- if doc.print_line_breaks and loop.index != 1 -%}<hr>{%- endif -%}
@@ -35,5 +22,18 @@
 		{% endfor %}
 	</div>
 	{% endfor %}
+
+	<div {% if print_settings.repeat_header_footer %} id="footer-html" class="visible-pdf" {% endif %}>
+		{% if not no_letterhead and footer %}
+		<div class="letter-head-footer">
+			{{ footer }}
+		</div>
+		{% endif %}
+		{% if print_settings.repeat_header_footer %}
+			<p class="text-center small page-number visible-pdf">
+				{{ _("Page {0} of {1}").format('<span class="page"></span>', '<span class="topage"></span>') }}
+			</p>
+		{% endif %}
+	</div>
 </div>
 {% endfor %}