Posted by grendzy on August 26, 2009 at 9:38pm
Jump to:
| Project: | Menu Trails |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Issue Summary
Though menutrails and menu_breadcrumb are similar, both of these modules have some unique strengths. I've found myself trying to combine the benefits of each; unfortunately I haven't found a way to use both modules together. (The menutrails breadcrumbs take precedence if enabled.)
Advantages of menutrails:
Sets crumbs for nodes that aren't in the menu, by content type and taxonomy.
Advantages of menu_breadcrumb:
Several useful display options(append title to crumbs, append as link, hide when only "home" is in the trail).
Seems to have a more efficient method of actually setting the active trail; unlike menutrails it doesn't require a phptemplate_...() override function.
Thanks!
Comments
#1
I'd be happy to merge in some features of that module. Especially, appending the current page's title to the breadcrumb sounds nice.
However, we'd need to turn this issue into a "meta issue" and spin off separate feature requests for the individual changes to separate issues.
Also, I'm not sure whether I'll have time to work on it, but definitely happy to review + commit any patches!
#2
See the corresponding issue on Menu breadcrumb: #560464: Consider merging with menutrails.
Indeed there is overlap between the two modules. The most useful initial step would be to remove the direct overlap to make is possible for the two modules to work together.
The core concern of menutrails is: Enable setting a default parent menu item for given nodes (nodes of a type, nodes with a taxonomy terms, nodes in a given organic group) which, implicitly at least, are not themselves included in the menu hierarchy.
The core concern of menu breadcrumb is: If there is a menu item for the current path, build a nested breadcrumb including the parents of the current path.
When the two are looked at in this way, there's no obvious reason for conflict. That is,
Main problems currently are:
Interestingly, although they achieve a similar result, the two modules take very different approaches to determining the breadcrumbs to set. Menu breadcrumb takes advantage of the fact that Drupal core already supports nested menu-based breadcrumbs (but only for the navigation menu). By setting as active the menu of the current menu item (by calling menu_set_active_menu_name()), drupal_get_breadcrumb() fetches the nested items. Menu trails in contrast constructs a custom breadcrumb based on menu information. Before any deep integration of the two, probably it would be necessary to decide on one or the other of these approaches.
The simplest and probably best short term approach would be:
#3
Created the following offshoot issues: