Values not showing up for right-hand column cells when resize the window

[francesc79]

• Have you already searched for similar issues? Please help us out and double-check first!

• Also, please don't be that person who deletes this template. It's here for a reason.

• Thanks!

---

Which version of React JS are you using?

✅ Officially supported ✅

• [ ] v15.4.x

⚠️ Not officially supported, expect warnings ⚠️

• [ ] v15.5.x
• [ ] v15.6.x

☣️ Not officially supported, expect warnings and errors ☣️

• [X ] v16.x.x

---

Which browser are you using?

✅ Officially supported ✅

• [ ] IE 9 / IE 10 / IE 11
• [ ] Edge
• [X ] Chrome

⚠️ Not officially supported, but "should work" ⚠️

• [ ] Firefox
• [ ] Safari

---

I'm submitting a ...

• [X ] 🐛 Bug Report
• [ ] 💡 Feature Request

👋 Need general support? Not sure about how to use React itself, or how to get started with the Grid? Please do not submit support request here. Instead see

https://github.com/adazzle/react-data-grid/blob/master/CONTRIBUTING.md

---

Issue Details

When I resize the window same column cells disappear. I debug the library and I find what hepped. Into ViewportScrollMixin module is defined componentWillReceiveProps livecycle hook that call updateScroll function. The problem for my opinion is that when call updateScroll without last parameter width the function getRenderedColumnCount consider this.props.columnMetrics.totalWidth that is not updated.

--

[animeshkayal]

Facing similar issue while increasing the width of a column and then horizontal scroll to right.

[zyxelvincenzo79]

i have the same problem

[thupi]

I have same issue and is currently looking for a table solution :-) Any progress on this :-) ?

[francesc79]

@thupi I opened a PR, the bug was just fix, I'm waiting the merge with master. If you want try the fix you can checkout francesc79/react-data-grid branch zazos-issue-1087.

[amanmahajan7]

This is a known issue where sometimes the cell content is empty. We are actively working on improving the cell rendering logic and v4 should address this issue. In the meantime re-rendering grid seems like a potential solution

window.addEventListener('resize', () => {
   // Re-render grid
   // May be recalculate grid's minHeight
   const minHeight = ....;
   this.setState({ minHeight });
   // Or may be forceUpdate (not a huge fan of this approach)
   this.forceUpdate();
});

[animeshkayal]

Hi Aman, Kindly let us know when can we expect the V4 release?

[rlreamy]

Is this fixed in any of the v4 releases?

[kevyu]

I think the bug still exists in v4.

[amanmahajan7]

Please check the change log for what is included in v4. We recently merged a PR that improves cell rendering logic and will fix this particular issue. It is a big rewrite and we discovered a few bugs during internal testing. This is being actively worked on but at this moment we do not know when this will be released. I will keep you all posted.

[MickaToast]

Hi, any update on this? Or at least could you bump @next on NPM with the PR you mentioned @amanmahajan7 ? Thanks.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please reopen this if you feel it has been incorrectly closed and we will do our best to look into it. Thank you for your contributions.

[rlreamy]

Is there any update on this? It looks like it was supposed to be fixed in v4.

[vellov]

Any update on this?

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please reopen this if you feel it has been incorrectly closed and we will do our best to look into it. Thank you for your contributions.