Hide mtime column by default, unless enabled

Byron · Nov 24, 2023 · 0f8377a · 0f8377a
1 parent 2bd06be
commit 0f8377a
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/src/interactive/widgets/entries.rs b/src/interactive/widgets/entries.rs
@@ -186,7 +186,12 @@ impl Entries {
                         Style { fg, ..style }
                     },
                 );
-                vec![mtime, bar, bytes, left_bar, percentage, right_bar, name]
+
+                if should_show_mtime_column(sort_mode) {
+                    vec![mtime, bar, bytes, left_bar, percentage, right_bar, name]
+                } else {
+                    vec![bytes, left_bar, percentage, right_bar, name]
+                }
             },
         );
 
@@ -221,3 +226,10 @@ impl Entries {
         }
     }
 }
+
+fn should_show_mtime_column(sort_mode: &SortMode) -> bool {
+    match sort_mode {
+        SortMode::MTimeAscending | SortMode::MTimeDescending => true,
+        _ => false,
+    }
+}
diff --git a/src/traverse.rs b/src/traverse.rs
@@ -266,7 +266,7 @@ mod tests {
     fn size_of_entry_data() {
         assert_eq!(
             std::mem::size_of::<EntryData>(),
-            if cfg!(windows) { 56 } else { 48 },
+            if cfg!(windows) { 72 } else { 64 },
             "the size of this should not change unexpectedly as it affects overall memory consumption"
         );
     }