Framework: TreeGrid with DetailHandler broken

Created on 2 Nov 2018 · 6Comments · Source: vaadin/framework

Vaadin Framework version 8.6.0
Browser version all Browsers, currently working with Chrome Version 70.0.3538.77 (64-bit)
Web container name and version Apache Tomcat Version 9.0.8
Description of the bug Combination of TreeGrid with a DetailHandler is broken. Initially the grid looks fine. But as soon as the tree grid collapses a row, the details are listet at wrong position, and row hight is sometimes faulty too.
Expected behavior: If a group of rows is collapsed, that part is hidden and the remaining rows move up to use the grid space. After collapsing and expanding rows, the grid should look the same.
Actual behavior
If you collapse and expand the detail handler, the position is faulty.
Minimal reproducible example
```private TreeGrid createTestGrid() {
TreeGrid classGrid = new TreeGrid();

TreeData treeData = ((TreeDataProvider) classGrid.getDataProvider()).getTreeData();
classGrid.addColumn(row -> row.getSimpleName()).setId("simpleName");
classGrid.setDetailsGenerator(row -> new Label("details for " + row.getSimpleName()));

treeData.clear();
Class listClazz = List.class;
treeData.addItem(null, listClazz);
Class arrayListClazz = ArrayList.class;
treeData.addItem(listClazz, arrayListClazz);
classGrid.expand(listClazz);

Class setClazz = Set.class;
treeData.addItem(null, setClazz);
Class hashSetClazz = HashSet.class;
treeData.addItem(setClazz, hashSetClazz);
classGrid.expand(setClazz);

classGrid.setDetailsVisible(listClazz, true);
classGrid.setDetailsVisible(arrayListClazz, true);
classGrid.setDetailsVisible(setClazz, true);
classGrid.setDetailsVisible(hashSetClazz, true);

return classGrid;
}
```