Panopoly Core has a feature to add the current page title to the end of all breadcrumb pages.

However, on taxonomy term pages, it confusingly puts the current page title before the parent page title:

screenshot showing the problem

Comments

dsnopek created an issue. See original summary.

dsnopek’s picture

Status: Active » Needs review
StatusFileSize
new1.61 KB

The problems seems to involve Panelizer, because the breadcrumbs appear in the correct order if the taxonomy term page isn't panelized. So, somehow, panopoly_core's breadcrumb magic in hook_init() is running before Panelizer is adding the parent term to the breadcrumbs, and showing them out of order.

Here's a patch that moves the breadcrumb magic from hook_init() to hook_preprocess_page() which will happen right before the breadcrumbs are rendered, and hopefully be able to put the page title as the last item in the breadcrumb, without any other modules putting other stuff after it.

dsnopek’s picture

StatusFileSize
new2.36 KB

Here's a new patch that makes extra sure that it's the last preprocess hook to run, to try and prevent issues if other modules are manipulating breadcrumbs in hook_preprocess_page().

cboyden’s picture

Updated the above patch to apply to panopoly_core instead of the fully-built distribution.

cboyden’s picture

Status: Needs review » Reviewed & tested by the community

We've been using this for a while and it's working as expected.

  • dsnopek committed 8dadad5 on 7.x-1.x
    Issue #3194807 by dsnopek, cboyden: Current page title on taxonomy term...
dsnopek’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for testing! Committed :-)

Status: Fixed » Closed (fixed)

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