Vaadin 8 TreeGrid: Does scrollTo(int) method make any sense?

Hi,

Does current implementation of scrollTo(int) method in TreeGrid make sense? It seems like it may scroll you to any row with the same index depending on the current state of the tree, whether some of the nodes are expanded or collapsed.

I agree. There is no specific implementation of scrollTo(…) in TreeGrid, so it is just the inherited method from Grid. And in certain cases it is not very well behaving with TreeGrid. This should be better documented I think.

We added JavaDoc note about this: https://github.com/vaadin/framework/pull/11399

Thanks!