From 60a802625b76e4f42640d0cbeec6bb6cfd4233f3 Mon Sep 17 00:00:00 2001 From: Xin Zhao Date: Wed, 5 Aug 2026 11:37:37 +0800 Subject: [PATCH] collab_ui: Fix inconsistent channel icons in collab panel (#62114) # Objective In PR #61397, the channel icons for public and private channels were updated, but the tab icon for channel notes was not covered, causing the tab icon to mismatch the channel icons shown in the collab panel, as shown below: before ## Solution Update the tab icon for channel notes to match the already-updated channel icons in the collab panel (public channels use `#`, private channels use a lock). ## Testing Built and tested locally, with before/after comparison attached below. ## Self-Review Checklist: - [x] I've reviewed my own diff for quality, security, and reliability - [x] Unsafe blocks (if any) have justifying comments - [x] The content adheres to Zed's UI standards ([UX/UI](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist) and [icon](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md) guidelines) - [ ] Tests cover the new/changed behavior - [x] Performance impact has been considered and is acceptable ## Showcase | Before | After | |:--:|:--:| |before|after| --- Release Notes: - Fixed channel tab icons for public and private channels not matching the icons shown in the collab panel. --- crates/collab_ui/src/channel_view.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/collab_ui/src/channel_view.rs b/crates/collab_ui/src/channel_view.rs index bcc91ac093..c77846d6db 100644 --- a/crates/collab_ui/src/channel_view.rs +++ b/crates/collab_ui/src/channel_view.rs @@ -457,8 +457,8 @@ impl Item for ChannelView { fn tab_icon(&self, _: &Window, cx: &App) -> Option { let channel = self.channel(cx)?; let icon = match channel.visibility { - ChannelVisibility::Public => IconName::Public, - ChannelVisibility::Members => IconName::Hash, + ChannelVisibility::Public => IconName::Hash, + ChannelVisibility::Members => IconName::Lock, }; Some(Icon::new(icon))