From: nfreear <nfreear>
Date: Fri, 19 May 2006 10:33:13 +0000 (+0000)
Subject: Fix bug 5434 - hardcoded missing Alt text, with a warning comment. Merged from branch... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=1211e732448bbfb78835a43209d44245508310ea;p=moodle.git

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

diff --git a/theme/formal_white/header.html b/theme/formal_white/header.html
index 75f1f1e902..e57fee1b58 100644
--- 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 c40d200c0e..2381383034 100644
--- 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