From fea917c8f2a8e9464ab5625f29f06fd622a26a04 Mon Sep 17 00:00:00 2001
From: Adhyan <adhyanpatel@gmail.com>
Date: Tue, 3 Sep 2024 14:52:09 -0600
Subject: [PATCH] removed unnecessary update_rows() call and minor refactoring

---
 src/list.rs       |  6 ++----
 src/list/state.rs | 14 ++++++--------
 2 files changed, 8 insertions(+), 12 deletions(-)

diff --git a/src/list.rs b/src/list.rs
index dc224f00..857d0ce9 100644
--- a/src/list.rs
+++ b/src/list.rs
@@ -67,9 +67,7 @@ fn handle_list(app_state: &mut AppState, stdout: &mut StdoutLock) -> Result<()>
                 }
 
                 match key.code {
-                    KeyCode::Char('q') => {
-                        return Ok(());
-                    }
+                    KeyCode::Char('q') => return Ok(()),
                     KeyCode::Down | KeyCode::Char('j') => list_state.select_next(),
                     KeyCode::Up | KeyCode::Char('k') => list_state.select_previous(),
                     KeyCode::Home | KeyCode::Char('g') => list_state.select_first(),
@@ -102,7 +100,7 @@ fn handle_list(app_state: &mut AppState, stdout: &mut StdoutLock) -> Result<()>
                             return Ok(());
                         }
                     }
-                    KeyCode::Char('s') | KeyCode::Char('/') => {
+                    KeyCode::Char('s' | '/') => {
                         list_state.message.push_str("search:|");
                         is_searching = true;
                     }
diff --git a/src/list/state.rs b/src/list/state.rs
index 8d5bf5b8..f1507667 100644
--- a/src/list/state.rs
+++ b/src/list/state.rs
@@ -374,23 +374,21 @@ impl<'a> ListState<'a> {
                 }
             })
             .enumerate()
-            .find_map(|(i, s)| {
-                if s.name.contains(self.search_query.as_str()) {
-                    Some(i)
+            .find_map(|(idx, exercise)| {
+                if exercise.name.contains(self.search_query.as_str()) {
+                    Some(idx)
                 } else {
                     None
                 }
             });
 
         match idx {
-            Some(i) => {
-                let exercise_ind = i;
+            Some(x) => {
+                let exercise_ind = x;
                 self.scroll_state.set_selected(exercise_ind);
-                self.update_rows();
             }
             None => {
-                let msg = String::from("[NOT FOUND]") + &self.message.clone();
-                self.message.clear();
+                let msg = String::from(" (not found)");
                 self.message.push_str(&msg);
             }
         }