Framework: Illuminate\Pagination\AbstractPaginator default view is set to bootstrap instead of default

[sebastiansulinski]

Documentation states that by default default.blade.php is loaded as default pagination (https://laravel.com/docs/5.6/pagination#customizing-the-pagination-view), however Illuminate\Pagination\AbstractPaginator has both properties (for simple and length aware) set to:

public static $defaultView = 'pagination::bootstrap-4';

public static $defaultSimpleView = 'pagination::simple-bootstrap-4';

[wilburpowery]

Just noticed the same issue. I have to set the default view in my AppServiceProvider manually. We either need to update the docs, or set the default view to default.blade.php.

[devcircus]

It really doesn't make sense to have a "default" file but not use it by default. This needs to be fixed. If no one else does, I'll take a look in a day or two.

[wilburpowery]

I'll submit a PR suggesting to change the default views now.

[devcircus]

Taylor made this change himself right around the 5.6 release. You may want to take into consideration his intentions, when making this pr.

[devcircus]

Here's his change. Looks like he just wanted bootstrap to be the default. Should be able to somehow do the same using default.

https://github.com/laravel/framework/commit/12d789de8472dbbd763cb680e896b3d419f954c0

Guess fixing this is technically a breaking change.

[wilburpowery]

Seems like. I'll let him close the PR. We should think about updating the documentation. It took me a couple of minutes to figure out what the problem was. 🤔

[sebastiansulinski]

I think he might have just wanted to test it - I wouldn't think setting default pagination to anything other than default view would be intentional.