Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Shouldn't the breadcrumbs be generated by the active trail only? If you have an expended menu then they all seem to appear in the breadcrumb trail.
Comment | File | Size | Author |
---|---|---|---|
#5 | mt.patch | 750 bytes | bcn |
#4 | menutrails-breadcrumbs.patch | 550 bytes | IncrediblyKenzi |
#1 | patch.patch | 708 bytes | jdelaune |
Comments
Comment #1
jdelaune CreditAttribution: jdelaune commentedI had a dig around and came up with this.
Comment #2
apostate CreditAttribution: apostate commentedRan into the same issue using a set of all-expanded primary links. This patch works great.
Comment #3
bcn CreditAttribution: bcn commentedI had the same problem, and the patch fixes this...
Comment #4
IncrediblyKenzi CreditAttribution: IncrediblyKenzi commentedLOL.. had the same issue. basically not honoring the current active trail.
An alternative patch (bit more readable).
Comment #5
bcn CreditAttribution: bcn commentedA reroll of the patch from #4, which didn't apply after the lastest dev release.
EDIT TO ADD:
Patch is against the 6--1 branch.
Comment #6
sunLast patch looks good and reasonable. Not only more readable, but also better in terms of performance.
Leaving final decision to Josh. Note, however, I did not test this patch.
Comment #7
Jeff Burnz CreditAttribution: Jeff Burnz commentedWorking well for me, nothing ill toward to report.
Comment #8
eojthebravePatch works as promised for me as well. Thanks!
joshk: do you need anything else in order to get this patch applied? It's a real simple fix, and solves a pretty nasty bug.
Comment #9
kmontyPatch #5 works for me with no errors.
Comment #10
sunThanks for reporting, reviewing, and testing! Committed to 6.x.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.