Theme not applicable on all list items
tklawsuc - April 5, 2007 - 16:14
| Project: | Lucid menu |
| Version: | 5.x-1.0 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Jump to:
Description
Not sure if this was intentional but in the lucid_menu_tree function if the item has children, the item is built using:
while items without children are built using the theming option:
<?php
$output .= theme('lucid_menu_item', $pid);
?>So if you try to override theme_lucid_menu_item the changes will only be applied to items that have no children. I made the change (to use theme_lucid_menu_item($pid); for parent items) in the module for my work (no sure how to create a patch...still new to drupal). So if this is intentional then ignore this bug...otherwise spare future users hours of frustration trying to override the item function and change accordingly.

#1
Sorry not sure what happened but the original post should say...the item with children is built using:
<?phpif (!$skip_parent)
$output .= theme_lucid_menu_item($pid);
?>