From 5b2f5dd3f488813f201a9e80ca6ccef83437f210 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Sat, 22 Dec 2007 15:52:22 -0600
Subject: [PATCH] Minor correction to template markup and change to script to
 allow for more clickable <label>s.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 admin/auth_subfields_structure.pl                         | 6 +++---
 .../prog/en/modules/admin/auth_subfields_structure.tmpl   | 8 ++++----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/admin/auth_subfields_structure.pl b/admin/auth_subfields_structure.pl
index e59d6b7564..ee6cb39dbb 100755
--- a/admin/auth_subfields_structure.pl
+++ b/admin/auth_subfields_structure.pl
@@ -214,7 +214,7 @@ if ($op eq 'add_form') {
 					-multiple=>0,
 					);
 		$row_data{authorised_value}  = CGI::scrolling_list(-name=>'authorised_value',
-					-id=>'authorised_value',
+					-id=>"authorised_value$i",
 					-values=> \@authorised_values,
 					-default=>$data->{'authorised_value'},
 					-size=>1,
@@ -222,7 +222,7 @@ if ($op eq 'add_form') {
 					-multiple=>0,
 					);
 		$row_data{frameworkcode}  = CGI::scrolling_list(-name=>'frameworkcode',
-					-id=>'frameworkcode',
+					-id=>"frameworkcode$i",
 					-values=> \@authtypes,
 					-default=>$data->{'frameworkcode'},
 					-size=>1,
@@ -230,7 +230,7 @@ if ($op eq 'add_form') {
 					-multiple=>0,
 					);
 		$row_data{value_builder}  = CGI::scrolling_list(-name=>'value_builder',
-					-id=>'value_builder',
+					-id=>"value_builder$i",
 					-values=> \@value_builder,
 					-default=>$data->{'value_builder'},
 					-size=>1,
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tmpl
index 1ff63664dd..34e0b456ea 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_subfields_structure.tmpl
@@ -61,9 +61,9 @@ function displayMoreConstraint(numlayer){
 
         <!-- TMPL_LOOP NAME="loop" -->
         <div id="sub<!-- TMPL_VAR NAME="tagsubfield" -->field">
-			<ol>
-                <input type="hidden" name="tagsubfield" value="<!-- TMPL_VAR name="tagsubfield" -->" />
-                <li><label for="repeatable<!-- TMPL_VAR NAME="row" -->">Repeatable: </label><!-- TMPL_VAR NAME="repeatable" -->&nbsp;</li>
+			<fieldset class="rows"><ol>
+                
+                <li><input type="hidden" name="tagsubfield" value="<!-- TMPL_VAR name="tagsubfield" -->" /><label for="repeatable<!-- TMPL_VAR NAME="row" -->">Repeatable: </label><!-- TMPL_VAR NAME="repeatable" -->&nbsp;</li>
                 <li><label for="mandatory<!-- TMPL_VAR NAME="row" -->">Mandatory: </label><!-- TMPL_VAR NAME="mandatory" -->&nbsp;</li>
                 <li><label for="liblibrarian<!-- TMPL_VAR NAME="row" -->">Text for librarian: </label><input id="liblibrarian<!-- TMPL_VAR NAME="row" -->" type="text" name="liblibrarian" value="<!-- TMPL_VAR NAME="liblibrarian" -->" size="40" maxlength="80" /></li>
                 <li><label for="libopac<!-- TMPL_VAR NAME="row" -->">Text for OPAC: </label><input type="text" id="libopac<!-- TMPL_VAR NAME="row" -->" name="libopac" value="<!-- TMPL_VAR NAME="libopac" -->" size="40" maxlength="80" /></li>
@@ -88,7 +88,7 @@ function displayMoreConstraint(numlayer){
                         </ol>
                     </fieldset>
                 </li>
-            </ol>
+            </ol></fieldset><br class="clear" />
         </div>
         <!-- /TMPL_LOOP -->
         </div>
-- 
2.39.2