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

Comments

jdelaune’s picture

jdelaune commented
Status: Active » Needs review
StatusFileSize
new patch.patch708 bytes

I had a dig around and came up with this.

apostate’s picture

apostate commented

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

bcn’s picture

bcn commented
Status: Needs review » Reviewed & tested by the community

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

IncrediblyKenzi’s picture

IncrediblyKenzi commented
StatusFileSize
new menutrails-breadcrumbs.patch550 bytes

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

An alternative patch (bit more readable).

bcn’s picture

bcn commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new mt.patch750 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

sun
Primary language German
Location Karlsruhe
commented
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

Jeff Burnz commented

Working well for me, nothing ill toward to report.

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented

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

kmonty
He/Him
Location Berlin, DE + SF, CA
commented

Patch #5 works for me with no errors.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
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.