Skip to content

Conversation

@dtaylor113
Copy link
Member

@dtaylor113 dtaylor113 commented Jul 7, 2017

By default pagination controls are not displayed in the pfTableView. Unfortunately, items where still being paginated; as a result only the first 10 items would be displayed and with no pagination controls, there was no way to get to the rest of the items.

This PR sets 'pagination=false' in the default dt options, and thus shows all items.

Fixes #513

This will all go away when we integrate the pfPagination component into pfTableView.
@akieling

address: "5 Tree Hill Lane",
city: "Boston",
state: "Massacusetts"
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added more items to the 'Basic' example in order to show that all 12 items are being displayed when pagination is turned off by default.

@cdcabrera cdcabrera self-requested a review July 10, 2017 13:47
Copy link
Member

@dlabrecq dlabrecq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@cdcabrera cdcabrera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cdcabrera cdcabrera merged commit 3a2b6f2 into patternfly:master Jul 10, 2017
@dtaylor113 dtaylor113 deleted the fix-513 branch October 19, 2017 18:29
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.

3 participants