Skip to content

[QuickGrid] Reevaluate the number of rows on the last page of a Virtualized QuickGrid #47139

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

Closed
Tracked by #47137
Nick-Stanton opened this issue Mar 10, 2023 · 1 comment
Labels
area-blazor Includes: Blazor, Razor Components feature-blazor-quickgrid

Comments

@Nick-Stanton
Copy link
Member

Nick-Stanton commented Mar 10, 2023

// ARIA's rowcount is part of the UI, so it should reflect what the human user regards as the number of rows in the table,
// not the number of physical <tr> elements. For virtualization this means what's in the entire scrollable range, not just
// the current viewport. In the case where you're also paginating then it means what's conceptually on the current page.
// TODO: This currently assumes we always want to expand the last page to have ItemsPerPage rows, but the experience might
// be better if we let the last page only be as big as its number of actual rows.
_ariaBodyRowCount = Pagination is null ? providerResult.TotalItemCount : Pagination.ItemsPerPage;

@ghost ghost added the untriaged label Mar 10, 2023
@Nick-Stanton Nick-Stanton changed the title Reevaluate the number of rows on the last page of a Virtualized QuickGrid [QuickGrid] Reevaluate the number of rows on the last page of a Virtualized QuickGrid Mar 10, 2023
@Nick-Stanton Nick-Stanton added area-blazor Includes: Blazor, Razor Components feature-blazor-quickgrid labels Mar 10, 2023
@MackinnonBuck
Copy link
Member

Closing this out. If customers report this being an issue, we will use another issue to track addressing it.

@MackinnonBuck MackinnonBuck closed this as not planned Won't fix, can't repro, duplicate, stale Mar 13, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-blazor Includes: Blazor, Razor Components feature-blazor-quickgrid
Projects
None yet
Development

No branches or pull requests

3 participants