From d3577bd1ab5e90d20d9baf6029b0bd0b7f61911d Mon Sep 17 00:00:00 2001
From: Clement Tsang <34804052+ClementTsang@users.noreply.github.com>
Date: Sat, 3 Dec 2022 01:18:47 -0500
Subject: [PATCH] other: don't draw on non-updating events (#924)

---
 src/bin/main.rs | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/bin/main.rs b/src/bin/main.rs
index 84b91828..f5e2371f 100644
--- a/src/bin/main.rs
+++ b/src/bin/main.rs
@@ -162,14 +162,17 @@ fn main() -> Result<()> {
                         break;
                     }
                     update_data(&mut app);
+                    try_drawing(&mut terminal, &mut app, &mut painter)?;
                 }
                 BottomEvent::MouseInput(event) => {
                     handle_mouse_event(event, &mut app);
                     update_data(&mut app);
+                    try_drawing(&mut terminal, &mut app, &mut painter)?;
                 }
                 BottomEvent::PasteEvent(paste) => {
                     app.handle_paste(paste);
                     update_data(&mut app);
+                    try_drawing(&mut terminal, &mut app, &mut painter)?;
                 }
                 BottomEvent::Update(data) => {
                     app.data_collection.eat_data(data);
@@ -278,6 +281,7 @@ fn main() -> Result<()> {
                         }
 
                         update_data(&mut app);
+                        try_drawing(&mut terminal, &mut app, &mut painter)?;
                     }
                 }
                 BottomEvent::Clean => {
@@ -286,9 +290,6 @@ fn main() -> Result<()> {
                 }
             }
         }
-
-        // TODO: [OPT] Should not draw if no change (ie: scroll max)
-        try_drawing(&mut terminal, &mut app, &mut painter)?;
     }
 
     // I think doing it in this order is safe...