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

Suggestion -- Add table sort #35

Open
icsy7867 opened this issue Dec 30, 2022 · 3 comments
Open

Suggestion -- Add table sort #35

icsy7867 opened this issue Dec 30, 2022 · 3 comments

Comments

@icsy7867
Copy link

icsy7867 commented Dec 30, 2022

This is built into your tools already and it could be quite helpful for sorting by location or item name.

After doing a test, you simply have to update your List.vue file and change the sortable from false to true:

            this.headers.push({
              text: item.displayName,
              value: item.name,
              align: 'left',
              item: item, // So you can access all the parameters. Above are given in the data table support format.
              sortable: true

Added this change in:
#37

@FoxUSA
Copy link
Owner

FoxUSA commented Dec 31, 2022

If my memory serves there was a problem with paging and this in built sort. When I initially implemented this I inserted something like 10,000 of records to check the performance.

@icsy7867
Copy link
Author

It might be that the sort only works on the items shown due to the pagination.

I'm my case I'm looking at hundreds of items, not thousands. However a "show all" would be nice. I saw someone that did this with vuejs, but I'm not familiar enough to know how trivial that is.

I wonder if its possible when you set the pagination to do the array variably.

If total results >100 pagination array = [10, 50, 100, 500]
If > 1000 pagination array = [10, 50, 100, 500, 1000, 10000]
etc....

@FoxUSA
Copy link
Owner

FoxUSA commented Jan 2, 2023

I implemented the pagination as a feature. You don't have to have it. However, if you have tens of thousands of entries trying to be rendered, the page will crash. So pagination is required at some scale

@FoxUSA FoxUSA reopened this Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants