Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

feat(pagination): Show ellipsis when rotating #3035

Closed
wants to merge 1 commit into from
Closed

feat(pagination): Show ellipsis when rotating #3035

wants to merge 1 commit into from

Conversation

distinctdan
Copy link

When we have more pages than can be shown and we're rotating, shows ellipsis' like:

Previous, ..., 3, 4, 5, 6, 7, ..., Next

This also adds a handy function in the tests to convert the pager to a string, so that we can intuitively check the state of the pager all at once, instead of having to check each element's text individually.

@distinctdan
Copy link
Author

This is for the feature request: #2924

@wesleycho
Copy link
Contributor

This appears to be a breaking change - this should be an opt-in feature.

@pkozlowski-opensource
Copy link
Member

Closing as this PR wasn't touched for 6 months and the [comment[(https://github.com//pull/3035#issuecomment-85867895) wasn't addressed. It seems like we've got another PR for the same: https://github.com//pull/3565

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

Successfully merging this pull request may close these issues.

5 participants