Skip to content

fix: video thumbnail loading#3584

Merged
isekovanic merged 3 commits intodevelopfrom
fix/video-thumbnail-loading
May 5, 2026
Merged

fix: video thumbnail loading#3584
isekovanic merged 3 commits intodevelopfrom
fix/video-thumbnail-loading

Conversation

@isekovanic
Copy link
Copy Markdown
Contributor

🎯 Goal

This PR fixes video thumbnail loading logic so that it is consistent across video thumbnails and image attachments.

Additionally, it:

  • Improves performance on Android by reducing the number of moving parts in the shimmering layer (same as we did for iOS)
  • Introduces an orchestrator so that each shimmering view doesn't create its own ValueAnimator unnecessarily
  • Makes sure offscreen views are not animating (they still stay registered though unless they actually unmount/detach)
  • Fixes offline images with the new FastImage library in the sample app

🛠 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 oliverlaz May 5, 2026 14:02
@Stream-SDK-Bot
Copy link
Copy Markdown
Contributor

SDK Size

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

@isekovanic isekovanic merged commit 632ff3c into develop May 5, 2026
5 checks passed
@isekovanic isekovanic deleted the fix/video-thumbnail-loading branch May 5, 2026 14:16
@github-actions github-actions Bot mentioned this pull request May 5, 2026
6 tasks
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