Skip to content
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

Exposes buildOptionText property of Options component to Pagination c… #196

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mdsturtevant
Copy link

Exposes buildOptionText property of Options component to Pagination component.

pageSizeBuildOptionText allows the user to define a method to overwrite the default buildOptionText method inside the Option component, allowing for custom text. The motivation for this is to put the data length as the last option in pageSizeOptions and then use this custom method to covert the display text to say "All".

…omponent.

pageSizeBuildOptionText allows the user to define a method to overwrite the default buildOptionText method inside the Option component, allowing for custom text. The motivation for this is to put the data length as the last option in pageSizeOptions and then use this custom method to covert the display text to say "All".
@vercel
Copy link

vercel bot commented Aug 30, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://rc-pagination-git-fork-mdsturtevant-master.react-component.now.sh

@coveralls
Copy link

Coverage Status

Coverage remained the same at 89.496% when pulling d638eac on mdsturtevant:master into f33c0f1 on react-component:master.

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.

2 participants