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

jhriggs drupal-devel at drupal.org
Wed Apr 6 19:43:18 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:   jhriggs
 Status:       patch

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"?


jhriggs



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

April 6, 2005 - 11: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 - 11:48 : killes at www.drop.org

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

Here's the patch.


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

April 6, 2005 - 13:34 : Dries

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





More information about the drupal-devel mailing list