on master help sign in
fix: ui/menu: Don't allow scrolling past the end of completion
1789dfaBlaž Hrastnik
helix-term/src/ui/menu.rs+2−2
@@ -90,13 +90,13 @@ impl<T: Item> Menu<T> {
90
90
91 pub fn move_up(&mut self) {
91 pub fn move_up(&mut self) {
92 // TODO: wrap around to end
92 // TODO: wrap around to end
93− let pos = self.cursor.map_or(0, |i| i.saturating_sub(1)) % self.options.len();
93+ let pos = self.cursor.map_or(0, |i| i.saturating_sub(1)) % self.matches.len();
94 self.cursor = Some(pos);
94 self.cursor = Some(pos);
95 self.adjust_scroll();
95 self.adjust_scroll();
96 }
96 }
97
97
98 pub fn move_down(&mut self) {
98 pub fn move_down(&mut self) {
99− let pos = self.cursor.map_or(0, |i| i + 1) % self.options.len();
99+ let pos = self.cursor.map_or(0, |i| i + 1) % self.matches.len();
100 self.cursor = Some(pos);
100 self.cursor = Some(pos);
101 self.adjust_scroll();
101 self.adjust_scroll();
102 }
102 }