lostecho/ncspot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: don't show error if user moves over list edge

Browse Source 
Ignore this error, as it's common user behavior and an error is distracting.
This commit is contained in:
Henrik Friedrichsen
2022-01-14 19:01:56 +01:00
parent 211ce66cbd
commit e5d51d63a3
1 changed files with 15 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
src/ui/listview.rs
View File

@@ -463,23 +463,27 @@ impl<I: ListItem + Clone> ViewExt for ListView<I> {
let last_idx = self.content.read().unwrap().len().saturating_sub(1);
match mode {
MoveMode::Up if self.selected > 0 => {
match amount {
MoveAmount::Extreme => self.move_focus_to(0),
MoveAmount::Integer(amount) => self.move_focus(-(*amount)),
MoveMode::Up => {
if self.selected > 0 {
match amount {
MoveAmount::Extreme => self.move_focus_to(0),
MoveAmount::Integer(amount) => self.move_focus(-(*amount)),
}
}
return Ok(CommandResult::Consumed(None));
}
MoveMode::Down if self.selected < last_idx => {
match amount {
MoveAmount::Extreme => self.move_focus_to(last_idx),
MoveAmount::Integer(amount) => self.move_focus(*amount),
MoveMode::Down => {
if self.selected < last_idx {
match amount {
MoveAmount::Extreme => self.move_focus_to(last_idx),
MoveAmount::Integer(amount) => self.move_focus(*amount),
}
}
if self.selected == last_idx && self.can_paginate() {
self.pagination.call(&self.content, self.library.clone());
}
return Ok(CommandResult::Consumed(None));
}
MoveMode::Down if self.selected == last_idx && self.can_paginate() => {
self.pagination.call(&self.content, self.library.clone());
}
_ => {}
}
}