Breadcrumbs: code "if loop last" removes delimiter in pure bootstrap about forkcms

[Sellato]

@alessandrocraeye Great thank you :) your code works fine!

One comment :) this was not a mine code, this is Breadcrumb.html.twig so it's need to be fixed in next release of Fork CMS.

PS: In ForkCMS delimiter hard coded in css file.

[alessandrocraeye]

I believe this is purely because your if statement is implemented wrong.

The class breadcrumb-item is what is needed by bootstrap to show the breadcrumb styling and delimiter.
By implementing it like you mentioned: <li {% if loop.last %} class="breadcrumb-item active" {% endif %} the needed breadcrumb-item class is only being added to the last item in the list.

It should actually be implemented like this: <li class="breadcrumb-item{% if loop.last %} active{% endif %}"> so the breadcrumb-item class gets added to every item. The active class is only needed on the last item.

[crydotsnake]

Hello 👋🏼

I created a pull request, that should fixes the issue!

Merry Christmas btw! 🎄

[Sellato]

I created a pull request, that should fixes the issue!

Great!, Thank you!

Merry Christmas