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

frjo - January 6, 2007 - 18:28

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

drpratten - January 14, 2007 - 03:09

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

drpratten - January 14, 2007 - 03:10

screen shot

AttachmentSize
temp.JPG 7.5 KB

#4

afey - April 16, 2007 - 09:34

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

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 = '

';
}

return $output;
}

#5

afey - April 16, 2007 - 09:36

In post above the tags are not shown.
Change the ul tag to class=menu instead of class=tree

#6

fonsiusgs - March 23, 2008 - 17:55

The POST #5 is the most correct. Thank you all.

#7

frjo - September 11, 2009 - 07:21
Status:active» closed
 
 

Drupal is a registered trademark of Dries Buytaert.