New template for Bootstrap 5

[Kvaksrud]

No description provided.

[codecov]

Codecov Report

Merging #16 (bba782f) into main (7a4db95) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #16   +/-   ##
=========================================
  Coverage     90.62%   90.62%           
  Complexity       62       62           
=========================================
  Files             9        9           
  Lines           192      192           
=========================================
  Hits            174      174           
  Misses           18       18           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a4db95...55f6e42. Read the comment docs.

[shengslogar]

Thanks for the PR. I'm hesitant to merge this in, since it's identical to the existing Bootstrap 4 template outside of the wrapping <nav> element. Is there something I'm missing here?

laravel-breadcrumbs/resources/views/bootstrap4.blade.php

Lines 1 to 15 in 7a4db95

	@unless ($breadcrumbs->isEmpty())
	<ol class="breadcrumb">
	@foreach ($breadcrumbs as $breadcrumb)
	@if ($breadcrumb->url && !$loop->last)
	<li class="breadcrumb-item"><a href="{{ $breadcrumb->url }}">{{ $breadcrumb->title }}</a></li>
	@else
	<li class="breadcrumb-item active">{{ $breadcrumb->title }}</li>
	@endif
	@endforeach
	</ol>
	@endunless

[Kvaksrud]

Renamed to Bootstrap 5 as it is based on it

[shengslogar]

Thanks. I deleted my last comment because I noticed the Bootstrap 4 spec was actually supposed to have the wrapping <nav> but it didn't make it into our bootstrap4 template and I was thinking about consistency. We have wrapping elements in other templates, though, so this makes sense.

I'll merge this in and tag shortly. Thanks! 🚀

[Kvaksrud]

Thanks! :)

[shengslogar]

Tagged v6.1.0