Fix bug 5434 - hardcoded missing Alt text, with a warning comment. Merged from branch...

diff --git a/theme/formal_white/header.html b/theme/formal_white/header.html
index 75f1f1e902ed37bf8edea5e59084d1250e49456d..e57fee1b58d49edbbb800f6b86d90b188c95be5b 100644 (file)
--- a/theme/formal_white/header.html
+++ b/theme/formal_white/header.html
@@ -17,17 +17,17 @@
       
 <div id="page">
 
-<?php //Accessibility: 'headermain' is now H1, see theme/standard/styles_layout.css: .headermain
+<?php //Accessibility: warning: ALT text will need editing if logo changes. 'headermain' is now H1, see theme/standard/styles_layout.css: .headermain
       if ($home) {  // This is what gets printed on the home page only
 ?>
     <div id="header-home" class="clearfix">
-        <h1 class="headermain"><img src='<?php echo $CFG->wwwroot.'/theme/'.current_theme() ?>/logo.jpg' border='0'/></h1>
+        <h1 class="headermain"><img alt='Moodle' src='<?php echo $CFG->wwwroot.'/theme/'.current_theme() ?>/logo.jpg' /></h1>
         <div class="headermenu"><?php echo $menu ?></div>
     </div>
 <?php } else if ($heading) {  // This is what gets printed on any other page with a heading 
 ?>
     <div id="header" class="clearfix">
-        <h1 class="headermain"><img src='<?php echo $CFG->wwwroot.'/theme/'.current_theme() ?>/logo_small.jpg' border='0'/></h1>
+        <h1 class="headermain"><img alt='Moodle' src='<?php echo $CFG->wwwroot.'/theme/'.current_theme() ?>/logo_small.jpg' /></h1>
         <div class="headermenu"><?php echo $menu ?></div>
     </div>
 <?php } ?>

diff --git a/theme/standardlogo/header.html b/theme/standardlogo/header.html
index c40d200c0ebb6a25b92745eef792df5b01fd488b..2381383034d0d90b5abe14404c733fc4e7a658ce 100644 (file)
--- a/theme/standardlogo/header.html
+++ b/theme/standardlogo/header.html
@@ -37,9 +37,11 @@
       } else {
           $standardlogo = $CFG->wwwroot.'/theme/'.current_theme().'/logo.gif';
       }
+      
+      //Accessibility: warning: ALT text will need editing if logo changes.
 ?>
     <div id="header-home">
-        <div class="headermain"><img src="<?php echo $standardlogo ?>" border="0" alt="" /></div>
+        <div class="headermain"><img alt="Moodle" src="<?php echo $standardlogo ?>" /></div>
         <div class="headermenu"><?php echo $menu ?></div>
     </div>
 <?php } else if ($heading) {  // This is what gets printed on any other page with a heading