From 07348366160fcc85062da9b7932a114b324f9fb9 Mon Sep 17 00:00:00 2001 From: nrq6993-a11y Date: Mon, 19 Oct 2026 00:03:50 +0300 Subject: [PATCH 1/2] Fix #2681: distinguish Appearance from Pathfinder panel icons The icon column used overlapping squares for both panels; Appearance now reads as a stacked list so it is not confused with Pathfinder (storytold/vectorcraft#2681). Co-authored-by: Cursor --- assets/icons/dc-appearance.svg | 2 +- crates/ui-egui/src/icons.rs | 13 +++++++++++++ 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/assets/icons/dc-appearance.svg b/assets/icons/dc-appearance.svg index 12054b4e5..41ba9d735 100644 --- a/assets/icons/dc-appearance.svg +++ b/assets/icons/dc-appearance.svg @@ -1 +1 @@ - + diff --git a/crates/ui-egui/src/icons.rs b/crates/ui-egui/src/icons.rs index 551aaf811..c7e275bc0 100644 --- a/crates/ui-egui/src/icons.rs +++ b/crates/ui-egui/src/icons.rs @@ -177,6 +177,19 @@ mod tests { } } + /// Appearance (effects) and Pathfinder both lived in the icon column; they used the same + /// overlapping-squares motif (storytold/vectorcraft#2681). + #[test] + fn appearance_panel_icon_is_not_the_pathfinder_icon() { + let appearance = String::from_utf8_lossy(include_bytes!("../../../assets/icons/dc-appearance.svg")); + let pathfinder = String::from_utf8_lossy(include_bytes!("../../../assets/icons/dc-pathfinder.svg")); + assert_ne!(appearance.as_bytes(), pathfinder.as_bytes()); + assert!( + appearance.contains(" Date: Tue, 20 Oct 2026 01:45:43 +0700 Subject: [PATCH 2/2] fmt (#2981) --- crates/ui-egui/src/icons.rs | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/crates/ui-egui/src/icons.rs b/crates/ui-egui/src/icons.rs index c7e275bc0..52476db6f 100644 --- a/crates/ui-egui/src/icons.rs +++ b/crates/ui-egui/src/icons.rs @@ -184,10 +184,7 @@ mod tests { let appearance = String::from_utf8_lossy(include_bytes!("../../../assets/icons/dc-appearance.svg")); let pathfinder = String::from_utf8_lossy(include_bytes!("../../../assets/icons/dc-pathfinder.svg")); assert_ne!(appearance.as_bytes(), pathfinder.as_bytes()); - assert!( - appearance.contains("