From 1fad7c1af0790a3ad054516a08c19c81f95f8299 Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Date: Tue, 8 Nov 2022 14:48:21 +0000
Subject: [PATCH] Bug 31752: Move hidden input fields

There didn't appear to be any reason to have the hidden form fields here
nested in the ordered list and they were causing undesired styling
issues.

This patch simply moves them to the beggining of the fieldset.

Test plan
1) Confirm the 'HTML message' label now aligns correctly

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
index f3f0f70c59..7fd3e6b674 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
@@ -500,6 +500,8 @@
                                             [% ELSE %]
                                                 <fieldset class="rows mtt">
                                             [% END %]
+                                                <input type="hidden" name="message_transport_type" value="[% letter.message_transport_type | html %]" />
+                                                <input type="hidden" name="lang" value="[% lang | html %]" />
                                                 <ol>
                                                     [% IF ( letter.tt_error ) %]
                                                         <li class="template_toolkit_error">
@@ -514,8 +516,6 @@
                                                         </li>
                                                     [% END %]
                                                     <li>
-                                                        <input type="hidden" name="message_transport_type" value="[% letter.message_transport_type | html %]" />
-                                                        <input type="hidden" name="lang" value="[% lang | html %]" />
                                                         <label for="is_html_[% letter.message_transport_type | html %]_[% lang | html %]">HTML message:</label>
                                                         [% IF letter.is_html %]
                                                             <input type="checkbox" name="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" id="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" value="1" checked="checked" />
-- 
2.39.5