Breadcrumb markup should allow some tags such as italics or strong

I rebased the merge request so it's mergeable 🎉 Or so I think! -- I'm still newish to merge request rebasing in the context of Drupal issue forks, hopefully I did it correctly 🤞

-------
BUT! I see the test failure, so it seems there's still work to do.

#84 patch introduced #3214648: Error in layout builder: Argument #1 ($title) must be of type string, null given → issue for me.

The last submitted patch, 88: Breadcrumb-markup-double-eascaped-2990464-88.patch, failed testing. View results →

ravi kant → made their first commit to this issue’s fork.

can somebody bring that up to the latest dev version

Rerolled #88

@Jelle_S thanks for your effort would it be possible to have that as an MR ?

Did move the patch +93 into an new mr

And adapted the Test "testGetTitleString" to assert for Drupal\Core\Render\Markup instead of String.

Would be nice to get that in after this long time... Would love to have at least a few RTBCs before merging.

spuky → changed the visibility of the branch 2990464-breadcrumb-markup-should to hidden.

Rerolled #93

@jonasane...

What is wrong with the MR that you rerroll 93 ? Why not download the plain diff of the MR use it as local patch... and give feedback on that...

For everybody still using a patch workflow here is the MR as a patch so we get feedback on whats actually to be merged...

I've tested on 10.2 and it works well, I no longer see `&`.

Note for those trying to apply the patch, confirm you are on the latest tag. The patch would not apply on 2.0.6.

Putting back to needs review... since not applying to 2.0.6 when we are on 2.0.8 is not something that needs work ;-)

There are conflicts in the MR, attaching reroll for latest 2.0.9.