When a parent node with expanded child nodes in a tree is updated, child nodes are wrongly displayed.

[jongeuncha]

In test/tree_editor.html, expand Africa and Kenya

And run this in the console to update Africa node.

testCountryStore.put(testCountryStore.get("AF"))

And the expanded child nodes are wrongly displayed like below.

By examining DOM, height style of div that wraps Africa's children is 75px that is supposed to 125px.

This seems to be caused by wrongly treating Promise result of grid.renderArray() in grid.expand function in tree.js.
This gets messier when the store is JsonRest.

[runtarm]

Which browser and dgrid version do you use?

It didn't seems to reproducible on Chrome and dgrid 0.3.7.

[jongeuncha]

Sorry to forget to mention that.
dgrid version is 0.3.7
It works in Chrome but does not work in firefox.

[justindoherty]

In firefox there seems to be a height set for the dgrid-tree-container's div which does not seem to exist in chrome.

[runtarm]

That height seems to be a trigger for css-transition and will get removed when the transition has ended.

The problem is a transitionend event won't fire sometime in Firefox.

[mvaerle]

We're having the same issue with Firefox only. There seems to be a race condition between the animation finishing and the response coming back from the server, but that's just from visually observing.

[kfranqueiro]

Can you try this again against dgrid master and see if it's resolved? This sounds like it could be the same issue as #874.

[jongeuncha]

This issue is tested on v0.3.15 and seems to be resolved.

[kfranqueiro]

If it's resolved in 0.3.15 then I guess a prior revision might have already addressed your case. I was thinking something we checked into master yesterday might've fixed it.

Thanks for re-testing.