--- rat/client/parameter.html	2023/12/22 13:38:03	1.91
+++ rat/client/parameter.html	2025/06/28 13:55:43	1.95
@@ -5,7 +5,7 @@
 The LearningOnline Network with CAPA
 Parameter Input Window
 //
-// $Id: parameter.html,v 1.91 2023/12/22 13:38:03 raeburn Exp $
+// $Id: parameter.html,v 1.95 2025/06/28 13:55:43 raeburn Exp $
 //
 // Copyright Michigan State University Board of Trustees
 //
@@ -642,27 +642,25 @@ function addIpRule(iptype) {
     var frame = window.frames["choices"];
     if (frame.document.getElementById('LC_string_ipacc_inner_'+iptype)) {
         var innerDiv = frame.document.getElementById('LC_string_ipacc_inner_'+iptype);
-        var count = innerDiv.childNodes.length 
         var ipDiv = frame.document.createElement('div');
-        ipDiv.innerHTML = '<input type="text" size="10" name="setip'+iptype+'" onblur="parent.ipstringeval();" />'+
-                          '<a href="#" onclick="parent.removeIpRule(\''+iptype+'\',\''+count+'\')">Remove</a>';
-        frame.document.getElementById('LC_string_ipacc_inner_'+iptype).appendChild(ipDiv);
+        ipDiv.innerHTML = '<input type="text" size="12" name="setip'+iptype+'" onblur="parent.ipstringeval();" />'+"\n"+
+                          '<a href="#" onclick="parent.removeIpRule(this,\''+iptype+'\');return false;">Remove</a>';
+        innerDiv.appendChild(ipDiv);
     }
-    return false;
+    return;
 }
 
-function removeIpRule(iptype,num) {
+function removeIpRule(caller,iptype) {
     var frame = window.frames["choices"];
     if (frame.document.getElementById('LC_string_ipacc_inner_'+iptype)) {
         var innerDiv = frame.document.getElementById('LC_string_ipacc_inner_'+iptype);
-        for (var i=0; i<innerDiv.childNodes.length; i++) {
-            if (i==num) {
-                innerDiv.removeChild(innerDiv.childNodes[i]);
-                ipstringeval();
-                break;
-            }
+        var divToRemove = caller.closest('div');
+        if (divToRemove) {
+            innerDiv.removeChild(divToRemove);
+            ipstringeval();
         }
     }
+    return;
 }
 
 function radiostringeval(newval) {
@@ -1182,20 +1180,20 @@ function draw() {
             }
             var curripaccess = [currallow,currdeny];
             tablestart('IP Number/Name');
-            choicewrite('<tr><th>Allow from</th><th>Deny from</th></tr><tr>');
+            choicewrite('<tr><th>Allow from</th><th>Deny from</th></tr><tr>'+"\n");
             var acctypes = ['allow','deny'];
             for (var i=0; i<acctypes.length; i++) {
-                choicewrite('<td valign="top">'+
-                            '<div class="LC_string_ipacc_wrap" id="LC_string_ipacc_'+acctypes[i]+'">'+
-                            '<div class="LC_string_ipacc_inner" id="LC_string_ipacc_inner_'+acctypes[i]+'">');
+                choicewrite(['<td valign="top">',
+                             '<div>',
+                             '<div id="LC_string_ipacc_inner_'+acctypes[i]+'">'].join("\n"));
                 for (var j=0; j<curripaccess[i].length; j++) {
-                    choicewrite('<div><input type="text" size="10" name="setip'+acctypes[i]+'" value="'+curripaccess[i][j]+'" onblur="parent.ipstringeval();" />');
-                    choicewrite('<a href="#" onclick="parent.removeIpRule(\''+acctypes[i]+'\',\''+j+'\')">Remove</a>');
-                    choicewrite('</div>');
+                    choicewrite(['<div><input type="text" size="12" name="setip'+acctypes[i]+'" value="'+curripaccess[i][j]+'" onblur="parent.ipstringeval();" />',
+                                 '<a href="#" onclick="parent.removeIpRule(this,\''+acctypes[i]+'\');return false;">Remove</a>',
+                                 '</div>'].join("\n"));
                 }
-                choicewrite('</div><button onclick="parent.addIpRule(\''+acctypes[i]+'\');">Add item</button>');
+                choicewrite('</div><button onclick="parent.addIpRule(\''+acctypes[i]+'\');return false;">Add item</button>'+"\n");
             }
-            choicewrite('</div></td></tr></table>');
+            choicewrite('</div></td></tr></table>'+"\n");
         }
         if (pscat=='fileext') {
             tablestart('Allowed File Extensions');
@@ -1847,7 +1845,7 @@ function init() {
      else if (pscat == 'fileext')      { sopt('fileext','File Extension'); }
      else if (pscat == 'useslots')     { sopt('useslots','Slots control access'); }
      else if (pscat == 'deeplink')     { sopt('deeplink','Deep-linked items'); }
-     else if (pscat == 'tex')   { sopt('texdisplay','TeX File Display'); }
+     else if (pscat == 'tex')          { sopt('texdisplay','TeX File Display'); }
      else { pscat = 'any'; }
      if (pscat != 'deeplink') { sopt('any','String Value'); }
   }