Skip to content

Conversation

@isekovanic
Copy link
Contributor

🎯 Goal

A unfortunate regression was introduced with this change, where thread messages were not being scrolled to once loaded properly. The reason for the bug is a bit deeper inside of how FlatList handles prop changes for MVCP specifically, but I won't go into that many details in the description. Simply making sure that we don't exit early if the list is empty with regards to whether scrolling should happen or not (on threads) is enough to work around this impediment.

Should close this 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

@isekovanic isekovanic requested a review from khushal87 July 9, 2025 08:27
@isekovanic
Copy link
Contributor Author

@khushal87 please test this out when you can and backport it to V7 as well if it indeed works.

@Stream-SDK-Bot
Copy link
Contributor

SDK Size

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

@khushal87 khushal87 merged commit 7ff1c06 into develop Jul 9, 2025
5 of 6 checks passed
@khushal87 khushal87 deleted the fix/thread-messages-not-autoscrolled branch July 9, 2025 13:39
@github-actions github-actions bot mentioned this pull request Jul 9, 2025
@stream-ci-bot
Copy link
Contributor

🎉 This PR is included in version 8.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛] Thread messages will not be visible, and additionally can render messages from the previous thread

5 participants