accessiblity, associating <th> with respective row or col

group/groupui/main-form.html

@@ -1,9 +1,9 @@
 <form name="groupeditform" id="groupeditform" action="">
     <table cellpadding="10" class="generaltable generalbox groupmanagementtable">
         <tr>
-            <th class="header groupmanagementtableheader"><?php print_string('groupings', 'group'); ?></th>
-            <th class="header groupmanagementtableheader"><?php print_string('groupsinselectedgrouping', 'group'); ?></th>
-            <th class="header groupmanagementtableheader"><?php print_string('membersofselectedgroup', 'group'); ?></th>
+            <th class="header groupmanagementtableheader" scope="col"><?php print_string('groupings', 'group'); ?></th>
+            <th class="header groupmanagementtableheader" scope="col"><?php print_string('groupsinselectedgrouping', 'group'); ?></th>
+            <th class="header groupmanagementtableheader" scope="col"><?php print_string('membersofselectedgroup', 'group'); ?></th>
         </tr>
         <tr>
             <td class="generalboxcontent">

lang/en_utf8/help/wiki/wikitype.html

@@ -6,25 +6,25 @@
 This leads to the following matrix of nine possibilities:
 <table border="1" width="100%"><tbody>
   <tr>
-    <th valign="top" width="25%"></th>
-    <th valign="top" width="25%">No Groups</th>
-    <th valign="top" width="25%">Separate Groups</th>
-    <th valign="top" width="25%">Visible Groups</th>
+    <th valign="top" width="25%" scope="col"></th>
+    <th valign="top" width="25%" scope="col">No Groups</th>
+    <th valign="top" width="25%" scope="col">Separate Groups</th>
+    <th valign="top" width="25%" scope="col">Visible Groups</th>
   </tr>
   <tr>
-    <th valign="top" width="25%">Teacher</th>
+    <th valign="top" width="25%" scope="row">Teacher</th>
     <td valign="top" width="25%">There is only one wiki which only the teacher can edit. Students can view the contents.</td>
     <td valign="top" width="25%">There is one wiki for every group which just the teacher can edit. Students can view the wiki of their group only.</td>
     <td valign="top" width="25%">There is one wiki for every group which just the teacher can edit. Students can view the wikis for all groups.</td>
   </tr>
   <tr>
-    <th valign="top" width="25%">Groups</th>
+    <th valign="top" width="25%" scope="row">Groups</th>
     <td valign="top" width="25%">There is only one wiki. The teacher and all students can view and edit this wiki.</td>
     <td valign="top" width="25%">There is one wiki per group. Students can view and edit the wiki of their own group only.</td>
     <td valign="top" width="25%">There is one wiki per group. Students can change the wiki of their own group only. They can view the wikis for all groups.</td>
   </tr>
   <tr>
-    <th valign="top" width="25%">Student</th>
+    <th valign="top" width="25%" scope="row">Student</th>
     <td valign="top" width="25%">Every student has their own wiki which only they and their teacher can view and edit.</td>
     <td valign="top" width="25%">Every student has their own wiki, which only they and their teacher can edit. Students can view the wikis of other students in their group.</td>
     <td valign="top" width="25%">Every student has their own wiki, which only they and their teacher can edit. Students can view the wikis of all other students in the course.</td>

lib/moodlelib.php

