fix: Skip padding time if it will overflow the allowed prefix length (#…

…1630)
atuinsh · Jan 26, 2024 · d9c40d6 · d9c40d6
1 parent 2e48e21
commit d9c40d6
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/atuin/src/command/client/search/history_list.rs b/atuin/src/command/client/search/history_list.rs
@@ -164,10 +164,10 @@ impl DrawState<'_> {
         let time = format_duration(since.try_into().unwrap_or_default());
 
         // pad the time a little bit before we write. this aligns things nicely
-        self.draw(
-            &SPACES[..((PREFIX_LENGTH - self.x) as usize - 4 - time.len())],
-            Style::default(),
-        );
+        // skip padding if for some reason it is already too long to align nicely
+        let padding =
+            usize::from(PREFIX_LENGTH).saturating_sub(usize::from(self.x) + 4 + time.len());
+        self.draw(&SPACES[..padding], Style::default());
 
         self.draw(&time, style);
         self.draw(" ago", style);