Bug 31752: Move hidden input fields
authorMartin Renvoize <martin.renvoize@ptfs-europe.com>
Tue, 8 Nov 2022 14:48:21 +0000 (14:48 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Thu, 10 Nov 2022 12:34:35 +0000 (09:34 -0300)
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

index f3f0f70..7fd3e6b 100644 (file)
                                             [% 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">
                                                         </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" />