Reduce some icon code duplication

Richard Feldman created 2 weeks ago

Change summary

crates/agent_ui/src/acp/model_selector.rs | 16 +++++++---------
1 file changed, 7 insertions(+), 9 deletions(-)

Detailed changes

crates/agent_ui/src/acp/model_selector.rs

@@ -293,15 +293,13 @@ impl PickerDelegate for AcpModelPickerDelegate {
                                         .w_full()
                                         .gap_1p5()
                                         .map(|this| match &model_info.icon {
-                                            Some(AgentModelIcon::Path(path)) => this.child(
-                                                Icon::from_external_svg(path.clone())
-                                                    .color(model_icon_color)
-                                                    .size(IconSize::Small),
-                                            ),
-                                            Some(AgentModelIcon::Named(icon)) => this.child(
-                                                Icon::new(*icon)
-                                                    .color(model_icon_color)
-                                                    .size(IconSize::Small),
+                                            Some(icon) => this.child(
+                                                match icon {
+                                                    AgentModelIcon::Path(path) => Icon::from_external_svg(path.clone()),
+                                                    AgentModelIcon::Named(icon) => Icon::new(*icon)
+                                                }
+                                                .color(model_icon_color)
+                                                .size(IconSize::Small)
                                             ),
                                             None => this,
                                         })