Skip to content

fix: segmented reactions on press - #3746

Merged
isekovanic merged 2 commits into
developfrom
fix/segmented-reactions-on-press
Jul 29, 2026
Merged

fix: segmented reactions on press#3746
isekovanic merged 2 commits into
developfrom
fix/segmented-reactions-on-press

Conversation

@isekovanic

Copy link
Copy Markdown
Contributor

🎯 Goal

This PR addresses a behavioural change that unfortunately slipped through with V9 that diverges from what we had in the previous versions. Namely, for segmented reactions we applied reaction toggling instead of targeted opening of the reaction info bottom sheet.

This diverged from the other SDKs and so here we align this divergence.

Additionally, we pass reactionType to additionalInfo and make sure it's propagated everywhere so that we can easily replicate the behaviour if needed, like so:

<Channel
  onPressMessage={({ emitter, additionalInfo, actionHandlers, defaultHandler }) => {
    if (emitter === 'reactionList' && additionalInfo?.reactionType) {
      actionHandlers?.toggleReaction(additionalInfo.reactionType as string);
      return;
    }
    defaultHandler?.();
  }}
/>

🛠 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 review from oliverlaz and szuperaz July 29, 2026 20:21
@Stream-SDK-Bot

Copy link
Copy Markdown
Contributor

SDK Size

title develop branch diff status
js_bundle_size 1990 KB 1990 KB +17 B 🟢

@isekovanic
isekovanic merged commit d8d8a1c into develop Jul 29, 2026
10 checks passed
@isekovanic
isekovanic deleted the fix/segmented-reactions-on-press branch July 29, 2026 20:26
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