From 60470e42e4c8a59e74e42f335343018f49fadbf4 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Tue, 14 Jul 2020 21:30:39 +0000
Subject: [PATCH] Bug 25987: Fix alignment of fields in label batch creation
 form

This patch makes minor markup changes to the template for the label
batch creation process so that form fields line up correctly.

Unrelated change: A hint has been moved out of the label area to below
the textarea in order to conform with established patterns.

To test, apply the patch and go to Tools > Label creator > New > Label
batch.

 - In the batch creation form the "Enter by itemnumber" and "Enter by
   barcode" should be aligned correctly with each other.
 - The "One number per line..." hint should appear below the textarea.
 - Test that the form fields are still correctly aligned after adding
   some items to the batch and the "description" field is shown in the
   form.

Works for me!

Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
---
 .../en/modules/labels/label-edit-batch.tt     | 23 +++++++++++--------
 1 file changed, 14 insertions(+), 9 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/labels/label-edit-batch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/labels/label-edit-batch.tt
index 27282a95b9..24b3de3e4b 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/labels/label-edit-batch.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/labels/label-edit-batch.tt
@@ -62,23 +62,28 @@
                                 <div>
                                     <fieldset class="rows" style="border-bottom: 0px; border: 0px;">
                                     <ol><li id="description">
-                                        <input type="hidden" name="op" value="add" />
-                                        <input type="hidden" name="batch_id" value="[% batch_id | html %]" />
-                                        <label for="description">Batch description: </label>
-                                        <input type="text" name="description" value="[% description | html %]"> <span id="change-status"></span>
-                                    </li>
+                                            <input type="hidden" name="op" value="add" />
+                                            <input type="hidden" name="batch_id" value="[% batch_id | html %]" />
+                                            <label for="description">Batch description: </label>
+                                            <input type="text" name="description" value="[% description | html %]"> <span id="change-status"></span>
+                                        </li>
                                     <li>
                                         <input type="radio" name="number_type" id="barcode_enter" value="barcode" checked />
-                                        <label for="barcode_enter">Enter by barcode:</label><br/>
+                                        <label for="barcode_enter">Enter by barcode:</label>
+                                    </li>
+                                    <li>
                                         <input type="radio" name="number_type" id="itemnum_enter" value="itemnumber" />
                                         <label for="itemnum_enter">Enter by itemnumber:</label>
                                     </li>
                                     <li>
-                                        <label for="number_list">Add by barcode(s) or itemnumbers(s):
-                                            <br /> <span class="hint">One number per line.</span>
-                                            <br /> <span class="hint">Leave empty to add via item search (itemnumber).</span>
+                                        <label for="number_list">
+                                            Add by barcode(s) or itemnumbers(s):
                                         </label>
                                         <textarea rows="5" id="number_list" name="number_list" tabindex="1" class="focus"></textarea>
+                                        <div class="hint">
+                                            One number per line.
+                                            <br /> Leave empty to add via item search (itemnumber)
+                                        </div>
                                     </li></ol>
                                     </fieldset>
                                 </div>
-- 
2.17.1