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

Tech: A demo that combines the sortable and pagination table. #93

Open
zoltan-dulac opened this issue May 28, 2024 · 2 comments · May be fixed by #188
Open

Tech: A demo that combines the sortable and pagination table. #93

zoltan-dulac opened this issue May 28, 2024 · 2 comments · May be fixed by #188
Assignees
Labels
component development adding to or creating a new component for Enable. M one day of effort P1 High priority work

Comments

@zoltan-dulac
Copy link
Collaborator

We don't have an example of combining the sortable and pagination table. I remember I tried doing this a long time ago but there was a snag and meant to revisit but didn't. This ticket would involve doing that.

This ticket would involve:

  • Putting an example on the Pagination Table page.
  • Linking to that example in the Sortable table page.
  • creating unit tests.
@alisonhall alisonhall added the component development adding to or creating a new component for Enable. label May 29, 2024
@zoltan-dulac zoltan-dulac added M one day of effort P1 High priority work labels May 31, 2024
@Neha-Singla Neha-Singla self-assigned this Jul 19, 2024
@Neha-Singla Neha-Singla linked a pull request Jul 26, 2024 that will close this issue
@Neha-Singla
Copy link
Collaborator

Adding the WIP MR:
#188

@Neha-Singla
Copy link
Collaborator

Neha-Singla commented Aug 15, 2024

Hi @zoltan-dulac
Please find the below inputs of this ticket.
http://localhost:8888/sortable-and-pagination-table.php --Page to see functionality on local

PR: #188
Basically, the existing sorting and pagination library is combined and changes are made in dom so that sorting is working along with Pagination.

Pending Items:

  1. on changing the order of sorting, page refresh is not working properly.Need to integrate the pagination render table function after sorting is done.
  2. he documentation part is pending.
  3. adding the unit testcases
    4.Accessibility testing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component development adding to or creating a new component for Enable. M one day of effort P1 High priority work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants