[EuiDataGrid] Columns resize infinite loop

[lior-fortifieddata]

I'm trying to resolve an issue when resizing a column in EuiDataGrid.
unfortunately I couldn't replicate it in CodeSandBox for more than 1.5 hours,
so I thought it worth asking the team if they know it already
I can share my screen if needed.

you can see the issue below:
when resizing a column, look at the scrollbar at the bottom.

It didnt happen when:

1. setting initialWidth to the last column
2. on full screen

[j-m]

This happened to me when I put flex items in a cell (#4160), are you doing anything special in RenderCellValue or in leading/trailing columns?

[lior-fortifieddata]

no, it still happens when commenting the original renderCellValue and trailing / leading columns
for some reason the rendering of the grid is much slower comparing the table (I switched the tables to grids in our portal for using columns resizing), so I must be doing something wrong

[lior-fortifieddata]

ok changing from (originally I had more flex items in this group)

where table is the code in the prev comment
to

resolved the issue

[chandlerprall]

We've had an internal report of this issue as well, which was not reproducible outside of that instance. Due to the interaction with flex grid - which the virtualization effort removes from data grid - I'm optimistic this is completely resolved in the near future without a workaround.

[lior-fortifieddata]

Ok no problem. Thanks. May i also ask if there should be a known reason for slower rendering than tables? I can try to create codesandbox and open an issue if that's not the expected behavior Get Outlook for Android<https://aka.ms/ghei36>

…

________________________________ From: Chandler Prall <[email protected]> Sent: Monday, December 14, 2020 10:46:38 PM To: elastic/eui <[email protected]> Cc: Lior Goldemberg <[email protected]>; Author <[email protected]> Subject: Re: [elastic/eui] EuiDataGrid columns resize infinite loop (#4364) CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. We've had an internal report of this issue as well, which was not reproducible outside of that instance. Due to the interaction with flex grid - which the virtualization effort removes from data grid - I'm optimistic this is completely resolved in the near future without a workaround. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#4364 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AOIHQW5RBX2BJUDCJXXUBJLSUZ2S5ANCNFSM4URV4L3Q>.

[chandlerprall]

We are tracking a few performance issues with EuiDataGrid in #4295 . In-progress work includes virtualization (#4170) and tracking hook calls (#4347).

The main performance lag with EuiDataGrid is DOM creation for each cell, which is very dense compared to the other table components. The virtualization effort helps here, but we'll continue looking for ways to improve that initialization time.

[lior-fortifieddata]

thanks. your work is highly appreciated

…

________________________________ From: Chandler Prall <[email protected]> Sent: Tuesday, December 15, 2020 6:04 PM To: elastic/eui <[email protected]> Cc: Lior Goldemberg <[email protected]>; Author <[email protected]> Subject: Re: [elastic/eui] EuiDataGrid columns resize infinite loop (#4364) CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. We are tracking a few performance issues with EuiDataGrid in #4295<#4295> . In-progress work includes virtualization (#4170<#4170>) and tracking hook calls (#4347<#4347>). The main performance lag with EuiDataGrid is DOM creation for each cell, which is very dense compared to the other table components. The virtualization effort helps here, but we'll continue looking for ways to improve that initialization time. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#4364 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AOIHQWZFZ2APSGTDMDZOOBLSU6CH5ANCNFSM4URV4L3Q>.

[cee-chen]

I'm assuming this issue is no longer applicable now that EuiDataGrid is virtualized, so going to go ahead and close this. @lior-fortifieddata, please feel free to reopen this if you're still seeing this issue.