- π¬π§United Kingdom alexpott πͺπΊπ
Over in [#3344200[ I'm proposing that we do not add empty titles to the breadcrumb as this is obviously wrong. That would fix the issue and make the breadcrumbs correct based on the current node.add path. I think the solution here might then be unnecessary. And then we can consider re-scoping this issue to be about moving the node.add path dependent on node_settings::use_admin_theme config setting.
- π¬π§United Kingdom AaronMcHale Edinburgh, Scotland
@alexpott: 100%, let's at least postpone this issue on π PathBasedBreadcrumbBuilder needs to be able to exclude pointless paths Needs work and see what happens.
- Status changed to Postponed
over 2 years ago 10:05pm 24 February 2023 - π¬π§United Kingdom alexpott πͺπΊπ
So if we do the current implementation on [#3344200 you get a breadcrumb of
Home >> Node
- it creates a title of Node based on the path. - π¦πΊAustralia acbramley
With the changes in π PathBasedBreadcrumbBuilder needs to be able to exclude pointless paths Needs work the breadcrumb on /node/add becomes just "Home" and /node/add/foo becomes Home > Add content. IMO this is fine. The discussion about moving node/add/* under admin/content has been had elsewhere but I don't think there's a conclusion yet. As the node subsystem maintainer, I am heavily against moving it for similar reasons mentioned in #58
With that said, IMO this issue could be closed as a duplicate.