Skip to content
Browse files

MDL-5400 "Accessibility modifications..." - fixing the 4 themes that …

…don't use <h1>. Merged.
  • Loading branch information...
1 parent 2162373 commit 343a1ea8e0cbb1ffc181893e64192e9052f394aa nfreear committed Aug 20, 2007
Showing with 11 additions and 9 deletions.
  1. +5 −3 theme/cornflower/header.html
  2. +2 −2 theme/metal/header.html
  3. +2 −2 theme/orangewhite/header.html
  4. +2 −2 theme/orangewhitepda/header.html
View
8 theme/cornflower/header.html
@@ -17,15 +17,17 @@
<div class="cornflowersurround">
<div id="page">
-<?php if ($home) { // home page ?>
+<?php //Accessibility: warning: ALT text will need editing when logo changes. 'headermain' is now H1.
+
+ if ($home) { // home page ?>
<div id="header-home">
- <div class="headermain"><img alt="" src="<?php echo $CFG->wwwroot.'/theme/'.current_theme() ?>/images/logo.jpg" width="457" height="64" /></div>
+ <h1 class="headermain"><img alt="[ REPLACE ME ]" src="<?php echo $CFG->wwwroot.'/theme/'.current_theme() ?>/images/logo.jpg" width="457" height="64" /></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-home">
- <div class="headermain"><img alt="" src="<?php echo $CFG->wwwroot.'/theme/'.current_theme() ?>/images/logo.jpg" width="457" height="64" /></div>
+ <h1 class="headermain"><img alt="[ REPLACE ME ]" src="<?php echo $CFG->wwwroot.'/theme/'.current_theme() ?>/images/logo.jpg" width="457" height="64" /></h1>
<div class="headermenu"><?php echo $menu ?></div>
</div>
<div class="clearer"></div>
View
4 theme/metal/header.html
@@ -20,13 +20,13 @@
<?php if ($home) { // This is what gets printed on the home page only
?>
<div id="header-home">
- <div class="headermain"><?php echo $heading ?></div>
+ <h1 class="headermain"><?php echo $heading ?></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">
- <div class="headermain"><?php echo $heading ?></div>
+ <h1 class="headermain"><?php echo $heading ?></h1>
<div class="headermenu"><?php echo $menu ?></div>
</div>
<?php } ?>
View
4 theme/orangewhite/header.html
@@ -20,15 +20,15 @@
<?php if ($home) { // This is what gets printed on the home page only
?>
<div id="header-home">
- <div class="headermain"><?php echo $heading ?></div>
+ <h1 class="headermain"><?php echo $heading ?></h1>
<div class="headermenu"><?php echo $menu ?></div>
</div>
<div class="navbar">
</div>
<?php } else if ($heading) { // This is what gets printed on any other page with a heading
?>
<div id="header">
- <div class="headermain"><?php echo $heading ?></div>
+ <h1 class="headermain"><?php echo $heading ?></h1>
<div class="headermenu"><?php echo $menu ?></div>
<!-- <div class="headermenu"><div id="new-menu"><?php echo $navmenulist ?></div></div> -->
</div>
View
4 theme/orangewhitepda/header.html
@@ -20,15 +20,15 @@
<?php if ($home) { // This is what gets printed on the home page only
?>
<div id="header-home">
- <div class="headermain"><?php echo $heading ?></div>
+ <h1 class="headermain"><?php echo $heading ?></h1>
<div class="headermenu"><?php echo $menu ?></div>
</div>
<div class="navbar">
</div>
<?php } else if ($heading) { // This is what gets printed on any other page with a heading
?>
<div id="header">
- <div class="headermain"><?php echo $heading ?></div>
+ <h1 class="headermain"><?php echo $heading ?></h1>
<div class="headermenu"><?php echo $menu ?></div>
<!-- <div class="headermenu"><div id="new-menu"><?php echo $navmenulist ?></div></div> -->
</div>

0 comments on commit 343a1ea

Please sign in to comment.
Something went wrong with that request. Please try again.