@@ -6254,10 +6254,10 @@ function unzip_show_status ($list,$removepath) {
         $strmodified = get_string("modified");
         $strstatus = get_string("status");
         echo "<table cellpadding=\"4\" cellspacing=\"2\" border=\"0\" width=\"640\">";
-        echo "<tr><th class=\"header\" align=\"left\">$strname</th>";
-        echo "<th class=\"header\" align=\"right\">$strsize</th>";
-        echo "<th class=\"header\" align=\"right\">$strmodified</th>";
-        echo "<th class=\"header\" align=\"right\">$strstatus</th></tr>";
+        echo "<tr><th class=\"header\" align=\"left\" scope=\"col\">$strname</th>";
+        echo "<th class=\"header\" align=\"right\" scope=\"col\">$strsize</th>";
+        echo "<th class=\"header\" align=\"right\" scope=\"col\">$strmodified</th>";
+        echo "<th class=\"header\" align=\"right\" scope=\"col\">$strstatus</th></tr>";
         foreach ($list as $item) {
             echo "<tr>";
             $item['filename'] = str_replace(cleardoubleslashes($removepath).'/', "", $item['filename']);

login/index_form.html

@@ -1,10 +1,10 @@
 <table width="90%" border="0" cellpadding="5" align="center" class="loginbox">
 <?php if ($show_instructions) { ?>
   <tr>
-    <th width="50%" class="header left">
+    <th width="50%" class="header left" scope="col">
       <?php  print_string("returningtosite") ?>
     </th>
-    <th width="50%" class="header right">
+    <th width="50%" class="header right" scope="col">
       <?php  print_string("firsttime") ?>
     </th>
   </tr>

mod/choice/lib.php

@@ -367,9 +367,9 @@ function choice_show_results($choice, $course, $cm, $forcepublish='') {
             $count = 0;
             foreach ($useranswer as $optionid => $userlist) {
                 if ($optionid) {
-                    echo "<th class=\"col$count header\" width=\"$tablewidth%\">";
+                    echo "<th class=\"col$count header\" width=\"$tablewidth%\" scope=\"col\">";
                 } else if ($choice->showunanswered) {
-                    echo "<th class=\"col$count header\" width=\"$tablewidth%\">";
+                    echo "<th class=\"col$count header\" width=\"$tablewidth%\" scope=\"col\">";
                 } else {
                     continue;
                 }
@@ -472,9 +472,9 @@ function choice_show_results($choice, $course, $cm, $forcepublish='') {
             $count = 0;
             foreach ($useranswer as $optionid => $userlist) {
                 if ($optionid) {
-                    echo "<th width=\"$tablewidth%\" class=\"col$count header\">";
+                    echo "<th width=\"$tablewidth%\" class=\"col$count header\" scope=\"col\">";
                 } else if ($choice->showunanswered) {
-                    echo "<th width=\"$tablewidth%\" class=\"col$count header\">";
+                    echo "<th width=\"$tablewidth%\" class=\"col$count header\" scope=\"col\">";
                 } else {
                     continue;
                 }

mod/data/report.php

@@ -49,9 +49,9 @@
     } else {
         echo "<table border=\"0\" cellpadding=\"3\" cellspacing=\"3\" class=\"generalbox\" width=\"100%\">";
         echo "<tr>";
-        echo "<th>&nbsp;</th>";
-        echo "<th><a href=\"report.php?id=$id&amp;sort=firstname\">$strname</a>";
-        echo "<th width=\"100%\"><a href=\"report.php?id=$id&amp;sort=rating\">$strrating</a>";
+        echo "<th scope=\"col\">&nbsp;</th>";
+        echo "<th scope=\"col\"><a href=\"report.php?id=$id&amp;sort=firstname\">$strname</a>";
+        echo "<th width=\"100%\" scope=\"col\"><a href=\"report.php?id=$id&amp;sort=rating\">$strrating</a>";
         foreach ($ratings as $rating) {
             if (has_capability('mod/data:manageentries', $context)) {
                 echo '<tr class="forumpostheadertopic">';

mod/forum/report.php

@@ -55,10 +55,10 @@
     } else {
         echo "<table border=\"0\" cellpadding=\"3\" cellspacing=\"3\" class=\"generalbox\" width=\"100%\">";
         echo "<tr>";
-        echo "<th>&nbsp;</th>";
-        echo "<th><a href=\"report.php?id=$post->id&amp;sort=u.firstname\">$strname</a>";
-        echo "<th width=\"100%\"><a href=\"report.php?id=$post->id&amp;sort=r.rating\">$strrating</a>";
-        echo "<th><a href=\"report.php?id=$post->id&amp;sort=r.time\">$strtime</a>";
+        echo "<th scope=\"col\">&nbsp;</th>";
+        echo "<th scope=\"col\"><a href=\"report.php?id=$post->id&amp;sort=u.firstname\">$strname</a>";
+        echo "<th width=\"100%\" scope=\"col\"><a href=\"report.php?id=$post->id&amp;sort=r.rating\">$strrating</a>";
+        echo "<th scope=\"col\"><a href=\"report.php?id=$post->id&amp;sort=r.time\">$strtime</a>";
         foreach ($ratings as $rating) {
             echo '<tr class="forumpostheader">';
             echo "<td>";

mod/glossary/report.php

@@ -49,10 +49,10 @@
     } else {
         echo "<table border=\"0\" cellpadding=\"3\" cellspacing=\"3\" class=\"generalbox\" width=\"100%\">";
         echo "<tr>";
-        echo "<th class=\"header\">&nbsp;</th>";
-        echo "<th class=\"header\"><a href=\"report.php?id=$entry->id&amp;sort=firstname\">$strname</a></th>";
-        echo "<th width=\"100%\" class=\"header\"><a href=\"report.php?id=$entry->id&amp;sort=rating\">$strrating</a></th>";
-        echo "<th class=\"header\"><a href=\"report.php?id=$entry->id&amp;sort=time\">$strtime</a></th>";
+        echo "<th class=\"header\" scope=\"col\">&nbsp;</th>";
+        echo "<th class=\"header\" scope=\"col\"><a href=\"report.php?id=$entry->id&amp;sort=firstname\">$strname</a></th>";
+        echo "<th width=\"100%\" class=\"header\" scope=\"col\"><a href=\"report.php?id=$entry->id&amp;sort=rating\">$strrating</a></th>";
+        echo "<th class=\"header\" scope=\"col\"><a href=\"report.php?id=$entry->id&amp;sort=time\">$strtime</a></th>";
         foreach ($ratings as $rating) {
             if (has_capability('mod/glossary:manageentries', $context)) {
                 echo '<tr class="teacher">';

mod/lesson/action/continue.html

@@ -118,7 +118,7 @@
         <td>
             <table align="right" width="150px" class="generaltable generalbox" cellspacing="0" cellpadding="5px" border="0" valign="top">
                 <tr>
-                    <th class="header"><?php print_string("timeremaining", "lesson") ?></th>
+                    <th class="header" scope="col"><?php print_string("timeremaining", "lesson") ?></th>
                 </tr>
                 <tr class="r0">
                     <td align="center" class="c0">

mod/lesson/edit.php

@@ -43,7 +43,7 @@
     if (empty($firstpage)) {
         // There are no pages; give teacher some options
         if (has_capability('mod/lesson:edit', $context)) {
-            print_simple_box( "<table cellpadding=\"5\" border=\"0\">\n<tr><th>".get_string("whatdofirst", "lesson")."</th></tr><tr><td>".
+            print_simple_box( "<table cellpadding=\"5\" border=\"0\">\n<tr><th scope=\"col\">".get_string("whatdofirst", "lesson")."</th></tr><tr><td>".
                 "<a href=\"import.php?id=$cm->id&amp;pageid=0\">".
                 get_string("importquestions", "lesson")."</a></td></tr><tr><td>".
                 "<a href=\"importppt.php?id=$cm->id&amp;pageid=0\">".
@@ -113,7 +113,7 @@
                     $page = $pages[$pageid];
 
                     echo "<tr><td>\n";
-                    echo "<table width=\"100%\" border=\"1\" class=\"generalbox\"><tr><th colspan=\"2\">".format_string($page->title)."&nbsp;&nbsp;\n";
+                    echo "<table width=\"100%\" border=\"1\" class=\"generalbox\"><tr><th colspan=\"2\" scope=\"col\">".format_string($page->title)."&nbsp;&nbsp;\n";
                     lesson_print_page_actions($cm->id, $page, $npages);
                     echo "</th></tr>\n";             
                     echo "<tr><td colspan=\"2\">\n";

mod/lesson/viewend.html

@@ -25,7 +25,7 @@
 
             <table align="right" width="150px" class="generaltable generalbox" cellspacing="0" cellpadding="5px" border="0" valign="top">
                 <tr>
-                    <th class="header"><?php print_string("timeremaining", "lesson") ?></th>
+                    <th class="header" scope="col"><?php print_string("timeremaining", "lesson") ?></th>
                 </tr>
                 <tr class="r0">
                     <td align="center" class="c0">

mod/quiz/config.html

@@ -109,11 +109,11 @@
 <table cellpadding="9" cellspacing="0" align="center">
 
 <tr valign="top">
-  <th align="right">&nbsp;</th>
-  <th align="left">
+  <th align="right" scope="col">&nbsp;</th>
+  <th align="left" scope="col">
     <?php print_string('sitedefault'); ?>
   </th>
-  <th align="center">
+  <th align="center" scope="col">
     <?php print_string('advancedsettings'); helpbutton("advancedsettings"); ?>
   </th>
 </tr>

mod/quiz/jstimer.php

@@ -28,7 +28,7 @@
     <td class="generalboxcontent" bgcolor="#ffffff" width="100%">
     <table class="generaltable" border="0" width="150" cellspacing="0" cellpadding="0">
     <tr>
-        <th class="generaltableheader" width="100%"><?php print_string("timeleft","quiz");?></th>
+        <th class="generaltableheader" width="100%" scope=\"col\"><?php print_string("timeleft","quiz");?></th>
     </tr>
     <tr>
         <td id="QuizTimer" class="generaltablecell" align="center" width="100%">

mod/resource/type/directory/resource.class.php

@@ -99,9 +99,9 @@ function display() {
     $strfile = get_string("file");
 
     echo '<table cellpadding="4" cellspacing="1" class="files">';
-    echo "<tr><th class=\"header name\">$strname</th>".
-         "<th align=\"right\" colspan=\"2\" class=\"header size\">$strsize</th>".
-         "<th align=\"right\" class=\"header date\">$strmodified</th>".
+    echo "<tr><th class=\"header name\" scope=\"col\">$strname</th>".
+         "<th align=\"right\" colspan=\"2\" class=\"header size\" scope=\"col\">$strsize</th>".
+         "<th align=\"right\" class=\"header date\" scope=\"col\">$strmodified</th>".
          "</tr>";
     foreach ($files as $file) {
         if (is_dir("$CFG->dataroot/$relativepath/$file")) {          // Must be a directory

question/editlib.php

@@ -330,15 +330,15 @@ function question_list($course, $categoryid, $quizid=0,
     print_simple_box_start('center', '100%', '#ffffff', 0);
     echo '<table id="categoryquestions" cellspacing="0"><tr>';
     $actionwidth = $canedit ? 95 : 70;
-    echo "<th width=\"$actionwidth\" nowrap=\"nowrap\" class=\"header\">$straction</th>";
+    echo "<th width=\"$actionwidth\" nowrap=\"nowrap\" class=\"header\" scope=\"col\">$straction</th>";
 
     $sortoptions = array('name, qtype ASC' => get_string("sortalpha", "quiz"),
                          'qtype, name ASC' => get_string("sorttypealpha", "quiz"),
                          'id ASC' => get_string("sortage", "quiz"));
     $orderselect  = choose_from_menu ($sortoptions, 'sortorder', $sortorder, false, 'this.form.submit();', '0', true);
     $orderselect .= '<noscript><input type="submit" value="'.get_string("sortsubmit", "quiz").'" /></noscript>';
-    echo "<th width=\"100%\" align=\"left\" nowrap=\"nowrap\" class=\"header\">$strquestionname $orderselect</th>
-    <th nowrap=\"nowrap\" class=\"header\">$strtype</th>";
+    echo "<th width=\"100%\" align=\"left\" nowrap=\"nowrap\" class=\"header\" scope=\"col\">$strquestionname $orderselect</th>
+    <th nowrap=\"nowrap\" class=\"header\" scope=\"col\">$strtype</th>";
     echo "</tr>\n";
     foreach ($questions as $question) {
         echo "<tr>\n<td nowrap=\"nowrap\">\n";

question/type/questiontype.php

@@ -1046,9 +1046,9 @@ function print_replacement_options($question, $course, $cmid='0') {
             echo "<td align=\"left\">\n";
             echo "<table cellpadding=\"5\" align=\"left\" class=\"generalbox\" width=\"100%\">\n";
             echo "<tr>\n";
-            echo "<th align=\"left\" valign=\"top\" nowrap=\"nowrap\" class=\"generaltableheader c0\">$strquizname</th>\n";
-            echo "<th align=\"center\" valign=\"top\" nowrap=\"nowrap\" class=\"generaltableheader c0\">$strdoreplace</th>\n";
-            echo "<th align=\"left\" valign=\"top\" nowrap=\"nowrap\" class=\"generaltableheader c0\">$straffectedstudents</th>\n";
+            echo "<th align=\"left\" valign=\"top\" nowrap=\"nowrap\" class=\"generaltableheader c0\" scope=\"col\">$strquizname</th>\n";
+            echo "<th align=\"center\" valign=\"top\" nowrap=\"nowrap\" class=\"generaltableheader c0\" scope=\"col\">$strdoreplace</th>\n";
+            echo "<th align=\"left\" valign=\"top\" nowrap=\"nowrap\" class=\"generaltableheader c0\" scope=\"col\">$straffectedstudents</th>\n";
             echo "</tr>\n";
             foreach($quizzes as $quiz) {
                 // work out whethere it should be checked by default

theme/index.php

@@ -59,7 +59,7 @@
     $sesskey = !empty($USER->id) ? $USER->sesskey : '';
 
     echo "<table align=\"center\" cellpadding=\"7\" cellspacing=\"5\">";
-    echo "<tr class=\"generaltableheader\"><th>$strtheme</th><th>$strinfo</th></tr>";
+    echo "<tr class=\"generaltableheader\"><th scope=\"col\">$strtheme</th><th scope=\"col\">$strinfo</th></tr>";
     foreach ($themes as $theme) {
 
         unset($THEME);