Accessibility: fix, replace <i> with <span> (Martin D post 10/3, OU-Bugz: 717)

diff --git a/blocks/search_forums/block_search_forums.php b/blocks/search_forums/block_search_forums.php
index 360c11ed7b9ad6af444e7faee22b21161cc293ef..30a39e1c8fce5b8d48016580c93d1db7022af1f3 100644 (file)
--- a/blocks/search_forums/block_search_forums.php
+++ b/blocks/search_forums/block_search_forums.php
@@ -31,7 +31,7 @@ class block_search_forums extends block_base {
         $this->content->text .= '<form name="search" action="'.$CFG->wwwroot.'/mod/forum/search.php" style="display:inline">';
         $this->content->text .= '<input name="id" type="hidden" value="'.$this->instance->pageid.'" />';  // course
         $this->content->text .= '<input name="search" type="text" size="16" value="" alt="search" />';
-        $this->content->text .= '<button type="submit" title="'.get_string('search').'"><img src="'.$CFG->wwwroot.'/pix/a/em1_bwgreater.gif" alt="" class="resize" /><i class="accesshide">'.get_string('search').'</i></button><br />'; 
+        $this->content->text .= '<button type="submit" title="'.get_string('search').'"><img src="'.$CFG->wwwroot.'/pix/a/em1_bwgreater.gif" alt="" class="resize" /><span class="accesshide">'.get_string('search').'</span></button><br />'; 
         $this->content->text .= '<a href="'.$CFG->wwwroot.'/mod/forum/search.php?id='.$this->instance->pageid.'">'.$advancedsearch.'</a>';
         $this->content->text .= helpbutton('search', $advancedsearch, 'moodle', true, false, '', true);
         $this->content->text .= '</form></div>';

diff --git a/lib/weblib.php b/lib/weblib.php
index efa9d907f9b373b477d03a5e2ae523620987045a..a97ca4b5f0eef537cd810d5156ca97e8397229b0 100644 (file)
--- a/lib/weblib.php
+++ b/lib/weblib.php
@@ -2399,7 +2399,7 @@ function print_navigation ($navigation) {
    if ($navigation) {
        //Accessibility: breadcrumb links now in a list, » replaced with image.
           $nav_text = get_string('youarehere','access');
-          echo '<i class="accesshide">'.$nav_text.':</i><ul>';
+          echo '<span class="accesshide">'.$nav_text.':</span><ul>';
        if (! $site = get_site()) {
            $site->shortname = get_string('home');
        }
@@ -4340,7 +4340,7 @@ function print_side_block($heading='', $content='', $list=NULL, $icons=NULL, $fo
             foreach ($list as $key => $string) {
                 echo '<li class="r'. $row .'">';
                 if ($icons) {
-                    echo '<i class="icon c0">'. $icons[$key] .'</i>';
+                    echo '<span class="icon c0">'. $icons[$key] .'</span>';
                 }
                 echo '<span class="c1">'. $string .'</span>';
                 echo "</li>\n";