Attachment 73812 Details for Bug 8643 – Bug 8643: (QA follow-up) Move input field up below mandatory

[patch] Bug 8643: (QA follow-up) Move input field up below mandatory

Bug-8643-QA-follow-up-Move-input-field-up-below-ma.patch (text/plain), 2.28 KB, created by Katrin Fischer on 2018-04-07 08:43:07 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2018-04-07 08:43:07 UTC

Size: 2.28 KB

patch

obsolete

>From 2501745d91bc4b5c461de6808b9afbed83d4955b Mon Sep 17 00:00:00 2001
>From: Katrin Fischer <katrin.fischer.83@web.de>
>Date: Sat, 7 Apr 2018 10:38:22 +0200
>Subject: [PATCH] Bug 8643: (QA follow-up) Move input field up below mandatory
>
>Changes sequence of input fields to be more logical by
>grouping repeatable, mandatory, important, like it's already
>done for the subfield configuration.
>---
> .../intranet-tmpl/prog/en/modules/admin/marctagstructure.tt  | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
>index 353bbbbd96..dd722c2d2b 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
>@@ -79,12 +79,6 @@
>             <input type="checkbox" name="mandatory" id="mandatory" value="1" />
>         [% END %]
>     </li>
>-    <li><label for="authorised_value">Authorized value: </label>
>-        <select name="authorised_value" id="authorised_value" size="1">
>-            <option value=""></option>
>-            [% PROCESS options_for_authorised_value_categories authorised_value_categories => AuthorisedValues.GetCategories( selected => authorised_value ) %]
>-        </select>
>-        (if you select a value here, the indicators will be limited to the authorized value list)</li>
>     <li><label for="important">Important: </label>
>         [% IF ( important ) %]
>             <input type="checkbox" name="important" id="important" value="1" checked="checked" />
>@@ -92,6 +86,12 @@
>             <input type="checkbox" name="important" id="important" value="1" />
>         [% END %]
>     </li>
>+    <li><label for="authorised_value">Authorized value: </label>
>+        <select name="authorised_value" id="authorised_value" size="1">
>+            <option value=""></option>
>+            [% PROCESS options_for_authorised_value_categories authorised_value_categories => AuthorisedValues.GetCategories( selected => authorised_value ) %]
>+        </select>
>+        (if you select a value here, the indicators will be limited to the authorized value list)</li>
> </ol></fieldset> 
>     <fieldset class="action">
>         <input type="submit" value="Save changes" />
>-- 
>2.14.1

Actions: View | Diff | Splinter Review

Attachments on bug 8643: 11621 | 11622 | 30640 | 31032 | 40232 | 40237 | 41259 | 41261 | 41264 | 41265 | 45834 | 45835 | 45836 | 45837 | 60554 | 60555 | 65148 | 65149 | 65682 | 72154 | 72155 | 72156 | 72157 | 73591 | 73808 | 73809 | 73810 | 73811 | 73812 | 73813 | 73814 | 73815 | 73816 | 73817 | 73818 | 74267 | 74268 | 74269 | 74270 | 74271 | 74272 | 74273 | 81032 | 85982 | 91537 | 92081 | 92384 | 96625 | 96627 | 97325 | 97326 | 98607 | 98608 | 98702 | 98703 | 98712