reactdatagrid icon indicating copy to clipboard operation
reactdatagrid copied to clipboard

At rtl direction when we have more then 14 columns - not displayed all cells data

Open art1990 opened this issue 5 years ago • 7 comments

  • what edition are you using - community
  • version 4.0.22

What you did: implemented a grid in rtl mode What happened: when we have more then 14 columns - not displayed all cells data Screenshot from 2021-01-26 22-20-11 Screenshot from 2021-01-26 22-20-22

art1990 avatar Jan 26 '21 20:01 art1990

Thanks for reporting - I'm sure it has to do with https://reactdatagrid.io/docs/api-reference#props-virtualizeColumnsThreshold So until we come up with a fix, you can increment that number if you have say 20 cols, so they are rendered at all times.

inovua-admin avatar Jan 27 '21 13:01 inovua-admin

thanks, this resolved my problem

art1990 avatar Jan 27 '21 14:01 art1990

Glad it fixes your issue. We'll keep this opened as it is a bug, and the above solution is just a workaround. When there are very many columns and you have rtl, you can't just make the threshold very big, since it will degrade the grid performance too much - so column virtualization is needed.

inovua-admin avatar Jan 28 '21 21:01 inovua-admin

i found another bug in rtl direction if we have more then 14 columns and columns props doesnot have props flex, screenshot bellow Снимок экрана 2021-02-08 в 08 27 40 if we added flex param to last columns it looks like good, but when we accept sort to columns (or filtering) table body is shifted and aligns when scrolling

https://user-images.githubusercontent.com/46848114/107184318-02311680-69e9-11eb-930c-1557ba7013c7.mov

art1990 avatar Feb 08 '21 06:02 art1990

thanks for the report. one thing in the video above is the country name is used for many cols - it can be reused, but in this case make sure each column has a unique id property. Probably this is not the cause of the issue, but it's a good practice to follow.

inovua-admin avatar Feb 11 '21 12:02 inovua-admin

Hello @inovua-admin, firstly I would like to thank you very much for this great library!

Are there any updates on solving this bug? It would be greatly appreciated as we have some rather large datasets.

Thanks!

Mohammad-Mohimmatech avatar May 29 '22 21:05 Mohammad-Mohimmatech

To solve the issue change the CSS style for the .inovua-react-virtual-list__view-container class, like this: .inovua-react-virtual-list__view-container { overflow: hidden !important }

adnanmirani1990 avatar Jul 02 '23 15:07 adnanmirani1990