Bug 22170: Improve display of add/edit library group modal
authorKatrin Fischer <katrin.fischer.83@web.de>
Tue, 5 Feb 2019 21:44:58 +0000 (21:44 +0000)
committerLucas Gass <lucas@bywatersolutions.com>
Fri, 8 Mar 2019 17:10:44 +0000 (17:10 +0000)
Tries to bring the emodal for adding/editing library root
and sub groups more in line with the rest of Koha.

To test:
 - Go to Administration > Library groups
 - With and without patch test following actions:
   - Add a new root group
   - Edit root group
   - Add sub-group entry
   - Edit sub-group entry
- Verify that with the patch the display of the labels
  is more standard (bold and aligned to the right) and
  the input fields are a bit longer to allow for
  easier data entry.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Mikaƫl Olangcay Brisebois <mikael.olangcay-brisebois@inLibro.com>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 90b43c6e5fceda3d6bcc4fb24c99934331df1acc)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit ca9f17517df3f97a5ee66e99b5f90f37f74989a5)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>

koha-tmpl/intranet-tmpl/prog/en/modules/admin/library_groups.tt

index 9dc7d39..d7cae65 100644 (file)
                         <h3 id="add-group-modal-label">Add group</h3>
                     </div>
                     <div class="modal-body">
-                        <input type="hidden" id="add-group-modal-parent-id" name="parent_id" value="" />
-                        <input type="hidden" id="add-group-modal-action" name="action" value="add">
-
-                        <p>
-                            <label for="add-group-modal-title">Title: </label>
-                            <input type="text" name="title" id="add-group-modal-title" required="required" />
-                            <i>required</i>
-                        </p>
-
-                        <p>
-                            <label for="add-group-modal-description">Description: </label>
-                            <input type="text" name="description" id="add-group-modal-description" />
-                        </p>
+                        <fieldset class="rows">
+                            <input type="hidden" id="add-group-modal-parent-id" name="parent_id" value="" />
+                            <input type="hidden" id="add-group-modal-action" name="action" value="add">
+                            <ol>
+                                <li>
+                                    <label for="add-group-modal-title">Title: </label>
+                                    <input type="text" size="40" maxlength="100" name="title" id="add-group-modal-title" required="required" />
+                                    <span class="required">Required</span>
+                                </li>
+                                <li>
+                                    <label for="add-group-modal-description">Description: </label>
+                                    <input type="text" size="40" name="description" id="add-group-modal-description" />
+                                </li>
+                            </ol>
+                        </fieldset>
                         <div id="root-group-features-add">
                             <h3>Features</h3>
                             <div class="checkbox">
                     <div class="modal-body">
                         <input type="hidden" id="edit-group-modal-id" name="id" value="" />
                         <input type="hidden" id="edit-group-modal-action" name="action" value="edit" />
-
-                        <p>
-                            <label for="edit-group-modal-title">Title: </label>
-                            <input type="text" id="edit-group-modal-title" name="title" value="" required="required" />
-                            <i>required</i>
-                        </p>
-
-                        <p>
-                            <label for="edit-group-modal-description">Description: </label>
-                            <input type="text" id="edit-group-modal-description" name="description" value="" />
-                        </p>
+                        <fieldset class="rows">
+                            <ol>
+                                <li>
+                                    <label for="edit-group-modal-title">Title: </label>
+                                    <input type="text" size="40" maxlength="100" id="edit-group-modal-title" name="title" value="" required="required" />
+                                    <span class="required">Required</span>
+                                </li>
+
+                                <li>
+                                    <label for="edit-group-modal-description">Description: </label>
+                                    <input type="text" size="40" id="edit-group-modal-description" name="description" value="" />
+                                </li>
+                            </ol>
+                        </fieldset>
                         <div id="root-group-features-edit">
                             <h3>Features</h3>
                             <div class="checkbox">