Remove pinned channel items customization from the demo

[gpunto]

Goal

Removing the custom background for pinned channels. We'll revisit it after the state is designed and bring it to the SDK.

Implementation

Remove the background modifier.

🎨 UI Changes

Pinned channels now look visually identical to regular ones

Testing

Launch the compose sample, pin a channel and verify that the corresponding channel item looks identical.

Summary by CodeRabbit

• Style
  • Removed visual styling previously applied to pinned channels.

[github-actions]

PR checklist ✅

All required conditions are satisfied:

• Title length is OK (or ignored by label).
• At least one pr: label exists.
• Sections ### Goal, ### Implementation, and ### Testing are filled.

🎉 Great job! This PR is ready for review.

[github-actions]

SDK Size Comparison 📏

SDK	Before	After	Difference	Status
stream-chat-android-client	5.25 MB	5.82 MB	0.57 MB	🔴
stream-chat-android-ui-components	10.60 MB	10.95 MB	0.36 MB	🟡
stream-chat-android-compose	12.81 MB	12.25 MB	-0.56 MB	🚀

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 049ec546-1d20-4eb7-80b3-cc745340ec27

📥 Commits

Reviewing files that changed from the base of the PR and between ec339bf and 89ccea3.

📒 Files selected for processing (1)

• stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/component/CustomChatComponentFactory.kt

---

Walkthrough

The pull request removes pinned channel visual styling logic from CustomChatComponentFactory. Previously, the code conditionally applied a background modifier based on channel pin status; the updated implementation removes this logic and consistently passes only Modifier.animateItem() to the ChannelItem component, eliminating unused imports.

Changes

Cohort / File(s)	Summary
Pinned Channel Styling Removal stream-chat-android-compose-sample/src/main/java/io/getstream/chat/android/compose/sample/ui/component/CustomChatComponentFactory.kt	Removed conditional pinned background modifier application logic; now always passes only Modifier.animateItem() to ChannelItem. Cleaned up associated imports (background and isPinned).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Modifiers simplified, the styling removed,
No more pinned backgrounds in the mood,
Just animate and flow so free,
Code now cleaner as can be! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: removing pinned channel customization from the demo app.
Description check	✅ Passed	The description covers Goal, Implementation, UI Changes, and Testing sections but omits several template sections like Contributor Checklist and comparison screenshots.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch rm-pinned-channel-design

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[andremion]

As I understood from https://getstream.slack.com/archives/C0A3JU6C6G3/p1775651311234899?thread_ts=1775568258.825169&cid=C0A3JU6C6G3, we should hide the pin option in the SDK.

[gpunto]

As I understood from https://getstream.slack.com/archives/C0A3JU6C6G3/p1775651311234899?thread_ts=1775568258.825169&cid=C0A3JU6C6G3, we should hide the pin option in the SDK.

Yeah, it's disabled by default in ChannelOptionItemVisibility

[andremion]

As I understood from https://getstream.slack.com/archives/C0A3JU6C6G3/p1775651311234899?thread_ts=1775568258.825169&cid=C0A3JU6C6G3, we should hide the pin option in the SDK.

Yeah, it's disabled by default in ChannelOptionItemVisibility

Oh, I'm seeing in the sample. Shouldn't we disable in the sample too?

[gpunto]

As I understood from https://getstream.slack.com/archives/C0A3JU6C6G3/p1775651311234899?thread_ts=1775568258.825169&cid=C0A3JU6C6G3, we should hide the pin option in the SDK.

Yeah, it's disabled by default in ChannelOptionItemVisibility

Oh, I'm seeing in the sample. Shouldn't we disable in the sample too?

Yeah, it makes sense

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud