Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Following the implemented functionality under https://www.drupal.org/node/2891840, i propose to work on it as a backport!
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | interdiff.txt | 1.85 KB | tatarbj |
| #8 | replace_breadcrumb-2895974-8.patch | 4.38 KB | tatarbj |
Comments
Comment #2
renatog commentedHi people.
In attachment the patch with the fix.
Admin:
Title
Good Work and Good Weekend.
Regards.
Comment #3
tatarbjLet me start the review
Comment #4
tatarbjTests are OK, but i've introduced under the D8 version a change because it seems a valid use case to use : character in a title that makes the code broken. So i've changed it to :: that makes no sense to be used in a title and also makes the code working in every cases :)
I attach an interdiff with the patch, please review them and let me know if you agree with it!
Bests,
Balazs.
Comment #5
renatog commentedHi @tatarbj
Really makes sense use :: in this case.
Thank you for replicate fix on 8.x Works good.
+RTBC.
Good Weekend.
Regards.
Comment #6
tatarbjFollowing the changes under d8 issue, i'm backporting it here. patch comes in a few minutes.
Comment #7
tatarbjImplemented changes from d8 version applied in the new patch here.
Review is needed.
Bests,
Balazs.
Comment #8
tatarbjAs we discovered with @gregboggs the new feature doesn't replace all the breadcrumbs, only the title (maybe it was originally planned like this, but why not allowing all the crumbs to be replaced!), i've made a new patch where it's possible. (Interdiff is also attached.)
Please review!
Comment #9
greg boggsComment #11
tatarbjReleased