Merge pull request #282 from tbergerd/dev

fix #281 : Select : prompt is no longer displayed
console-rs · Sep 21, 2023 · c5b4084 · c5b4084
2 parents 7aa4afa + 339693b
commit c5b4084
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/src/paging.rs b/src/paging.rs
@@ -42,6 +42,15 @@ impl<'a> Paging<'a> {
         }
     }
 
+    pub fn update_page(&mut self, cursor_pos: usize) {
+        if cursor_pos != !0
+            && (cursor_pos < self.current_page * self.capacity
+                || cursor_pos >= (self.current_page + 1) * self.capacity)
+        {
+            self.current_page = cursor_pos / self.capacity;
+        }
+    }
+
     /// Updates all internal based on the current terminal size and cursor position
     pub fn update(&mut self, cursor_pos: usize) -> Result {
         let new_term_size = self.term.size();
@@ -65,12 +74,7 @@ impl<'a> Paging<'a> {
             self.term.clear_last_lines(self.capacity)?;
         }
 
-        if cursor_pos != !0
-            && (cursor_pos < self.current_page * self.capacity
-                || cursor_pos >= (self.current_page + 1) * self.capacity)
-        {
-            self.current_page = cursor_pos / self.capacity;
-        }
+        self.update_page(cursor_pos);
 
         Ok(())
     }

diff --git a/src/prompts/select.rs b/src/prompts/select.rs
@@ -214,7 +214,7 @@ impl Select<'_> {
         }
 
         term.hide_cursor()?;
-        paging.update(sel)?;
+        paging.update_page(sel);
 
         loop {
             if let Some(ref prompt) = self.prompt {