Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
menu system
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
15 Jul 2006 at 11:11 UTC
Updated:
12 Aug 2014 at 17:30 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
LAsan commentedFeature request, moving to cvs.
Comment #2
beginner commentedI just posted about this on the dev list.
Comment #3
beginner commentedsee :
http://lists.drupal.org/pipermail/development/2008-June/030337.html
and
#150854: fix breadcrumbs arbitration bug
Comment #4
pwolanin commentedWell, the timing problem certainly is a problematic bug - seems like a simple solution might be (even for D6) to take several alternate BC arrays with each one getting a weight.
something like this:
Comment #5
pwolanin commentedhere's a patch that might also be suitable for 6.x as a bug fix.
This is a minimal change that just allows a developer to be sure that their BC will have priority over one being set (for example) by a core module.
Comment #6
beginner commentedI agree it's a bug. See my follow up on the other issue: #150854: fix breadcrumbs arbitration bug
I prefer the patch there, because it paves the way to a much for powerful hook_breadcrumbs() with $op, $path, $args, etc.
With that new hook_breadcrumbs(), themes would be able to get an array of breadcrumbs and display more than one set of breadcrumbs, which is the current feature request.
Comment #7
pwolanin commentedComment #8
moshe weitzman commentedThis can't be a bug IMO. The system is working as designed. Adding a second breadcrumb is a new feature. Seems clear cut to me.
Comment #9
pwolanin commented@moshe - changing the title to reflect what the patch does.
Basically - if the breadcrumb is set in the page callback, I think there is pretty much no way to override it unless you hack the theme layer. That's what I'm identifying as the bug. The change proposed should be a backward-compatible and opens the window by adding the weight parameter (we could call it $priority or something else if you like).
Comment #10
beginner commentedpwolanin: you are hijacking this issue. See the other issue for the bug you are trying to fix.
I mentioned your patch in the other issue.
Having several breadcrumbs is a bona fide feature request.
Comment #11
pwolanin commented@beginner - sorry - I missed that link - didn't mean to hijack.
Comment #12
mdupontComment #13
kika commentedWas proposing it back at 2002. +1
Comment #14
sunBetter title.
FWIW, I once started the http://drupal.org/project/breadcrumb module with a seemingly identical goal; multiple breadcrumbs, and breadcrumbs as blocks. Given D8, I think this ties deeply into Blocks and Layouts.
Comment #15
Crell commentedBreadcrumbs are now blocks. This is done. :-)