-
Notifications
You must be signed in to change notification settings - Fork 848
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
Hide redundant Page Size buttons #648
Conversation
… count and less than the next button
… count and less than the next button
Sorry got a typo in my first commit. Either way TravisCI passed but CicleCI didn't? I followed the contribution guidelines. Not sure what else to do. Admin please let me know. |
There some overlap between this pull request and #656 Lots of good ideas. However, hiding page size buttons is probably not suitable in every situation. As explained in the the other pull request, we might need to introduce a new option:
Also, what I think is also important is not to load up on options as this can make it harder to work with the library. Alternative solution:
|
There's a pull request to implement infinite scroll (#560) As commented:
|
@christianacca I like your ideas you posted on #656 did you want to assign anyone to start doing those PR's? I am willing to do some work to join these PR's if you are willing to give me some support to get them done to be merged. |
Cool. When do you want to start on this? The first thing is to do the consolidation work:
I'll can do a quick peer review and merge. If you still got time, we can get onto the enhancements. If you stop there, your contribution will still be of benefit to the project. C |
Ok I will look into the refactor for the pagination settings. Should I refactor for the current settings to start and then do additional PR's when adding features or should I include some of those settings right off the get go. |
Let's keep the refactoring into a separate PR |
I just submitted a separate PR. #674 |
Yup, just merging it with some stuff I've just pushed. Also, I'm got a couple of small feedback items around the commit |
Add logic to show only page size if the row count is greater than the count and less than the next count button.