As I looked for this issue into the code of the book.module i realized that there are a lot of hardcoded html in it
and this is now what i want to fix.
Beside this the only(!) themeable function in book.module is theme_book_navigation() and has far too much hardcoded

I will release a patch within this day...

greets tobias

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Tobias Maier’s picture

Status: Active » Needs review
FileSize
3.72 KB

and here it is :D

I cleaned up theme_book_navigation()
now it does what it promises it makes just the book navigation

I introduced a new themeable function theme_book_export_html()

there are more unthemed html tags. but i think these are to minor to be themed...

moshe weitzman’s picture

the theme_export function looks good ... to me, it makes sense to keep breadcrumb generation out of theme function. thats semantic, not presentation.

Tobias Maier’s picture

Status: Needs review » Reviewed & tested by the community

has anyone concern about it?
I think it is not that overcomplicated patch so I set it to ready to be commited

Dries’s picture

Status: Reviewed & tested by the community » Needs work

Haven't looked or reviewed this patch but clearly, it no longer applies against HEAD.

Tobias Maier’s picture

Status: Needs work » Reviewed & tested by the community
FileSize
3.71 KB

CVS HEAD...

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD.

Anonymous’s picture

Status: Fixed » Closed (fixed)