fix: properly set minIndexForVisible

[isekovanic]

🎯 Goal

This PR addresses an issue where the MessageList's underlying FlatList would get caught in a weird infinite scrolling (into nothing) state whenever we removed the last item inside of it.

This was particularly obvious on threads, with the following reproduction steps:

• Set deletedMessagesVisibilityType={'never'} on the Channel component encapsulating a Thread
• Create a thread from a message
• Write a single message into the thread
• Delete the message (no Deleted Message default as we set the visibility to never)
• The FlatList would now spin endlessly

The actual issue is that the FlatList completely bugs out whenever we pass it an index it can clearly not find (since the message got deleted, the index no longer exists. Instead, we keep the minIndexForVisible set to 1 for anything other than an empty list.

Should also close this Github issue.

🛠 Implementation details

🎨 UI Changes

iOS

Before	After

Android

Before	After

🧪 Testing

☑️ Checklist

• I have signed the Stream CLA (required)
• PR targets the develop branch
• Documentation is updated
• New code is tested in main example apps, including all possible scenarios
  • SampleApp iOS and Android
  • Expo iOS and Android

[Stream-SDK-Bot]

SDK Size

title	develop	branch	diff	status
js_bundle_size	450 KB	450 KB	0 B	🟢

[stream-ci-bot]

🎉 This PR is included in version 7.2.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