Bug 10790: fix misalignment in item type form when noItemTypeImages is off
authorBernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Thu, 20 Mar 2014 01:25:07 +0000 (22:25 -0300)
committerGalen Charlton <gmc@esilibrary.com>
Mon, 7 Apr 2014 15:51:02 +0000 (15:51 +0000)
This patch fixes the odd behavior when noItemTypesImages is set
to 'Don't show'. Just a misplaced </div>

To test:

1) Set syspref noItemTypeImages to 'Don't show'
2) Go to Admininstration > Item types
3) Edit some item type, you will view a display
similar to the attached image
4) Apply the patch
5) Reload the page, now the display is fixed
6) Change syspref to 'Show' to check that
the display is also good

Also fixed a missing '/' on a closing select

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt

index 45d64c8..745dc6b 100644 (file)
@@ -158,8 +158,8 @@ Item types administration
 </ul>
   <br class="clear" />
 </div>
 </ul>
   <br class="clear" />
 </div>
-[% END %]
 </div>
 </div>
+[% END %]
 <ol>
       <li>
           <label for="notforloan">Not for loan: </label>   [% IF ( notforloan ) %]
 <ol>
       <li>
           <label for="notforloan">Not for loan: </label>   [% IF ( notforloan ) %]
@@ -191,7 +191,7 @@ Item types administration
               [% ELSE %]
                   <option value="alert">Alert</option>
               [% END %]
               [% ELSE %]
                   <option value="alert">Alert</option>
               [% END %]
-          <select>
+          </select>
       </li>
       <li>
           <label for="summary">Summary: </label>
       </li>
       <li>
           <label for="summary">Summary: </label>