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.

CommentFileSizeAuthor
#5 mt.patch750 bytesbcn
#4 menutrails-breadcrumbs.patch550 bytesIncrediblyKenzi
#1 patch.patch708 bytesjdelaune
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jdelaune’s picture

Status: Active » Needs review
FileSize
708 bytes

I had a dig around and came up with this.

apostate’s picture

Ran into the same issue using a set of all-expanded primary links. This patch works great.

bcn’s picture

Status: Needs review » Reviewed & tested by the community

I had the same problem, and the patch fixes this...

IncrediblyKenzi’s picture

FileSize
550 bytes

LOL.. had the same issue. basically not honoring the current active trail.

An alternative patch (bit more readable).

bcn’s picture

Status: Reviewed & tested by the community » Needs review
FileSize
750 bytes

A 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.

sun’s picture

Status: Needs review » Reviewed & tested by the community

Last 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.

Jeff Burnz’s picture

Working well for me, nothing ill toward to report.

eojthebrave’s picture

Patch 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.

kmonty’s picture

Patch #5 works for me with no errors.

sun’s picture

Status: Reviewed & tested by the community » Fixed

Thanks 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.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.