-
Notifications
You must be signed in to change notification settings - Fork 161
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
IGX Datagrid autosize feature with performance improvements #10205
Comments
There has been no recent activity and this issue has been marked inactive. |
There has been no recent activity and this issue has been marked inactive. |
We will look into possible ways to improve the performance, however to ensure we are correctly addressing your issue I’d like to ask some additional questions on your scenario. When checking with our latest version 13.1.1 with the an updated version of your sample: https://stackblitz.com/edit/angular-zz2ek7 I’ve noticed in the performance profile that the actual measuring of the sizes is much smaller than the grid reflow call. While the getAutoSize call seem to be around 4-5ms per column. Could you perhaps share a sample where it takes 2 seconds for the operation so that we can take a look at that particular scenario? Perhaps there’s some other bottleneck that we’re not aware of or we may have some other ideas on how to improve the performance in that scenario. In the latest versions there’s probably no need to call he grid reflow explicitly as the width setter already marks the grid for size recalculations on the next detection cycle, so you could skip it from your improved version of the resizing. |
There has been no recent activity and this issue has been marked inactive. |
Is your feature request related to a problem? Please describe.
We are using IGX Grid in Angular with numerous rows and columns. Our grids are required to have all columns autosized once data is loaded. We do this by manually calling
column.autosize
and notice that is a performance bottleneck, sometimes taking up to 2.5 seconds to complete for a particularly large datagrid. This frustrates the user, especially when there are numerous datagrids being displayed and autosized.Describe the solution you'd like
We'd like:
@Input autosize: boolean
on the igx-grid component.Describe alternatives you've considered
We have made our own performance improvements on the IGX datagrid by avoiding
datagrid.reflow()
from being called until all columns have been resized. Please see our stackblitz example to compare the old way to our new way:https://stackblitz.com/edit/github-jxvrgd-9sbr3f?file=src/app/grid/grid-resizing-sample/grid-resizing-sample.component.ts
We'd like this performance to be further improved. We've found that the main bottleneck is happening in the
column.getAutoSize()
method.Additional context
Product Details
COMPONENT / igx-grid
PLATFORM / Angular
BUILD NUMBER / 10.2.24
The text was updated successfully, but these errors were encountered: