Currently, catalog breadcrumbs just show the name of the view, which is "Product catalog." And when using Bootstrap with the "breadcrumb_title" setting, that view title is duplicated. Incoming patch to clean this up.

CommentFileSizeAuthor
#2 2987671-2.patch4.84 KBmglaman

Comments

mglaman created an issue. See original summary.

mglaman’s picture

Status: Active » Needs review
StatusFileSize
new4.84 KB
mglaman’s picture

+++ b/commerce_demo.module
@@ -61,3 +61,18 @@ function commerce_demo_modules_installed($modules) {
+  // Special check if Bootstrap is enabled and disable the breadcrumb_title
+  // for catalog pages.

We should expand this comment. Need to explain that when on a page with facets, the current page title is already rendered and the setting causes it to display twice

mglaman’s picture

Committing with improved comment:

  // The Bootstrap theme provides a feature that appends the current page title
  // to the end of the breadcrumbs. On the catalog page this is awkward, as
  // Facets already appends the facet source page title when a facet filter
  // is active.
  //
  // This temporarily disables that setting, as module preprocess hooks will run
  // before the theme's, without actually changing the theme setting.

  • mglaman committed 1e56129 on 8.x-1.x
    Issue #2987671 by mglaman: Provide better catalog breadcrumbs
    
mglaman’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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