trigger update only after sending commit

src/app.rs

@@ -137,8 +137,10 @@ impl App {
     ///
     pub fn event(&mut self, ev: Event) {
         trace!("event: {:?}", ev);
-        if self.commit.event(ev) {
-            self.update_diff();
+        if self.commit.is_visible() && self.commit.event(ev) {
+            if !self.commit.is_visible() {
+                self.update();
+            }
             return;
         }
 
@@ -183,6 +185,9 @@ impl App {
                         self.index_reset();
                         self.update();
                     }
+                    keys::OPEN_COMMIT if !self.index.is_empty() => {
+                        self.commit.show();
+                    }
                     _ => (),
                 };
             }

src/components/commit.rs

@@ -60,7 +60,6 @@ impl Component for CommitComponent {
     }
 
     fn event(&mut self, ev: Event) -> bool {
-        if self.visible {
         if let Event::Key(e) = ev {
             return match e.code {
                 KeyCode::Esc => {
@@ -82,14 +81,6 @@ impl Component for CommitComponent {
                 _ => false,
             };
         }
-        } else {
-            if ev == Event::Key(KeyCode::Char('c').into()) {
-                if !git_utils::index_empty() {
-                    self.show();
-                    return true;
-                }
-            }
-        }
 
         false
     }

src/git_utils.rs

@@ -1,7 +1,6 @@
 use git2::{
     build::CheckoutBuilder, DiffFormat, DiffOptions, IndexAddOption,
-    ObjectType, Repository, RepositoryOpenFlags, StatusOptions,
-    StatusShow,
+    ObjectType, Repository, RepositoryOpenFlags,
 };
 use scopetime::scope_time;
 use std::path::Path;
@@ -134,21 +133,6 @@ pub fn commit(msg: &String) {
     .unwrap();
 }
 
-///
-pub fn index_empty() -> bool {
-    scope_time!("index_empty");
-
-    let repo = repo();
-
-    let statuses = repo
-        .statuses(Some(
-            StatusOptions::default().show(StatusShow::Index),
-        ))
-        .unwrap();
-
-    statuses.is_empty()
-}
-
 pub fn stage_add(path: &Path) -> bool {
     scope_time!("stage_add");
 

src/keys.rs

@@ -15,3 +15,4 @@ pub const STATUS_RESET_FILE: KeyEvent = no_mod(KeyCode::Char('D'));
 pub const STATUS_STAGE_FILE: KeyEvent = no_mod(KeyCode::Enter);
 pub const EXIT_1: KeyEvent = no_mod(KeyCode::Esc);
 pub const EXIT_2: KeyEvent = no_mod(KeyCode::Char('q'));
+pub const OPEN_COMMIT: KeyEvent = no_mod(KeyCode::Char('c'));