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

Datagrid: Add virtualization setting #1288

Closed
inforrpunay opened this issue May 4, 2022 · 4 comments
Closed

Datagrid: Add virtualization setting #1288

inforrpunay opened this issue May 4, 2022 · 4 comments

Comments

@inforrpunay
Copy link

Is your feature request related to a problem? Please describe.
A slowness in datagrid process is encountered when there are already more than a hundred rows available.

Describe the solution you'd like
Angular implementation for this sample: https://design.infor.com/code/ids-enterprise/4.60.0/demo/components/datagrid/test-large-dataset-performance-virtualized.html

Additional context
We noticed that virtualized option is removed for the angular implementation of soho-datagrid
image

@tmcconechy tmcconechy added type: bug 🐛 [8] Velocity rating (Fibonacci) labels May 4, 2022
@tmcconechy
Copy link
Member

The reason this is disabled is it doesn't work very well. I think to fix it we will have to redo the core of the datagrid. We have started this in the new version with this in mind.

That example works ok but its clunky when scrolling and almost every feature doesn't work well like editing ect. It might be ok with an entirely readonly grid tho. But any other features we would have to recode them.

So not wanting to introduce this as it will introduce more issues that are pretty much impossible to fix without rewrite from the core up so dont think its worth allowing teams to use this at this point.

@tmcconechy tmcconechy changed the title Datagrid virtualization Datagrid: Add virtualization setting May 4, 2022
@tmcconechy
Copy link
Member

Using paging for now... As a way to solve this.

@EdwardCoyle EdwardCoyle added the team: m3 Issues for m3 label May 10, 2022
@EdwardCoyle
Copy link
Contributor

Noting here that:

  • Currently we aren't able to integrate virtualization into the 4.x datagrid per Tim's comments above
  • M3 is unable to use paging as a workaround (their users are not expecting the pager to be added)
  • M3 is currently waiting on a fully-supported version of the WebComponents datagrid so they can use virtualization.

@tmcconechy
Copy link
Member

Also noting I am working on it this sprint next and the remaining items are listed on infor-design/enterprise-wc#737 based one what was told was being used

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants