After creating a book, empty book divs shouldn't be created [#13099]

This code creates a styled section beneath a new book:

<div class="book"><div class="nav"> <div class="links"><div class="prev">&nbsp;</div><div class="next">&nbsp;</div></div> <div class="titles"></div></div></div>  </div>

If there is no data to be displayed, empty divs should not be output.

Comment	File	Size	Author
#3	book.module_8.patch	3.06 KB	crunchywelch
#3
#2	book.module_7.patch	7.89 KB	crunchywelch
#2
	book_divs.png	9.83 KB	chrismessina

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Robin Monks CreditAttribution: Robin Monks commented 16 June 2005 at 18:26

The <div class="book"><div class="nav"> <div class="links"><div class="prev"> </div><div class="next"> </div></div> <div class="titles"></div></div></div></div> still exists. Bumping up to HEAD.

Robin

Comment #2

crunchywelch CreditAttribution: crunchywelch commented 3 September 2005 at 21:28

Status:

Active

» Needs review

File	Size
book.module_7.patch	7.89 KB

Here is a patch that fixes this, it rearranges the theme_book_navigation function logic to only output the navigation book div if there is content to display.