[drupal-devel] [bug] Missing <div> container on HTML output

stefan nagtegaal drupal-devel at drupal.org
Thu Apr 7 12:23:34 UTC 2005


Issue status update for http://drupal.org/node/20079

 Project:      Drupal
 Version:      cvs
 Component:    book.module
 Category:     bug reports
 Priority:     minor
 Assigned to:  Anonymous
 Reported by:  budda
 Updated by:   stefan nagtegaal
 Status:       patch

Why don't we remove the menu-<div>??
We can style it using .block-user etc, right?
Stefan.


stefan nagtegaal



Previous comments:
------------------------------------------------------------------------

April 6, 2005 - 19:46 : budda

Line 588 should be made to output the tree navigation within a 
container the same as the core navigation block - to keep consistent.
new line 588:
    return '<div class="menu"><ul>'. $tree .'</ul></div>';

You are still able to format the book tree navigation different to the
core navigation block as this tree is a child of css class
'book-block'.


------------------------------------------------------------------------

April 6, 2005 - 19:48 : killes at www.drop.org

Attachment: http://drupal.org/files/issues/book_28.patch (558 bytes)

Here's the patch.


------------------------------------------------------------------------

April 6, 2005 - 21:34 : Dries

Where does Drupal core emit a class="menu" ?


------------------------------------------------------------------------

April 6, 2005 - 21:43 : jhriggs

Line 89 of menu.module in menu_block():
  $data['content'] = '<div class="menu">'. theme('menu_tree', $delta)
.'</div>' ;

Should the change maybe be made to theme_menu_tree() instead to take
care of all of them at once?  And if so, should the class be called
"tree" rather than "menu"?


------------------------------------------------------------------------

April 7, 2005 - 11:26 : budda

Class "tree" would be good. But would this change break any existing
themes?
Also - the tree menu is used within the book pages at the footer to
show nested pages - so would you want that to be within the same class?





More information about the drupal-devel mailing list