When viewing printer friendly version e.g. http://drupal.org/book/print/336, this produces each of the above headings as h1, when the headings should cascade downward like shown below to properly reflect the hierarchy of nodes. If anyone agrees about this opinion, I will add a feature request for book.module to follow
this behavior.

h1 Section: Drupal handbook
h2 Documentation standards
h3 API documentation standards
h3 Adding code examples
h3 Adding screenshots
h3 Creating lists
h2 How and when do I add a page?
h2 How and when do I link to other pages?
h2 How and when do I update a page?
h2 What is this book about?

CommentFileSizeAuthor
#4 book_printer_friendly_headings.patch1.44 KBmoshe weitzman

Comments

tlindgren’s picture

tlindgren commented
Title: headings in book.module printer friendly version » I agree

I would support you on this feature request. I noticed the same problem recently and I wondered how to make the printer-friendly version more useful, especially with large books.

moshe weitzman’s picture

moshe weitzman commented
Title: I agree » headings in book.module printer friendly version
Kjartan’s picture

Kjartan commented
Category: bug » feature
moshe weitzman’s picture

moshe weitzman commented
StatusFileSize
new book_printer_friendly_headings.patch1.44 KB

the attached brief patch implements this suggestion. prior to this patch, depths were available to be formatted via CSS classes but now we present same titles within h1, h2, etc. which is semantically preferable.

moshe weitzman’s picture

moshe weitzman commented
Assigned: Unassigned » moshe weitzman

oops. patch has been attached here for months. changing Status.

Kjartan’s picture

Kjartan commented

Shoudn't the counter be capped at 6? AFAIK HTML only allows from H1-6.

dries’s picture

dries commented

The patch no longer applies: marking this 'active'.

What would happen when the user uses <hx>-tags in page bodies? This is quite common to do as the user assumes that the page title is using <h1> (which is true in the non-printer friendly case).

moshe weitzman’s picture

moshe weitzman commented
Assigned: moshe weitzman » Unassigned