Skip to content

Conversation

@arnautov-anton
Copy link
Contributor

🎯 Goal

Use the capabilities of WithComponents to propagate component overrides, do not use Channel as a proxy.

BREAKING CHANGE: Channel component no longer supports component overrides, use WithComponents instead.

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Size Change: -911 B (-0.32%)

Total Size: 283 kB

Filename Size Change
./dist/Channel-B7a1pNDF.js 23.4 kB +23.4 kB (new file) 🆕
./dist/Channel-pqo4jV1s.js 0 B -24.4 kB (removed) 🏆
./dist/ChannelPreview-BeYOhWLM.js 17.8 kB +17.8 kB (new file) 🆕
./dist/ChannelPreview-BNyNO58P.js 0 B -17.8 kB (removed) 🏆
./dist/cjs/emojis.js 3.29 kB +1 B (+0.03%)
./dist/cjs/experimental.js 5.8 kB -1 B (-0.02%)
./dist/cjs/index.js 165 kB +60 B (+0.04%)
ℹ️ View Unchanged
Filename Size
./dist/audioProcessing-BbOs2wMd.js 1.32 kB
./dist/cjs/mp3-encoder.js 1.27 kB
./dist/css/v2/emoji-mart.css 1.84 kB
./dist/css/v2/emoji-replacement.css 300 B
./dist/css/v2/index.css 39.4 kB
./dist/css/v2/index.layout.css 22.8 kB
./dist/MessageInputContext-Buw2XInb.js 314 B

compressed-size-action

Copy link
Member

@oliverlaz oliverlaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to adjust the docs too

@arnautov-anton
Copy link
Contributor Author

Good to adjust the docs too

Yes, that's the next step as discussed earlier. :)

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.03%. Comparing base (d2d41b6) to head (6ab4788).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2904      +/-   ##
==========================================
+ Coverage   80.00%   80.03%   +0.03%     
==========================================
  Files         506      506              
  Lines       10777    10776       -1     
  Branches     2709     2709              
==========================================
+ Hits         8622     8625       +3     
+ Misses       1902     1898       -4     
  Partials      253      253              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

<ChatView.Selector />
<ChatView.Channels>
<ChannelList
Avatar={ChannelAvatar}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arnautov-anton don't we want to apply the same logic to ChannelList? I mean to not to allow passing Components as props and force the use of WithComponents.

Copy link
Contributor Author

@arnautov-anton arnautov-anton Jan 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, I can adjust the PR to include ChannelList too.

Edit: I'll open up separate PR to include renames to the ChannelList-related components.

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.

4 participants