mitsuo_makuda/linutil
1
0
Fork 0
mirror of https://github.com/ChrisTitusTech/linutil.git synced 2024-12-25 04:49:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix crash when trying to scroll on empty results

Browse Source
This commit is contained in:
afonsofrancof 2024-07-30 22:49:51 +01:00
parent e85cc943b3
commit b7c2ecedd9
No known key found for this signature in database
1 changed files with 17 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/list.rs
View File

@ -242,6 +242,8 @@ impl CustomList {
pub fn reset_selection(&mut self) { pub fn reset_selection(&mut self) {
if !self.filtered_items.is_empty() { if !self.filtered_items.is_empty() {
self.list_state.select(Some(0)); self.list_state.select(Some(0));
} else {
self.list_state.select(None);
} }
} }
@ -317,8 +319,11 @@ impl CustomList {
} }
fn try_scroll_up(&mut self) { fn try_scroll_up(&mut self) {
self.list_state if let Some(selected) = self.list_state.selected() {
.select(Some(self.list_state.selected().unwrap().saturating_sub(1))); if selected > 0 {
self.list_state.select(Some(selected.saturating_sub(1)));
}
}
} }
fn try_scroll_down(&mut self) { fn try_scroll_down(&mut self) {
@ -332,15 +337,16 @@ impl CustomList {
self.filtered_items.len() self.filtered_items.len()
}; };
let curr_selection = self.list_state.selected().unwrap(); if let Some(curr_selection) = self.list_state.selected() {
if self.at_root() { if self.at_root() {
self.list_state self.list_state
.select(Some((curr_selection + 1).min(count - 1))); .select(Some((curr_selection + 1).min(count - 1)));
} else { } else {
// When we are not at the root, we have to account for 1 more "virtual" node, `..`. So // When we are not at the root, we have to account for 1 more "virtual" node, `..`. So
// the count is 1 bigger (select is 0 based, because it's an index) // the count is 1 bigger (select is 0 based, because it's an index)
self.list_state self.list_state
.select(Some((curr_selection + 1).min(count))); .select(Some((curr_selection + 1).min(count)));
}
} }
} }