search

silevis / reactgrid

Add spreadsheet-like behavior to your React app
https://reactgrid.com
MIT License
1.24k stars 135 forks source link

Part of rows hidden #116

Closed othman-19 closed 6 months ago

othman-19 commented 2 years ago

A part of rows is hidden, it only appears when I click on the cell position.

https://user-images.githubusercontent.com/42877188/179425290-f89ddee2-5869-4204-b84f-6d7da2f3e76a.mov

robertgutu commented 2 years ago

I have the same issue with a grid. I set the height of grid wrapper to a fixed value with scroll enabled and when the page loads last few rows are invisible until I scroll down. Does anyone found a fix for this?

https://user-images.githubusercontent.com/48117048/193790000-4c52017f-c67d-47b9-8cec-e3a113ce642e.mp4

Matt-Jensen commented 2 years ago

Was having this same problem. This is a really hacky non-performant fix but you can set disableVirtualScrolling={true}

However looking into ensuring the configured scrollable element is correct wouldn't be a bad idea. However I couldn't figure out how to set that.

alonshmiel commented 1 year ago

Hi @othman-19 , please close this issue if it's not relevant or add a codesandbox that reproduces it. Thanks!

thuanhoang97 commented 1 year ago

I also face this issue with cells that have a span. Part of it will be hidden when I scroll. I have updated the example below. Does we any way to workaround util the issue got fixed?

https://codesandbox.io/s/reactgrid-liquidity-planner-forked-nk7dv2?file=/src/LiquidityPlanner.tsx

https://github.com/silevis/reactgrid/assets/38289417/0e44b56d-0f5a-4b51-aa00-3694a71bcddd

alonshmiel commented 1 year ago

Hi, Do you mean the header, right?

thuanhoang97 commented 1 year ago

yes, the header cell with span. We really need disableVirtualScrolling feature on to use with large data.

DLowHP commented 1 year ago

Hi @thuanhoang97! I investigated your example and found a workaround.

Unfortunately your issue, as well as other mentioned in this discussion, require major changes in the grid renderer so it will take us significant amount of time. Because of that I think it will most likely be postponed until we fix the smaller bugs that we can move with faster.

I hope you understand, take care!