List row render to default height between cache.clear(i) and recomputeRowHeights(i), mess up scrollTop position

We are using List and CellMeasurer in our project, we noticed our list scroll up after cache.clear(10) is called. After logging everything, here is what happens:

Basic code:

cache.clear(10);
this.ref.recomputeRowHeights(10);

Logs:

• Current List scrollTop: 18440
• Current cached index 10 row height: 130
• We notice item data has changed, So we run cache.clear(10)
• Log shows cache.rowHeight() is running, because index 10 is cleared, it returns our default row - height: 90
• The list height become shorter and re-rendering
• recomputeRowHeights(10) is finally finished
• recompute result shows new height is still 130, Current cached index 10 row height set to 130 again.
• Log shows cache.rowHeight() is running again, The list height become taller and re-rendering
• onScroll event on List is triggerd, current List scrollTop now auto changed to 18400, exactly the 40 difference between measured height(130) and default height(90).

Basically, cache.clear(index) causing the list item to re-render at default height. Then recomputeRowHeights(index) fixes the height but scrollTop is broken due to this.

I am not sure why recomputeRowHeights does not update cache directly. And wondering if there is correct way for handling this.

[imensha]

In my project I was able to solve this problem (or at least one very similar) by replacing the call to recomputeRowHeights() with a call to forceUpdateGrid(). I have yet to see any problems with this alternative approach, but whether it’s the proper solution I don’t know. The documentation states that forceUpdateGrid is useful when data has changed but heights have not, however I suspect (without looking into the source) that when rows have been cleared from the cache, re-rendering the grid also forces CellMeasurer to recompute the row heights, while bypassing the step where default heights are used.

[jcarroll2007]

Any news on how to fix this? I’m running into the same issue.