From b1771b4c42ffb9bead365c5adf556fc0c4e6ccaf Mon Sep 17 00:00:00 2001
From: Jonatan Pettersson <jonatan.pettersson@proton.me>
Date: Fri, 26 Dec 2025 09:56:32 +0100
Subject: [PATCH] refactor: simplify mime type limit logic

---
 src/tab.rs | 23 +++++++----------------
 1 file changed, 7 insertions(+), 16 deletions(-)

diff --git a/src/tab.rs b/src/tab.rs
index 9c48d76..8f0344f 100644
--- a/src/tab.rs
+++ b/src/tab.rs
@@ -5941,29 +5941,20 @@ impl Tab {
         mime_types.sort_by(|(_, v1), (_, v2)| v2.cmp(v1));
 
         // Limit the number of displayed mime types
-        mime_types.truncate(10);
+        let limit = usize::min(10, mime_types.len());
 
-        let mut mime_types_total: u64 = 0;
-
-        let mut mime_types: Vec<String> = mime_types
-            .into_iter()
-            .map(|(mime, count)| {
-                mime_types_total += count;
-                format!("{} ({})", mime, count)
-            })
+        let mut mime_type_strings: Vec<String> = mime_types[..limit]
+            .iter()
+            .map(|(mime, count)| format!("{} ({})", mime, count))
             .collect();
 
-        if selected_items
-            .len()
-            .saturating_sub(mime_types_total as usize)
-            > 0
-        {
-            mime_types.push(format!("..."));
+        if mime_types.len() > limit {
+            mime_type_strings.push("...".to_string());
         }
 
         details = details.push(widget::text::body(fl!(
             "type",
-            mime = mime_types.join(", ")
+            mime = mime_type_strings.join(", ")
         )));
 
         let size = {