Closed (fixed)
Project:
Easy Breadcrumb
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
12 Sep 2017 at 07:46 UTC
Updated:
25 Apr 2018 at 15:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
greg boggsIf 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
Comment #3
dddbbb commentedHmmm... thanks for the pointer will check and report back.
Comment #4
dddbbb commentedYep, 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.
Comment #5
greg boggsI 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
Comment #6
dddbbb commentedIf 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).
Comment #7
greg boggsYep. It's a good addition :)
Comment #8
greg boggsIf anyone is interested in this feature, contributions are welcome!
Comment #9
greg boggsComment #10
useernamee commentedSo 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.
Comment #11
dddbbb commentedIt'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):
Comment #12
useernamee commentedI have created a patch for this.
Comment #13
useernamee commentedComment #14
greg boggslooks good!
Comment #16
greg boggs