michael-s-molina commented on code in PR #22135:
URL: https://github.com/apache/superset/pull/22135#discussion_r1025418956
##########
superset-frontend/src/components/Table/Table.overview.mdx:
##########
@@ -183,14 +183,27 @@ The table displays a set number of rows at a time, the
user navigates the table
The default page size and page size options for the menu are configurable via
the `pageSizeOptions` and `defaultPageSize` props.
NOTE: Pagination controls will only display when the data for the table has
more records than the default page size.
-<Story id="design-system-components-table-examples--many-columns" />
+<Story id="design-system-components-table-examples--pagination" />
```
<Table pageSizeOptions={[5, 10, 15, 20, 25] defaultPageSize={10} />
```
---
+### Virtualization for Performance
+
+The Table virtualization can be used to enable viewing data with many columns
and or rows without paging.
Review Comment:
I think we can disassociate virtualization and pagination. You can have a
paginated table with column virtualization.
```suggestion
Table virtualization can enable viewing data with many columns and/or rows.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org
