Skip to content

[8.x] Update Bootstrap pagination style #36467

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 6 commits into from
Closed

[8.x] Update Bootstrap pagination style #36467

wants to merge 6 commits into from

Conversation

jrd-lewis
Copy link
Contributor

This PR updates Laravel's built-in Bootstrap pagination (bootstrap-4.blade.php) to be styled similarly to the built-in Tailwind pagination (tailwind.blade.php).

bootstrap-pagination

ryangjchandler and others added 6 commits March 4, 2021 08:06
* feature: add new VendorTagPublished event

* fix: only fire event when something was published

* refactor: fire event before erroring in console

* revert: move event dispatch into else arm

* chore: update comment on $paths

* chore: publish to publishable

* chore: add missing comment

* Update VendorPublishCommand.php

* Update VendorTagPublished.php

Co-authored-by: Taylor Otwell <[email protected]>
* feature: add new Stringable::matches method

* tests: new Stringable::matches method

* refactor: use match instead of matchAll

* chore: rename method from matches to test
@driesvints
Copy link
Member

This'll break the design for anyone currently using Bootstrap 3. This most likely would need to go into master.

@jrd-lewis
Copy link
Contributor Author

jrd-lewis commented Mar 4, 2021

This'll break the design for anyone currently using Bootstrap 3. This most likely would need to go into master.

I checked with Paginator::useBootstrapThree(); in the AppServiceProvider and the Bootstrap 3 pagination seems to use default.blade.php

@GrahamCampbell
Copy link
Member

This is still breaking, because people using the bootstrap 4 one will find their app suddenly looks different.

@jrd-lewis jrd-lewis changed the base branch from 8.x to master March 4, 2021 17:19
@driesvints
Copy link
Member

Seems to have conflicts after switching branches. Can you please rebranch anew and send in a new pr?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants