Hey @vasi,

Thanks for this great module!

As this module adds the current title to breadcrumbs, when use it in a website which has a bootstrap-based theme with option "Show current page title at end" checked, the current title is duplicated.

Maybe we should add an option to let user chose whether wants or not to add the current title.

Comments

tregismoreira created an issue. See original summary.

tregismoreira’s picture

I've tried to code a patch for this, adding a new field which is depending on `breadcrumbs` config, but couldn't get this configuration outside the Class. Could you help me? :)

Here's the initial patch.

mvc’s picture

Version: 8.x-1.0-rc1 » 8.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new3.7 KB

This picks up on tregismoreira's work and makes the config option he added change the display.

tregismoreira’s picture

Status: Needs review » Reviewed & tested by the community

Great work, @mvc. It works like a charm! Thanks :D

  • jigarius committed 4884afc on 8.x-1.x authored by mvc
    Issue #2802047 by tregismoreira, mvc: Make current page title optional...
jigarius’s picture

Status: Reviewed & tested by the community » Fixed

This fix is now in the dev and will be included in the next release.

jigarius’s picture

Status: Fixed » Closed (fixed)