The main motivation for this is to be able to hide the breadcrumb when it only links to the home page and nothing more (which is kinda pointless depending on your use case). This is the one thing we found lacking while evaluating this module for a current project.

Comments

danbohea created an issue. See original summary.

greg boggs’s picture

If you are getting a link to the home page and only t he home page, then you're most likely not using the easy breadcrumb module yet. The default breadcrumbs do that. Easy breadcrumb fixes that.

~Greg

dddbbb’s picture

Hmmm... thanks for the pointer will check and report back.

dddbbb’s picture

Yep, as initially reported, I'm getting a link to the home page and only the home page. This is happening on pages that are in current main menu, at the top level.

easy_breadcrumb is enabled and configured. All other pages are behaving as desired.

greg boggs’s picture

I see! And you have unchecked show current page crumb? Seems like a reasonable feature to add if you're interested in working on it.

For top landing pages most people would go with the crumb Home | Landing Page

dddbbb’s picture

For top landing pages most people would go with the crumb Home | Landing Page

If they're displaying the title of the current page at he end of the breadcrumb, then yes I agree. But if they're not (many people argue that this is a duplication of the page title which is typically quite near the breadcrumb) then you'd be left with just "Home" (which is pretty pointless).

greg boggs’s picture

Yep. It's a good addition :)

greg boggs’s picture

If anyone is interested in this feature, contributions are welcome!

greg boggs’s picture

Issue tags: +good-first-issue
useernamee’s picture

So if I understand this correctly you would like to have option Include the front page as a segment in the breadcrumb checked, Include the currend pase as a segment in the breadcrumb unchecked and hide breadcrumb when only home is visible. Because if you uncheck Include the front page as a segment you come close to the behaviour you are trying to achieve.

dddbbb’s picture

It's very simple.

Add a checkbox to the admin form: "Hide link to home page if it's the only breadcrumb item".

Logic applied if that checkbox is checked (pseudo-code):

if (new checkbox has been checked) {
  if (user has enabled the link to the home page && total number of items in the breadcrumb == 1) {
    hide the link to the home page
  }
}
useernamee’s picture

I have created a patch for this.

useernamee’s picture

Status: Active » Needs review
greg boggs’s picture

Status: Needs review » Reviewed & tested by the community

looks good!

  • Greg Boggs committed 521ba8c on 8.x-1.x authored by useernamee
    Issue #2908189 by useernamee, Greg Boggs, danbohea: Add option to not...
greg boggs’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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