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

Total Item Count and Filtered Item Count when paging on the server side #457

Closed
wants to merge 3 commits into from
Closed

Total Item Count and Filtered Item Count when paging on the server side #457

wants to merge 3 commits into from

Conversation

rappdw
Copy link

@rappdw rappdw commented May 28, 2013

The footer display of Total Items and Showing Items are correct within the context of all data on the client. However, in the case of server side paging, neither of these counts align with the total data on the server. This is demonstrated by: http://plnkr.co/edit/dG1MkH

c0bra and others added 3 commits May 29, 2013 10:48
…nabled

The footer display of Total Items and Showing Items are correct within the context of all data on the client. However, in the case of server side paging, neither of these counts align with the total data on the server. This commit adjusts the maxRows function in footer.js and the totalFilteredItemsLength function in grid.js to account for server side paging
@orneryd
Copy link
Contributor

orneryd commented Jun 4, 2013

what's going on here? can we close this with the release of 2.0.6?

@rappdw
Copy link
Author

rappdw commented Jun 4, 2013

From my reading of the documentation, I would expect the "total item" count to represent the number of items on the server, not the number of items currently being rendered in the page on the client.

With the release of 2.0.6, I'm still seeing the "total item" count only reflecting the current page side.

@msiniy
Copy link

msiniy commented Jun 5, 2013

It seems that docs and examples for server side paging should be updated, because the place where totalServerItems is set changed (#332).

Updated plunker example where I try to set totalServerItems everywhere, but the problem still exist with 2.0.6 -
http://plnkr.co/edit/A0rlyY?p=preview

@jeffpogo
Copy link

It is a problem in 2.0.7 too.

@jonricaurte
Copy link
Contributor

Sorry about the documentation. Here is a newer example on how to use it:

http://plnkr.co/edit/4LyAek?p=preview

Will update the documentation when I get back.

[EDIT] Oops wrong plunker:

http://plnkr.co/edit/50vJrs?p=preview

@jonricaurte jonricaurte mentioned this pull request Jun 28, 2013
@jonricaurte
Copy link
Contributor

I just updated the documentation. This should be fixed.

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