site map does not display correctly with garland
drpratten - January 6, 2007 - 03:36
| Project: | Site map |
| Version: | HEAD |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
With Garland theme, the top level menu items are displayed out in the left margin.

#1
I just tried with a clean install of the latest Drupal 5 HEAD and Sitemap HEAD and could not see this problem. Please try with clean installs and if the problem persist attach a screenshot of it.
#2
Just tried site_map head with fresh drupal rc2 with same results.
Same result for IE7 and Firefox 2
Problem is duplication of list markers
E.g. if Navigation menu is displayed for user#1 then "Create Content" has a solid list marker triangle and an open garland list marker overlayed.
#3
screen shot
#4
I had the same problem, but have been able to fix it by editing site_map.module.
In the function below, I've changed
to
, this fixed it for me.
best regards,
Andy
function _site_map_menu_tree($pid = 1, $menu = NULL) {
// based on functions theme_menu_tree and menu_tree
if (isset($menu['visible'][$pid]) && $menu['visible'][$pid]['children']) {
foreach ($menu['visible'][$pid]['children'] as $mid) {
$children = isset($menu['visible'][$mid]['children']) ? $menu['visible'][$mid]['children'] : NULL;
$menu_tree .= theme('menu_item', $mid, _site_map_menu_tree($mid, $menu), count($children) == 0);
}
}
if ($menu_tree) {
$output = '
'. $menu_tree .'
';
}
return $output;
}
#5
In post above the tags are not shown.
Change the ul tag to class=menu instead of class=tree
#6
The POST #5 is the most correct. Thank you all.
#7