Fix bug 5434 - hardcoded missing Alt text, with a warning comment.
This commit is contained in:
@@ -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 } ?>
|
||||
|
||||
@@ -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
|
||||
|
||||
Reference in New Issue
Block a user