Bug 21164: Fix alignment on new basket form in acquisitions
authorOwen Leonard <oleonard@myacpl.org>
Mon, 6 Aug 2018 18:15:10 +0000 (18:15 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Tue, 14 Aug 2018 11:45:41 +0000 (11:45 +0000)
This patch removes some unnecessary "&nbsp;"'s from the template for
creating a new basket in acquisitions. This fixes the alignment on the
form fields.

To test, apply the patch and go to Acquisitions -> Vendor -> New basket.
All the form fields should be correctly left-aligned with each other.

Signed-off-by: Pierre-Luc Lapointe <pierreluc.lapointe@inLibro.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketheader.tt

index 240226e..15c64e8 100644 (file)
                     </li>
                 [% END %]
                 <li>
-                    <label for="basketname" class="required">Basket name: </label> &nbsp;
-                        <input type="text" name="basketname" id="basketname" size="40" maxlength="80" value="[% basketname %]" required="required" class="required" />
-                        <span class="required">Required</span>
+                    <label for="basketname" class="required">Basket name: </label>
+                    <input type="text" name="basketname" id="basketname" size="40" maxlength="80" value="[% basketname %]" required="required" class="required" />
+                    <span class="required">Required</span>
                 </li>
-                 <li>
-                    <label for="billingplace">Billing place:</label>
+                <li>
+                    <label for="billingplace">Billing place: </label>
                     <select name="billingplace" id="billingplace">
                         <option value="">--</option>
                         [% PROCESS options_for_libraries libraries => Branches.all( selected => billingplace ) %]
                     </select>
                 </li>
                 <li>
-                    <label for="deliveryplace">Delivery place:</label>
+                    <label for="deliveryplace">Delivery place: </label>
                     <select name="deliveryplace" id="deliveryplace">
                         <option value="">--</option>
                         [% PROCESS options_for_libraries libraries => Branches.all( selected => deliveryplace ) %]
                     </select>
                 </li>
                 <li>
-                    <label for="basketbooksellerid">Vendor: </label> &nbsp;
+                    <label for="basketbooksellerid">Vendor: </label>
                     <select name="basketbooksellerid" id="basketbooksellerid">
                     [% FOREACH b IN booksellers %]
                         [% IF booksellerid == b.id %]
             </select>
                 </li>
                 <li>
-                    <label for="basketnote">Internal note: </label> &nbsp;
+                    <label for="basketnote">Internal note: </label>
                     <textarea name="basketnote" id="basketnote" rows="5" cols="40">[% basketnote |html %]</textarea>
                 </li>
                 <li>
-                    <label for="basketbooksellernote">Vendor note: </label> &nbsp;
+                    <label for="basketbooksellernote">Vendor note: </label>
                     <textarea name="basketbooksellernote" id="basketbooksellernote" rows="5" cols="40">[% basketbooksellernote |html %]</textarea>
                 </li>
                 [% IF ( contractloop ) %]