Skip to content

fix: modify FlashList scrollto workaround and provide upstream fix#3609

Merged
isekovanic merged 2 commits into
V8-releasefrom
fix/flashlist-patch-fix
May 21, 2026
Merged

fix: modify FlashList scrollto workaround and provide upstream fix#3609
isekovanic merged 2 commits into
V8-releasefrom
fix/flashlist-patch-fix

Conversation

@isekovanic
Copy link
Copy Markdown
Contributor

@isekovanic isekovanic commented May 21, 2026

🎯 Goal

In the pursuit of finding a better workaround for this PR, I believe I've found the underlying cause (or part of it at least) for why the issue actually happens. The technical details are in this upstream PR for FlashList.

πŸ›  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
Copy link
Copy Markdown
Contributor

Stream-SDK-Bot commented May 21, 2026

SDK Size

title develop branch diff status
js_bundle_size 368 KB 304 KB -64573 B πŸš€

@github-actions
Copy link
Copy Markdown

No changelog generated for the current changes on this pull request.

@isekovanic isekovanic requested review from oliverlaz and szuperaz May 21, 2026 13:45
@isekovanic isekovanic merged commit 0e96638 into V8-release May 21, 2026
8 checks passed
@isekovanic isekovanic deleted the fix/flashlist-patch-fix branch May 21, 2026 13:58
isekovanic added a commit that referenced this pull request May 22, 2026
## 🎯 Goal

This is a forward fix of [this
PR](#3609)
towards V9.

## πŸ›  Implementation details

<!-- Provide a description of the implementation -->

## 🎨 UI Changes

<!-- Add relevant screenshots -->

<details>
<summary>iOS</summary>


<table>
    <thead>
        <tr>
            <td>Before</td>
            <td>After</td>
        </tr>
    </thead>
    <tbody>
        <tr>
            <td>
                <!--<img src="" /> -->
            </td>
            <td>
                <!--<img src="" /> -->
            </td>
        </tr>
    </tbody>
</table>
</details>


<details>
<summary>Android</summary>

<table>
    <thead>
        <tr>
            <td>Before</td>
            <td>After</td>
        </tr>
    </thead>
    <tbody>
        <tr>
            <td>
                <!--<img src="" /> -->
            </td>
            <td>
                <!--<img src="" /> -->
            </td>
        </tr>
    </tbody>
</table>
</details>

## πŸ§ͺ Testing

<!-- Explain how this change can be tested (or why it can't be tested)
-->

## β˜‘οΈ Checklist

- [ ] I have signed the [Stream
CLA](https://docs.google.com/forms/d/e/1FAIpQLScFKsKkAJI7mhCr7K9rEIOpqIDThrWxuvxnwUq2XkHyG154vQ/viewform)
(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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants