Change the context menu and Copilot settings icon (#14501)
Felt the link we were using for menu items that open a browser page was not the best. That one is most typically used for attachments within scope, as opposed to opening external links. Noticed that via the "Copilot Settings" menu, which also felt like it could have a bit more descriptive label. Also reduced the size of the rendered icon in this component. --- Release Notes: - N/A
This commit is contained in:
@@ -286,7 +286,7 @@ impl InlineCompletionButton {
|
||||
self.build_language_settings_menu(menu, cx)
|
||||
.separator()
|
||||
.link(
|
||||
"Copilot Settings",
|
||||
"Go to Copilot Settings",
|
||||
OpenBrowser {
|
||||
url: COPILOT_SETTINGS_URL.to_string(),
|
||||
}
|
||||
|
||||
@@ -178,7 +178,7 @@ impl ContextMenu {
|
||||
|
||||
action: Some(action.boxed_clone()),
|
||||
handler: Rc::new(move |_, cx| cx.dispatch_action(action.boxed_clone())),
|
||||
icon: Some(IconName::Link),
|
||||
icon: Some(IconName::ArrowUpRight),
|
||||
});
|
||||
self
|
||||
}
|
||||
@@ -360,7 +360,7 @@ impl Render for ContextMenu {
|
||||
h_flex()
|
||||
.gap_1()
|
||||
.child(Label::new(label.clone()))
|
||||
.child(Icon::new(*icon))
|
||||
.child(Icon::new(*icon).size(IconSize::Small))
|
||||
.into_any_element()
|
||||
} else {
|
||||
Label::new(label.clone()).into_any_element()
|
||||
|
||||
Reference in New Issue
Block a user