Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Since D6 the menu tree i no more using class=tree. Please add this back... now it's "class=menu" and this causes some CSS inheritance and by this way troubles... :-(
Comment | File | Size | Author |
---|---|---|---|
#11 | site_map_add_theme_functions.patch | 3.39 KB | hass |
#9 | site_map_kill_css_ul-menu_class.patch | 2.68 KB | hass |
Comments
Comment #1
frjo CreditAttribution: frjo commentedPlease clarify, or better yet make a patch :-).
Comment #2
hass CreditAttribution: hass commentedIn D5 the site_map module have created the sitemap tree with
<ul class="tree">
. Now in D6 it is generated with "class=menu" and this produce some troubles with normal Navigation. If Navigation looks *very* different to the site map tree, it is difficult to style this tree with CSS. I could write ~50 class overwrites or better we add the "tree" class back.See http://www.yaml-fuer-drupal.de/sitemap as an example. The left Navigation is very different from the site map tree. With the current site_map D6 version the sitemap tree looks like the left Navigation menu, but should look like the old version.
Comment #3
poehnix CreditAttribution: poehnix commented@ hass
I am playing with the sitemap module and looking and I have seen the sitemap at http://www.yaml-fuer-drupal.de/sitemap
There's a chance to know how were you able to show all the sitemap menus open (showing all their branches) even if they are closed in the menu?
To be more clear, your navigation menu is:
Template (closed, no children shown)
Tutorial (closed, no children shown)
while your sitemap looks like:
# Template Einführung
* Qualität
* Entwicklung
* Webserver
.....
# Tutorials
* Drupal Bugs
* Modul - Node Style
* Modul - Panels
......
How is this possible if the expanded property is set to 0 ?
Thanks
poehnix
Comment #4
frjo CreditAttribution: frjo commentedIt's function theme_menu_tree() that sets this class and it's called from menu_tree_output(), the function site_map use to build the tree.
You can change the class via the theme function but that would change it for all menus.
I believe the best way is to add your own css that change only the site map page. Something like:
.site-map ul.menu {
[whateverneedstobedone]
}
.site-map ul.menu li {
[whateverneedstobedone]
}
Comment #5
frjo CreditAttribution: frjo commentedComment #6
hass CreditAttribution: hass commentedHave you been able to add the UL class?
Comment #7
frjo CreditAttribution: frjo commentedI still think my suggestion in#4 is the way to go.
I can not find any default styling in Drupal for the class "tree" so I guess what you really want is to replace class "menu" for menues with something else. For that I would need to stop using the Drupal function menu_tree_output() to build the menu tree and use a site map specific function instead. That will most likely not happen.
Comment #8
hass CreditAttribution: hass commented#4 will not work as the styles are all inherited from the standard menu's and a sitemap tree looks *very* typically totally not like a site menu. In D5 this worked this way very well... and we haven't had any inheritance issues in D5.
Comment #9
hass CreditAttribution: hass commentedWould you so kind to take a look to this patch, please? I do not like to maintain this myself and I'm pretty sure others will have the same CSS inheritance issues.
Comment #10
hass CreditAttribution: hass commentedI'm going to re-role the patch as #187631: Patch to add list item theming is nearly the same request, but also requests to theme
theme_menu_item
.Comment #11
hass CreditAttribution: hass commentedNew patch with clean theme implementation.
Comment #12
frjo CreditAttribution: frjo commentedThanks! Committed to the new 6--2 branch. All the new functionality will be put there.