Detailed changes
@@ -17,9 +17,8 @@ use gpui::{
actions, canvas, div, fill, list, overlay, point, prelude::*, px, AnyElement, AppContext,
AsyncWindowContext, Bounds, ClipboardItem, DismissEvent, Div, EventEmitter, FocusHandle,
FocusableView, FontStyle, FontWeight, InteractiveElement, IntoElement, ListOffset, ListState,
- Model, MouseDownEvent, ParentElement, Pixels, Point, PromptLevel, Render, RenderOnce,
- SharedString, Styled, Subscription, Task, TextStyle, View, ViewContext, VisualContext,
- WeakView, WhiteSpace,
+ Model, MouseDownEvent, ParentElement, Pixels, Point, PromptLevel, Render, SharedString, Styled,
+ Subscription, Task, TextStyle, View, ViewContext, VisualContext, WeakView, WhiteSpace,
};
use menu::{Cancel, Confirm, SelectNext, SelectPrev};
use project::{Fs, Project};
@@ -2296,7 +2295,7 @@ impl CollabPanel {
h_flex()
.id(channel_id as usize)
.child(Label::new(channel.name.clone()))
- .children(face_pile.map(|face_pile| face_pile.render(cx))),
+ .children(face_pile.map(|face_pile| face_pile.render().p_1())),
),
)
.child(
@@ -57,6 +57,7 @@ impl Render for CollabTitlebarItem {
let current_user = self.user_store.read(cx).current_user();
let client = self.client.clone();
let project_id = self.project.read(cx).remote_id();
+ let workspace = self.workspace.upgrade();
h_flex()
.id("titlebar")
@@ -100,6 +101,7 @@ impl Render for CollabTitlebarItem {
true,
room.is_speaking(),
room.is_muted(),
+ None,
&room,
project_id,
¤t_user,
@@ -113,6 +115,12 @@ impl Render for CollabTitlebarItem {
}))
.children(
remote_participants.iter().filter_map(|collaborator| {
+ let player_color = player_colors
+ .color_for_participant(collaborator.participant_index.0);
+ let is_following = workspace
+ .as_ref()?
+ .read(cx)
+ .is_being_followed(collaborator.peer_id);
let is_present = project_id.map_or(false, |project_id| {
collaborator.location
== ParticipantLocation::SharedProject { project_id }
@@ -124,6 +132,7 @@ impl Render for CollabTitlebarItem {
is_present,
collaborator.speaking,
collaborator.muted,
+ is_following.then_some(player_color.selection),
&room,
project_id,
¤t_user,
@@ -134,13 +143,7 @@ impl Render for CollabTitlebarItem {
v_flex()
.id(("collaborator", collaborator.user.id))
.child(face_pile)
- .child(render_color_ribbon(
- player_colors
- .color_for_participant(
- collaborator.participant_index.0,
- )
- .cursor,
- ))
+ .child(render_color_ribbon(player_color.cursor))
.cursor_pointer()
.on_click({
let peer_id = collaborator.peer_id;
@@ -468,11 +471,12 @@ impl CollabTitlebarItem {
is_present: bool,
is_speaking: bool,
is_muted: bool,
+ leader_selection_color: Option<Hsla>,
room: &Room,
project_id: Option<u64>,
current_user: &Arc<User>,
cx: &ViewContext<Self>,
- ) -> Option<FacePile> {
+ ) -> Option<Div> {
if room.role_for_user(user.id) == Some(proto::ChannelRole::Guest) {
return None;
}
@@ -481,56 +485,72 @@ impl CollabTitlebarItem {
let followers = project_id.map_or(&[] as &[_], |id| room.followers_for(peer_id, id));
let extra_count = followers.len().saturating_sub(FACEPILE_LIMIT);
- let pile = FacePile::default()
- .child(
- Avatar::new(user.avatar_uri.clone())
- .grayscale(!is_present)
- .border_color(if is_speaking {
- cx.theme().status().info
- } else {
- // We draw the border in a transparent color rather to avoid
- // the layout shift that would come with adding/removing the border.
- gpui::transparent_black()
- })
- .when(is_muted, |avatar| {
- avatar.indicator(
- AvatarAudioStatusIndicator::new(ui::AudioStatus::Muted).tooltip({
- let github_login = user.github_login.clone();
- move |cx| Tooltip::text(format!("{} is muted", github_login), cx)
- }),
+ Some(
+ div()
+ .m_0p5()
+ .p_0p5()
+ // When the collaborator is not followed, still draw this wrapper div, but leave
+ // it transparent, so that it does not shift the layout when following.
+ .when_some(leader_selection_color, |div, color| {
+ div.rounded_md().bg(color)
+ })
+ .child(
+ FacePile::default()
+ .child(
+ Avatar::new(user.avatar_uri.clone())
+ .grayscale(!is_present)
+ .border_color(if is_speaking {
+ cx.theme().status().info
+ } else {
+ // We draw the border in a transparent color rather to avoid
+ // the layout shift that would come with adding/removing the border.
+ gpui::transparent_black()
+ })
+ .when(is_muted, |avatar| {
+ avatar.indicator(
+ AvatarAudioStatusIndicator::new(ui::AudioStatus::Muted)
+ .tooltip({
+ let github_login = user.github_login.clone();
+ move |cx| {
+ Tooltip::text(
+ format!("{} is muted", github_login),
+ cx,
+ )
+ }
+ }),
+ )
+ }),
)
- }),
- )
- .children(
- followers
- .iter()
- .take(FACEPILE_LIMIT)
- .filter_map(|follower_peer_id| {
- let follower = room
- .remote_participants()
- .values()
- .find_map(|p| (p.peer_id == *follower_peer_id).then_some(&p.user))
- .or_else(|| {
- (self.client.peer_id() == Some(*follower_peer_id))
- .then_some(current_user)
- })?
- .clone();
-
- Some(Avatar::new(follower.avatar_uri.clone()))
- }),
- )
- .children(if extra_count > 0 {
- Some(
- div()
- .ml_1()
- .child(Label::new(format!("+{extra_count}")))
- .into_any_element(),
- )
- } else {
- None
- });
-
- Some(pile)
+ .children(followers.iter().take(FACEPILE_LIMIT).filter_map(
+ |follower_peer_id| {
+ let follower = room
+ .remote_participants()
+ .values()
+ .find_map(|p| {
+ (p.peer_id == *follower_peer_id).then_some(&p.user)
+ })
+ .or_else(|| {
+ (self.client.peer_id() == Some(*follower_peer_id))
+ .then_some(current_user)
+ })?
+ .clone();
+
+ Some(Avatar::new(follower.avatar_uri.clone()))
+ },
+ ))
+ .children(if extra_count > 0 {
+ Some(
+ div()
+ .ml_1()
+ .child(Label::new(format!("+{extra_count}")))
+ .into_any_element(),
+ )
+ } else {
+ None
+ })
+ .render(),
+ ),
+ )
}
fn window_activation_changed(&mut self, cx: &mut ViewContext<Self>) {
@@ -1,13 +1,13 @@
-use gpui::{div, AnyElement, IntoElement, ParentElement, RenderOnce, Styled, WindowContext};
+use gpui::{div, AnyElement, Div, IntoElement, ParentElement, Styled};
use smallvec::SmallVec;
-#[derive(Default, IntoElement)]
+#[derive(Default)]
pub struct FacePile {
pub faces: SmallVec<[AnyElement; 2]>,
}
-impl RenderOnce for FacePile {
- fn render(self, _: &mut WindowContext) -> impl IntoElement {
+impl FacePile {
+ pub fn render(self) -> Div {
let player_count = self.faces.len();
let player_list = self.faces.into_iter().enumerate().map(|(ix, player)| {
let isnt_last = ix < player_count - 1;
@@ -17,7 +17,7 @@ impl RenderOnce for FacePile {
.when(isnt_last, |div| div.neg_mr_1())
.child(player)
});
- div().p_1().flex().items_center().children(player_list)
+ div().flex().items_center().children(player_list)
}
}