I want to use in custom breadcrumb replacement. But it only replace with current page title when I select `Use the real page title when available` option.
I only want to add page title for custom breadcrumbs. When we add in the custom breadcrumb settings I think we should replace with title without any condition check.

CommentFileSizeAuthor
#2 easy_breadcrumb_3338946_2.patch971 bytespmkanse

Issue fork easy_breadcrumb-3338946

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

pmkanse created an issue. See original summary.

pmkanse’s picture

pmkanse commented
StatusFileSize
new easy_breadcrumb_3338946_2.patch971 bytes

Patch

greg boggs’s picture

greg boggs
he/him
Primary language English
Location Portland Oregon
commented

That makes sense, can we make this an MR?

rpayanm made their first commit to this issue’s fork.

rpayanm opened merge request !76

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
Status: Active » Needs review

I added an MR, please review.

greg boggs’s picture

greg boggs
he/him
Primary language English
Location Portland Oregon
commented

code looks good, we need to test it with the various options and see if there are any unexpected side affects.

diego_mow’s picture

diego_mow commented
Status: Needs review » Reviewed & tested by the community

Using MR from #3 and it is working fine with several options.

greg boggs’s picture

greg boggs
he/him
Primary language English
Location Portland Oregon
commented
Status: Reviewed & tested by the community » Needs work

Needs a rebase against latest dev

rollins’s picture

rollins commented

@greg-boggs do we still need fixed MR here?

spuky’s picture

spuky commented
Related issues: +#3441176: Replaced breadcrumb title is reset when an alternative title field name is configured

should get fixed by

spuky’s picture

spuky commented
Status: Needs work » Closed (outdated)

should be fixed in dev now

spuky closed merge request !76