[Request] Allow for breadcrumb items without links

[dwightwatson]

For some items in a breadcrumb, there isn't actually a page for the hierarchy it represents, but we'd like to display it anyway just as text. At the moment, passing nothing to the second parameter in the push() method just generates an empty link.

I think it would be more fitting if passing no second parameter to the method means the crumb is displayed as simply text.

[d13r]

Sounds like a good idea to me. I'll work on it when I get a chance, or if you'd like to make a PR please do.

In the mean time you could work around it by creating your own template (see the readme for instructions) which checks if the URL is blank. You'll still have to pass the blank/null second parameter though for now.

@if ($breadcrumbs)
    <ul class="breadcrumb">
        @foreach ($breadcrumbs as $breadcrumb)
            @if ($breadcrumb->last)
                <li class="active">{{{ $breadcrumb->title }}}</li>
            @elseif ($breadcrumb->url)
                <li><a href="{{{ $breadcrumb->url }}}">{{{ $breadcrumb->title }}}</a></li>
            @else
                <li>{{{ $breadcrumb->title }}}</li>
            @endif
        @endforeach
    </ul>
@endif

(Edited: Changed from Bootstrap 2 to Bootstrap 3 format)

[d13r]

Turns out I was wrong, the second parameter to push() is already optional - so it's just the templates that need updating.

[d13r]

Released in 2.1.0.

[dwightwatson]

Ah too quick! Thanks so much for this, great package!