Permalink
Browse files

MDL-33198 book: Adding h tags to book titles to increase accessibilit…

…y while printing a book
  • Loading branch information...
1 parent c8ac480 commit 6f51394b9415eaac2e9916ca759eb091947eaa34 @ankitagarwal ankitagarwal committed Sep 10, 2012
Showing with 10 additions and 9 deletions.
  1. +9 −4 mod/book/tool/print/index.php
  2. +1 −5 mod/book/tool/print/locallib.php
@@ -91,18 +91,19 @@
</head>
<body>
<a name="top"></a>
+ <h1 class="book_title"><?php echo format_string($book->name, true, array('context'=>$context)) ?></h1>
<div class="chapter">
<?php
if (!$book->customtitles) {
if (!$chapter->subchapter) {
$currtitle = book_get_chapter_title($chapter->id, $chapters, $book, $context);
- echo '<p class="book_chapter_title">'.$currtitle.'</p>';
+ echo '<h2 class="book_chapter_title">'.$currtitle.'</h2>';
} else {
$currtitle = book_get_chapter_title($chapters[$chapter->id]->parent, $chapters, $book, $context);
$currsubtitle = book_get_chapter_title($chapter->id, $chapters, $book, $context);
- echo '<p class="book_chapter_title">'.$currtitle.'<br />'.$currsubtitle.'</p>';
+ echo '<h2 class="book_chapter_title">'.$currtitle.'</h2><h3 class="book_chapter_title">'.$currsubtitle.'</h3>';
}
}
@@ -128,7 +129,7 @@
</head>
<body>
<a name="top"></a>
- <p class="book_title"><?php echo format_string($book->name, true, array('context'=>$context)) ?></p>
+ <h1 class="book_title"><?php echo format_string($book->name, true, array('context'=>$context)) ?></h1>
<p class="book_summary"><?php echo format_text($book->intro, $book->introformat, array('noclean'=>true, 'context'=>$context)) ?></p>
<div class="book_info"><table>
<tr>
@@ -162,7 +163,11 @@
}
echo '<div class="book_chapter"><a name="ch'.$ch->id.'"></a>';
if (!$book->customtitles) {
- echo '<p class="book_chapter_title">'.$titles[$ch->id].'</p>';
+ if (!$chapter->subchapter) {
+ echo '<h2 class="book_chapter_title">'.$titles[$ch->id].'</h2>';
+ } else {
+ echo '<h3 class="book_chapter_title">'.$titles[$ch->id].'</h3>';
+ }
}
$content = str_replace($link1, '#ch', $chapter->content);
$content = str_replace($link2, '#top', $content);
@@ -60,11 +60,7 @@ function booktool_print_get_toc($chapters, $book, $cm) {
$toc .= html_writer::tag('a', '', array('name' => 'toc')); // Representation of toc (HTML).
- if ($book->customtitles) {
- $toc .= html_writer::tag('h1', get_string('toc', 'mod_book'));
- } else {
- $toc .= html_writer::tag('p', get_string('toc', 'mod_book'), array('class' => 'book_chapter_title'));
- }
+ $toc .= html_writer::tag('h2', get_string('toc', 'mod_book'), array('class' => 'book_chapter_title'));
$toc .= html_writer::start_tag('ul');
foreach ($chapters as $ch) {
if (!$ch->hidden) {

0 comments on commit 6f51394

Please sign in to comment.