You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Considered the value versus complexity for all users of the library as well as library maintenance
Considered if this can be a storybook or documentation example
Is your feature request related to a problem? Please describe
A clear and concise description of what the feature is.
Please refer to #466
This can be fixed by removing the hardcoded page "1" in tableReducer.js
If we want only the data displayed in the current page to be sorted, and user is already at a particular page, it will be nice to keep the user on the current page instead of sending them to page 1.
Describe the solution you'd like
If sortServer is false, and paginationServer is true, edit logic to avoid setting currentPage to 1
OR
Create a new prop persistCurrentPageOnSort that keeps currentPage on sort in case of server-side pagination.
A clear and concise description of what you want to happen.
In tableReducer.js -> SORT_CHANGE, add logic described in the solution.
Example: const keepPage = pagination && paginationServer && persistCurrentPageOnSort;
currentPage: keepPage ? state.currentPage : 1,
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered:
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Feature Check list
Is your feature request related to a problem? Please describe
A clear and concise description of what the feature is.
Please refer to #466
This can be fixed by removing the hardcoded page "1" in tableReducer.js
If we want only the data displayed in the current page to be sorted, and user is already at a particular page, it will be nice to keep the user on the current page instead of sending them to page 1.
Describe the solution you'd like
If sortServer is false, and paginationServer is true, edit logic to avoid setting currentPage to 1
OR
Create a new prop persistCurrentPageOnSort that keeps currentPage on sort in case of server-side pagination.
A clear and concise description of what you want to happen.
In tableReducer.js -> SORT_CHANGE, add logic described in the solution.
Example:
const keepPage = pagination && paginationServer && persistCurrentPageOnSort;
currentPage: keepPage ? state.currentPage : 1,
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: