From 1b252f294ce7272a0b316aed8f1431b5e0b37af6 Mon Sep 17 00:00:00 2001 From: clover caruso Date: Fri, 2 Oct 2026 22:18:19 -0700 Subject: [PATCH] fix: delete confirmations name the section without its file's extension, and alerts say Couldn't Deleting a section asked about Garden.one; it names Garden, as the group's already named the group. Check for Updates' failure read "Unable to check for updates" where every other alert says Couldn't. Assisted-by: claude-opus-5.5 --- crates/snowbound/src/menus.rs | 10 ++++------ crates/snowbound/src/update.rs | 2 +- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/crates/snowbound/src/menus.rs b/crates/snowbound/src/menus.rs index c3331e2855fe34a5d85e3c0fef1d7a01f33d768e..3face8fe9d207b91999ce2aac19ec2b32bfa586d 100644 --- a/crates/snowbound/src/menus.rs +++ b/crates/snowbound/src/menus.rs @@ -817,9 +817,10 @@ impl State { None } (Target::Section { library, path }, Action::Delete) => { + let name = crate::library::entry_name(&path).to_owned(); platform::confirm( "Are you sure you want to move this section to this notebook's Recycle Bin?", - &path.clone(), + &name, "Cancel", "Delete", self.reply(|state, ()| { @@ -831,13 +832,10 @@ impl State { None } (Target::Group { library, path }, Action::Delete) => { + let name = crate::library::entry_name(&path).to_owned(); platform::confirm( "Are you sure you want to move the sections in this section group to this notebook's Recycle Bin?", - path.rsplit('/') - .next() - .unwrap_or_default() - .to_owned() - .as_str(), + &name, "Cancel", "Delete", self.reply(|state, ()| { diff --git a/crates/snowbound/src/update.rs b/crates/snowbound/src/update.rs index 8f855d662323443d1989d2f9578c1b10cbb07108..fc19bf4d14ba5bea4b82e4b1d6eebd5187b85000 100644 --- a/crates/snowbound/src/update.rs +++ b/crates/snowbound/src/update.rs @@ -755,7 +755,7 @@ impl State { "Snowbound is up to date", &format!("You have {}.", describe_running()), ), - Status::Failed(reason) => platform::alert("Unable to check for updates", reason), + Status::Failed(reason) => platform::alert("Couldn't check for updates", reason), Status::Idle | Status::Downloading(_) => {} } } -- 2.54.0