From: nfreear <nfreear>
Date: Mon, 22 May 2006 10:00:18 +0000 (+0000)
Subject: Ongoing, fix bug 5400, navbar/breadcrumb TABLE to DIV, to improve semantics. Merged... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=e99e2d126d3dbcfaa4603c3e2d75868be4588a95;p=moodle.git

Ongoing, fix bug 5400, navbar/breadcrumb TABLE to DIV, to improve semantics. Merged from stable branch.
---

diff --git a/theme/standardlogo/header.html b/theme/standardlogo/header.html
index 6d94d83653..06311a5c9f 100644
--- a/theme/standardlogo/header.html
+++ b/theme/standardlogo/header.html
@@ -38,29 +38,28 @@
           $standardlogo = $CFG->wwwroot.'/theme/'.current_theme().'/logo.gif';
       }
       
-      //Accessibility: warning: ALT text will need editing if logo changes.
+      //Accessibility: warning: ALT text will need editing if logo changes. 'headermain' is now H1.
 ?>
-    <div id="header-home">
+    <div id="header-home" class="clearfix">
         <h1 class="headermain"><img alt="Moodle" src="<?php echo $standardlogo ?>" /></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 id="header" class="clearfix">
         <h1 class="headermain"><?php echo $heading ?></h1>
         <div class="headermenu"><?php echo $menu ?></div>
     </div>
 <?php } ?>
-    <div class="clearer">&nbsp;</div>
-<?php if ($navigation) { // This is the navigation table with breadcrumbs  ?>
-    <table class="navbar"><tr><td>
+<?php //Accessibility: breadcrumb trail/navbar now a DIV, not a table.
+      if ($navigation) { // This is the navigation table with breadcrumbs  ?>
+    <div class="navbar clearfix">
         <div class="breadcrumb"><?php print_navigation($navigation); ?></div>
         <div class="navbutton"><?php echo $button; ?></div>
-    </td></tr></table>
+    </div>
 <?php } else if ($heading) { // If no navigation, but a heading, then print a line  
 ?>
         <hr size="1" noshade="noshade" />
 <?php } ?>
-    <div class="clearer">&nbsp;</div>
     <!-- END OF HEADER -->
     <div id="content">