From 2b41e36362aed743ac81eab710a37a189725e499 Mon Sep 17 00:00:00 2001 From: electria Date: Thu, 27 Aug 2026 10:31:34 -0700 Subject: [PATCH] fix: remove edge-case wrapping_add left this on accident when I decided wrapping around isn't useful (especially without it scrolling to show the selection) --- src/app/mod.rs | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/app/mod.rs b/src/app/mod.rs index d08935a..29e61bc 100644 --- a/src/app/mod.rs +++ b/src/app/mod.rs @@ -127,12 +127,8 @@ impl State { } Message::SelectionLeft => self.selected_field.previous(), Message::SelectionDown => { - self.selected_entry = if self.selected_entry == self.db_view.len() - 1 { - // essentially do a wrapping_add up to the highest index in the view - 0 - } else { + self.selected_entry = (self.selected_entry.saturating_add(1)).clamp(0, self.db_view.len() - 1) - } } Message::SelectionUp => { self.selected_entry =