Steps to reproduce:
1. Add a link to the menu with as the path.
(I am doing this in a links.menu.yml file and passing <nolink> as the route)
2. Clear cache and see the items with no link are not behaving appropriately.
It seems like the styling is breaking because the <a> is replaced with a <span>.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | adminimal_admin_toolbar_nolink.patch | 953 bytes | IreneV |
| #5 | nolink-menu-item-style-breakdown.png | 22.03 KB | sonfd |
Comments
Comment #2
sonfdComment #3
sonfdComment #4
energee commentedCan you post an image?
Comment #5
sonfdSure, here's one. You can see that "Add Page" is just up in the middle of nowhere.
I think I looked into this further and traced this issue all the way back to one of the core themes. I wish I would've been responsible and updated this issue, but it was a while ago now and can't remember. I think it would be an issue with the default toolbar and is not something specific to the Adminimal toolbar. I think there are just a bunch of styles that target < a >s specifically. The < nolink > text gets wrapped with < span >, so we could probably update the styles to target both < a > and < span >. I meant to look into it again before now, however I will look again sometime if it's not resolved by then.
Comment #6
energee commentedI would accept a patch for this even tho issue stems from dependency, but moving it out of bugs category.
Comment #7
IreneV commentedPatch for issue
Comment #8
IreneV commentedComment #9
artsays commentedIt works! Great, thank you IreneV
Comment #10
artsays commentedComment #12
energee commented