From 573e2da84a02618056590dcf39e5d21af5faa1ee Mon Sep 17 00:00:00 2001
From: dotCooCoo <RobertLeeLW@gmail.com>
Date: Tue, 13 Oct 2026 03:59:05 -0700
Subject: [PATCH] Swatches: dropping an .ase file on the window loads its
 swatches

open_dropped sent every dropped file that is not a document to
file.place, so a swatch exchange (.ase) file dropped on the desktop
window failed with a "Can't Place the File" alert: "invalid parameters
for `file.place`: unsupported or corrupt image".

open_dropped now sends .ase files to swatch.load with the dropped path,
as Load Swatches in the Swatches panel menu does with the picked path.
The web app already sent dropped .ase files to swatch.load. The
swatches are added to the active document as Load Swatches adds them,
and if the file can't be loaded, the status line shows the error.
---
 ROADMAP.md                |  1 +
 crates/ui-egui/src/lib.rs | 28 +++++++++++++++++++++++++++-
 2 files changed, 28 insertions(+), 1 deletion(-)

diff --git a/ROADMAP.md b/ROADMAP.md
index 8a0f82071..bf06425bc 100644
--- a/ROADMAP.md
+++ b/ROADMAP.md
@@ -156,6 +156,7 @@ Newest first.
 - **2026-10-12** · Words in capitals hyphenate like capitalized words (FERRUM COM-MODI), as in InDesign; Hyphenate Capitalized Words off keeps them whole. No readiness percentage change.
 - **2026-10-11** · `transform.move {copy}`, Edit › Duplicate and Step and Repeat return the copies' ids, so a script can go on with them. No readiness percentage change.
 - **2026-10-11** · `object.fill` and `object.stroke` refuse a swatch name the document doesn't have (it painted nothing) and suggest the name meant (`Paper` → `[Paper]`). No readiness percentage change.
+- **2026-10-13** · Dropping an `.ase` file on the window loads its swatches, as Load Swatches does (the drop failed with "Can't Place the File"). No readiness percentage change.
 - **2026-10-11 (printing):** `file.printers` and `file.print` use craft-libs/craft-print for CUPS and Windows. DesignCraft retains PDF export, page selection, spreads, marks and bleed; dry runs create no temporary PDF or print job. Windows selects matching or default driver paper and shrinks oversized sheets without enlarging smaller ones; queue duplex defaults are preserved. Headless tests exercise exported bytes, job settings, dry runs and refusals; shared native Windows CI passes, including paused-queue submission and cleanup. PostScript, production print controls and physical output validation remain open. No readiness percentage change.
 
 - **2026-10-10 (UI.4–UI.5):** Panel tabs can move into other groups, split the dock, float, resize and return to their saved locations through shared docking transactions. Layouts persist in custom workspaces; UI and control commands share the same validated action path. Shared pane geometry and widget-state theme application retain all five palettes and local fonts. Split panes send keyboard input only to the focused canvas and scope contextual toolbar IDs per pane. No readiness percentage change.
diff --git a/crates/ui-egui/src/lib.rs b/crates/ui-egui/src/lib.rs
index fc7a3e634..568587759 100644
--- a/crates/ui-egui/src/lib.rs
+++ b/crates/ui-egui/src/lib.rs
@@ -969,8 +969,12 @@ impl DesignApp {
         }
     }
 
-    /// A file dropped on the window: documents open, anything else is placed.
+    /// A file dropped on the window: documents open, the swatches in a swatch exchange (.ase) file
+    /// are loaded, and anything else is placed.
     pub fn open_dropped(&mut self, path: &str) -> Result<Value, String> {
+        if path.to_ascii_lowercase().ends_with(".ase") {
+            return self.run("swatch.load", json!({"path": path}));
+        }
         let cmd = if opens_as_document(path) { "file.open" } else { "file.place" };
         self.open_file(cmd, json!({"path": path}))
     }
@@ -1888,6 +1892,28 @@ mod tests {
         assert_eq!(alert(&app), ("Can't Place the File".to_string(), e));
     }
 
+    #[test]
+    fn dropping_an_ase_file_loads_its_swatches() {
+        let mut source = Session::new();
+        source.execute("file.new", &json!({})).unwrap();
+        source.execute("swatch.create", &json!({"name": "Dropped Teal", "color": "#108080"})).unwrap();
+        let data = source.execute("swatch.save", &json!({"names": ["Dropped Teal"]})).unwrap();
+        let bytes = designcraft_engine::cmd::base64_decode(data["base64"].as_str().unwrap());
+        for name in ["palette.ase", "PALETTE.ASE"] {
+            let path = std::env::temp_dir().join(format!("designcraft-ui-{}-{name}", std::process::id()));
+            std::fs::write(&path, &bytes).unwrap();
+            let mut app = DesignApp::new(Session::new(), Services::default());
+            app.run("file.new", json!({})).unwrap();
+            let r = app.open_dropped(&path.to_string_lossy());
+            let _ = std::fs::remove_file(&path);
+            assert!(app.ui.dialog.is_none(), "{name}: {:?}", app.ui.dialog.as_ref().map(|d| &d.fields));
+            assert_eq!(r, Ok(json!({"added": 1, "replaced": 0})), "{name}");
+            let doc = &app.session.doc().unwrap().doc;
+            assert!(doc.swatches.iter().any(|s| s.name == "Dropped Teal"), "{name}");
+            assert!(doc.assets.is_empty(), "{name} isn't placed");
+        }
+    }
+
     #[test]
     fn file_open_failures_show_an_alert() {
         let path = unreadable_file("open.designcraft");