From 69743e3a09aa6e8cb41f0c9eab1fa7d02ed9dd63 Mon Sep 17 00:00:00 2001
From: enricobuehler <buehler@unom.io>
Date: Wed, 29 Jul 2026 14:34:54 +0200
Subject: [PATCH] feat(client/windows): the profile surfaces get their icons
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Reset carries Undo, Edit profile carries Edit, Duplicate/Delete/Close carry Copy, Delete
and Accept — the icon support the bumped reactor added (#4736), applied where a glyph
disambiguates faster than a word. The rest of the shell already had its icons.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 clients/windows/src/app/settings.rs | 24 +++++++++++++++---------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/clients/windows/src/app/settings.rs b/clients/windows/src/app/settings.rs
index d7617cb4..c7f54e2d 100644
--- a/clients/windows/src/app/settings.rs
+++ b/clients/windows/src/app/settings.rs
@@ -274,7 +274,7 @@ fn edit_profile_modal(
         buttons.push(
             {
                 let (id, set_scope) = (id.clone(), set_scope.clone());
-                button("Duplicate").on_click(move || {
+                button("Duplicate").icon(Symbol::Copy).on_click(move || {
                     let mut catalog = ProfilesFile::load();
                     let Some(source) = catalog.find_by_id(&id).cloned() else {
                         return;
@@ -299,7 +299,9 @@ fn edit_profile_modal(
         buttons.push(
             {
                 let set_delete = set_delete.clone();
-                button("Delete\u{2026}").on_click(move || set_delete.call(Some(id.clone())))
+                button("Delete\u{2026}")
+                    .icon(Symbol::Delete)
+                    .on_click(move || set_delete.call(Some(id.clone())))
             }
             .into(),
         );
@@ -307,12 +309,15 @@ fn edit_profile_modal(
     buttons.push(
         {
             let (set_edit, set_rev) = (set_edit.clone(), set_rev.clone());
-            button("Close").accent().on_click(move || {
-                set_edit.call(false);
-                // The deferred repaint: the pane dropdown (and any pinned tiles) pick up the
-                // rename now, in one pass, instead of remounting per keystroke.
-                set_rev.call(rev + 1);
-            })
+            button("Close")
+                .accent()
+                .icon(Symbol::Accept)
+                .on_click(move || {
+                    set_edit.call(false);
+                    // The deferred repaint: the pane dropdown (and any pinned tiles) pick up the
+                    // rename now, in one pass, instead of remounting per keystroke.
+                    set_rev.call(rev + 1);
+                })
         }
         .into(),
     );
@@ -549,7 +554,7 @@ fn described_overridable(
                 .max_width(360.0)
                 .horizontal_alignment(HorizontalAlignment::Left)
                 .vertical_alignment(VerticalAlignment::Center),
-            button("Reset").on_click(move || {
+            button("Reset").icon(Symbol::Undo).on_click(move || {
                 let mut catalog = ProfilesFile::load();
                 if let Some(p) = catalog.profiles.iter_mut().find(|p| p.id == scope) {
                     p.overrides.clear(field);
@@ -1473,6 +1478,7 @@ pub(crate) fn settings_page(
             let set_edit = set_edit.clone();
             rows.push(
                 button("Edit profile\u{2026}")
+                    .icon(Symbol::Edit)
                     .on_click(move || set_edit.call(true))
                     .into(),
             );