Some changes for XHTML and accessibility compliance

diff --git a/course/lib.php b/course/lib.php
index 8ceea5b608a846fbb58df8fb6e8de0b90ef1c5b8..d25bfb167aec621ec94a6118aa820813e0626825 100644 (file)
--- a/course/lib.php
+++ b/course/lib.php
@@ -1261,11 +1261,11 @@ function print_course($course, $width="100%") {
     echo "<table width=\"100%\">";
     echo '<tr valign="top">';
     echo '<td valign="top" width="50%" class="courseboxinfo">';
-    echo '<p><font size="3"><b><a title="'.get_string('entercourse').'"'.
+    echo '<b><a title="'.get_string('entercourse').'"'.
          $linkcss.' href="'.$CFG->wwwroot.'/course/view.php?id='.$course->id.'">'.
-         $course->fullname.'</a></b></font></p>';
+         $course->fullname.'</a></b><br />';
     if ($teachers = get_course_teachers($course->id)) {
-        echo "<p><font size=\"1\">\n";
+        echo "<span class=\"courseboxteachers\">\n";
         foreach ($teachers as $teacher) {
             if ($teacher->authority > 0) {
                 if (!$teacher->role) {
@@ -1276,7 +1276,7 @@ function print_course($course, $width="100%") {
                      '&amp;course='.SITEID.'">'.$fullname.'</a><br />';
             }
         }
-        echo "</font></p>";
+        echo "</span>\n";
     }
 
     echo $enrol->get_access_icons($course);
@@ -1284,6 +1284,7 @@ function print_course($course, $width="100%") {
     echo '</td><td valign="top" width="50%" class="courseboxsummary">';
     $options = NULL;
     $options->noclean = true;
+    $options->para = false;
     echo format_text($course->summary, FORMAT_MOODLE, $options,  $course->id);
     echo "</td></tr>";
     echo "</table>";

diff --git a/mod/glossary/filter.php b/mod/glossary/filter.php
index 5ded39bd71e49b2da238848ec3e1528763dd7f94..f3900a9172b7e84aa4832028aad5d6caf2877c9c 100644 (file)
--- a/mod/glossary/filter.php
+++ b/mod/glossary/filter.php
@@ -78,7 +78,7 @@
                         $encodedconcept = urlencode($concept->concept);
                         $title = str_replace('"', "'", strip_tags("$glossary->name: $concept->concept"));
                         $href_tag_begin = "<a target=\"entry\" class=\"autolink\" title=\"$title\" href=\"$CFG->wwwroot/mod/glossary/showentry.php?courseid=$courseid&amp;concept=$encodedconcept\" ".
-                             "onClick=\"return openpopup('/mod/glossary/showentry.php?courseid=$courseid\&amp;concept=$encodedconcept', 'entry', 'menubar=0,location=0,scrollbars,resizable,width=600,height=450', 0);\">";
+                             "onclick=\"return openpopup('/mod/glossary/showentry.php?courseid=$courseid\&amp;concept=$encodedconcept', 'entry', 'menubar=0,location=0,scrollbars,resizable,width=600,height=450', 0);\">";
                     }
                     $currentconcept = $concept->concept;
                     if ( $currentconcept = trim(strip_tags($currentconcept)) ) {