| author |  clover caruso <git@paperclover.net> 2026-10-02 22:18:19-07:00 |
| committer | clover caruso <git@paperclover.net> 2026-10-03 01:50:33-07:00 |
| log | 1b252f294ce7272a0b316aed8f1431b5e0b37af6 |
| tree | 2908e87df2ef4a0f2f35fb2587652648a386eebd |
| parent | a4b935962eea65ff06dd374addd162164c932203 |
| signature | Signed by SSH key SHA256:52mNGHRsVFBDED9IAX5pe+LRWUefqTbxEReunq21QvU |
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.52 files changed, 5 insertions(+), 7 deletions(-)
crates/snowbound/src/menus.rs+4-6
| ... | @@ -817,9 +817,10 @@ impl State { | ... | @@ -817,9 +817,10 @@ impl State { |
| 817 | None | 817 | None |
| 818 | } | 818 | } |
| 819 | (Target::Section { library, path }, Action::Delete) => { | 819 | (Target::Section { library, path }, Action::Delete) => { |
| 820 | let name = crate::library::entry_name(&path).to_owned(); | ||
| 820 | platform::confirm( | 821 | platform::confirm( |
| 821 | "Are you sure you want to move this section to this notebook's Recycle Bin?", | 822 | "Are you sure you want to move this section to this notebook's Recycle Bin?", |
| 822 | &path.clone(), | 823 | &name, |
| 823 | "Cancel", | 824 | "Cancel", |
| 824 | "Delete", | 825 | "Delete", |
| 825 | self.reply(|state, ()| { | 826 | self.reply(|state, ()| { |
| ... | @@ -831,13 +832,10 @@ impl State { | ... | @@ -831,13 +832,10 @@ impl State { |
| 831 | None | 832 | None |
| 832 | } | 833 | } |
| 833 | (Target::Group { library, path }, Action::Delete) => { | 834 | (Target::Group { library, path }, Action::Delete) => { |
| 835 | let name = crate::library::entry_name(&path).to_owned(); | ||
| 834 | platform::confirm( | 836 | platform::confirm( |
| 835 | "Are you sure you want to move the sections in this section group to this notebook's Recycle Bin?", | 837 | "Are you sure you want to move the sections in this section group to this notebook's Recycle Bin?", |
| 836 | path.rsplit('/') | 838 | &name, |
| 837 | .next() | ||
| 838 | .unwrap_or_default() | ||
| 839 | .to_owned() | ||
| 840 | .as_str(), | ||
| 841 | "Cancel", | 839 | "Cancel", |
| 842 | "Delete", | 840 | "Delete", |
| 843 | self.reply(|state, ()| { | 841 | self.reply(|state, ()| { |
crates/snowbound/src/update.rs+1-1
| ... | @@ -755,7 +755,7 @@ impl State { | ... | @@ -755,7 +755,7 @@ impl State { |
| 755 | "Snowbound is up to date", | 755 | "Snowbound is up to date", |
| 756 | &format!("You have {}.", describe_running()), | 756 | &format!("You have {}.", describe_running()), |
| 757 | ), | 757 | ), |
| 758 | Status::Failed(reason) => platform::alert("Unable to check for updates", reason), | 758 | Status::Failed(reason) => platform::alert("Couldn't check for updates", reason), |
| 759 | Status::Idle | Status::Downloading(_) => {} | 759 | Status::Idle | Status::Downloading(_) => {} |
| 760 | } | 760 | } |
| 761 | } | 761 | } |