devextreme-reactive icon indicating copy to clipboard operation
devextreme-reactive copied to clipboard

Virtual scrolling does not extend for expanded groups

Open lukefanning opened this issue 7 years ago • 7 comments

Hi, I am using VirtualTable, GroupingState and IntegratedGrouping and I have noticed that when expanding groups the vertical scrollbar does not extend leading to rows overflowing and being hidden. It seems as though expanding and retracting groups does not cause a recalculation of the virtual table height.

  • [x] I have searched this repository's issues and believe that this is not a duplicate.

I'm using ...

  • [x] React Grid
  • [ ] React Chart
  • [ ] React Scheduler

Current Behaviour

VirtualTable scrollbar does not increase as groups are expanded.

Expected Behaviour

Scrollbar length would increase as more groups are expanded allowing you to view all grouped rows.

Steps to Reproduce (for Bugs)

Create a VirtualTable and add grouping. Group data by columns and then expand those columns.

Environment

  • devextreme-reactive: 1.10.0
  • react: 16.6.3
  • browser: Chrome 71.0.3578.98
  • bootstrap: 4.1.3
  • react-bootstrap: none
  • material-ui: 1.10.0

lukefanning avatar Feb 05 '19 11:02 lukefanning

I reverted back to Table but the problem is the same.

lukefanning avatar Feb 05 '19 14:02 lukefanning

Same here - Grouping and Virtual table dont work as intended - rows are disappearing when group is expanded.

in version 1.10.1 problem is still available

reproduction: https://devexpress.github.io/devextreme-reactive/react/grid/demos/featured/virtual-scrolling/

try open all groups in your demo - are all rows still visible? nope

PhobosX avatar Feb 07 '19 08:02 PhobosX

Hi, I couldn't reproduce the problem using both a sample project and our demo, see the attached screencast. @ssg-luke could you please provide us with a sample project where the issue is reproduced?

ushkal avatar Feb 07 '19 11:02 ushkal

Hi @ushkal I managed to reproduce it in a sandbox. It seems to be caused by using PagingState & IntegratedPaging with grouping. You check it out here

lukefanning avatar Feb 07 '19 12:02 lukefanning

Ok fixed ^_^. Paging must come before grouping. Then it works perfectly! Thanks for taking a look

lukefanning avatar Feb 07 '19 12:02 lukefanning

@ushkal Is it a bug tho that it only works if it grouping appears after paging? I can definitely see a case for wanting grouping across all data rather than just the current page with working scrolling... Should i reopen? 😅

lukefanning avatar Feb 11 '19 10:02 lukefanning

Hi @ssg-luke ,   At present, the DevExtreme Reactive Grid does not support a combination of paging and a virtual table. We'll notify you once it is implemented.   Sorry for the late response.

ushkal avatar Jun 06 '19 14:06 ushkal