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

[WC-2757] show numbers of rows pagination for virtual scrolling and load more #1392

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

Conversation

gjulivan
Copy link
Collaborator

Pull request type


Description

iobuhov
iobuhov previously approved these changes Jan 13, 2025
Copy link
Collaborator

@iobuhov iobuhov left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -1428,13 +1428,13 @@ exports[`Table with selection method checkbox render an extra column and add cla
<span
class="sr-only sr-only-focusable"
>
Currently showing 11 to 20
Currently showing 11 to 0
Copy link
Contributor

Choose a reason for hiding this comment

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

This new snapshot seems to be incorrect.

@@ -1154,7 +1154,7 @@ exports[`Table renders the structure correctly with sorting 1`] = `
class="test widget-datagrid"
>
<div
class="widget-datagrid-content sticky-table-container infinite-loading"
class="widget-datagrid-content sticky-table-container"
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we removing this class?

@gjulivan gjulivan force-pushed the WC-2757-dg-pagination branch from f2fe6a0 to d51c4e9 Compare January 16, 2025 10:15
@gjulivan gjulivan force-pushed the WC-2757-dg-pagination branch from bdd248b to d51c4e9 Compare January 16, 2025 10:54
@gjulivan gjulivan force-pushed the WC-2757-dg-pagination branch 2 times, most recently from 702a692 to 58e4634 Compare January 16, 2025 13:40
@gjulivan gjulivan force-pushed the WC-2757-dg-pagination branch from 58e4634 to acf0349 Compare January 16, 2025 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants