Fixing markup: <select ..> was <select ../>, multiple="multiple" and onfocus (was...

diff --git a/admin/roles/assign.html b/admin/roles/assign.html
index 668123d30b3af2587c6409765ed49c1fb3cac9f5..2ea0a35717c4d37fc3de05e116a89e94f36d31b2 100755 (executable)
--- a/admin/roles/assign.html
+++ b/admin/roles/assign.html
@@ -18,10 +18,10 @@
     </tr>
     <tr>
       <td valign="top">
-          <select name="removeselect[]" size="20" id="removeselect" multiple
-                  onFocus="document.assignform.add.disabled=true;
+          <select name="removeselect[]" size="20" id="removeselect" multiple="multiple"
+                  onfocus="document.assignform.add.disabled=true;
                            document.assignform.remove.disabled=false;
-                           document.assignform.addselect.selectedIndex=-1;" />
+                           document.assignform.addselect.selectedIndex=-1;">
           <?php
               foreach ($contextusers as $contextuser) {
                   $fullname = fullname($contextuser, true);
@@ -45,8 +45,10 @@
         <br />
       </td>
       <td valign="top">
-          <select name="addselect[]" size="20" id="addselect" multiple
-                  onFocus="document.assignform.add.disabled=false;
+     <label for="addselect"><?php print_string('potentialusers', 'role', $usercount); ?></label>
+     <br />
+          <select name="addselect[]" size="20" id="addselect" multiple="multiple"
+                  onfocus="document.assignform.add.disabled=false;
                            document.assignform.remove.disabled=true;
                            document.assignform.removeselect.selectedIndex=-1;">
           <?php
@@ -82,7 +84,7 @@
          </select>
          <br />
          <input type="text" name="searchtext" size="30" value="<?php p($searchtext, true) ?>"
-                  onFocus ="document.assignform.add.disabled=true;
+                  onfocus ="document.assignform.add.disabled=true;
                             document.assignform.remove.disabled=true;
                             document.assignform.removeselect.selectedIndex=-1;
                             document.assignform.addselect.selectedIndex=-1;"