maintainVisibleContentPosition Not Working

[nickcherry]

Current behavior

When a maintainVisibleContentPosition prop is given to a FlashList and items are prepended to the data array, the user sees the list content move. This can be observed in the video below. On the left tab, we have React Native's FlatList, which does not scroll as content is prepended to its data array. On the right tab, we have FlashList, which does scroll as content is prepended, even when providing a valid keyExtractor and a maintainVisibleContentPosition of { minIndexForVisible: 0 }.

maintain-visible-content-position.mov

See https://github.com/nickcherry/flashlist-maintain-visible-content-position for reproducible demo.

Expected behavior

I would expect FlashList to behave like FlatList and not scroll when content is prepended to its data and maintainVisibleContentPosition is not undefined.

To Reproduce

https://github.com/nickcherry/flashlist-maintain-visible-content-position

Platform:

• iOS
• Android

Environment

1.2.1

[naqvitalha]

maintainVisibleContent position is unsupported right now. We will update here once we get to it.

[hirbod]

The last missing piece for a almost perfect library :)

[aweffr]

+1 we really need maintainVisibleContentPosition, please

[zedelashvililevani]

+1 I copy repo and make custom bridge for android :/

[harrisontaee]

+1, extremely valuable prop... would perfect the library!

[xmflsct]

👀

[robertontiu]

Any news on this one?

[isaachinman]

Just ran into this myself. Might need to revert to FlatList. Would be grateful if anyone is able to work on this feature.

[irisjae]

FlashList as of now does not support the maintainVisibleContentPosition prop or similar functionality.

For those of you who may need similar functionality, I've published an analysis of this problem and a patch to solve it here; feel free to test it out.

Note that instead of attempting to replicate ScrollView's maintainVisibleContentPosition prop, my patch creates a new prop preserveVisiblePosition instead.

[Marvin-Bai]

Hey team, any updates on this issue? I see there is a fix about an experimental feature supporting MVCP from discord forked repo. Can't we merge it into this? 👀

[Rohit3523]

Hey team, any updates on this issue? I see there is a fix about an experimental feature supporting MVCP from discord forked repo. Can't we merge it into this? 👀

Can you share the fork link @Marvin-Bai

[Marvin-Bai]

Hey team, any updates on this issue? I see there is a fix about an experimental feature supporting MVCP from discord forked repo. Can't we merge it into this? 👀

Can you share the fork link @Marvin-Bai

Here it is.
https://github.com/discord/flash-list

[irisjae]

If what you're referring to is this branch, the commits appear to be just a squash of this PR, which for the reasons I explain in the link from my above comment, cannot fully solve the issue (with problems like not working well during scrolling and not working if the indexes shift too much).

See if my linked fork/patch works well for you, and if it does, it would be great if you added support for my PR to get thoughts and attention of the maintainers.

P.S.
With respect to the two flaws I specifically linked to, my fork/patch addresses it respectively in:

• Mechanism -> Details -> 'Refixing' -> Timing section (_queueLayoutRefix in the recyclerlistview code)
• Mechanism -> Details -> List changes section (handleDatasetChange in the recyclerlistview code)

Besides these two flaws, my patch addresses a good deal of other edge cases; in the latter part of my explainer, essentially each section corresponds to the presence of an edge case, and how we have dealt with/can deal with it properly.

[Nasseratic]

Would be amazing to get this 🥲

[GeorgeFlorian]

Any news on this ?

[Rohit3523]

Any update?

[RichardSleet]

Any update?