From 8f16265107c49b7dde6c8c0b8f2362f1183a3467 Mon Sep 17 00:00:00 2001
From: spinline <bilal.karatatr.@gmail.com>
Date: Tue, 3 Feb 2026 22:30:43 +0300
Subject: [PATCH] chore(frontend): remove unused imports and variables

---
 frontend/src/components/layout/statusbar.rs |  23 ++-
 frontend/src/components/torrent/table.rs    | 187 ++++++++++++--------
 2 files changed, 121 insertions(+), 89 deletions(-)

diff --git a/frontend/src/components/layout/statusbar.rs b/frontend/src/components/layout/statusbar.rs
index 369ba09..296eaeb 100644
--- a/frontend/src/components/layout/statusbar.rs
+++ b/frontend/src/components/layout/statusbar.rs
@@ -1,5 +1,4 @@
 use leptos::*;
-use wasm_bindgen::JsCast;
 
 #[component]
 pub fn StatusBar() -> impl IntoView {
@@ -34,10 +33,10 @@ pub fn StatusBar() -> impl IntoView {
                         base.to_string()
                     }
                 }>
-                    <div 
-                        tabindex="0" 
-                        role="button" 
-                        class="btn btn-ghost btn-xs btn-square" 
+                    <div
+                        tabindex="0"
+                        role="button"
+                        class="btn btn-ghost btn-xs btn-square"
                         title="Change Theme"
                         on:click=move |_| set_theme_open.update(|v| *v = !*v)
                     >
@@ -49,8 +48,8 @@ pub fn StatusBar() -> impl IntoView {
                     <Show when=move || theme_open.get() fallback=|| ()>
                         // Backdrop to close on outside click
                         // iOS Safari requires cursor:pointer inline style for click events on div elements
-                        <div 
-                            class="fixed inset-0 z-[99] bg-black/0" 
+                        <div
+                            class="fixed inset-0 z-[99] bg-black/0"
                             style="cursor: pointer; -webkit-tap-highlight-color: transparent;"
                             role="button"
                             tabindex="-1"
@@ -65,19 +64,19 @@ pub fn StatusBar() -> impl IntoView {
                     <ul tabindex="0" class="dropdown-content z-[100] menu p-2 shadow bg-base-200 rounded-box w-52 mb-2 border border-base-300 max-h-96 overflow-y-auto block">
                         {
                             let themes = vec![
-                                "light", "dark", "cupcake", "dracula", "cyberpunk", 
-                                "emerald", "luxury", "nord", "sunset", "winter", 
+                                "light", "dark", "cupcake", "dracula", "cyberpunk",
+                                "emerald", "luxury", "nord", "sunset", "winter",
                                 "night", "synthwave", "retro", "forest"
                             ];
                             themes.into_iter().map(|theme| {
                                 view! {
                                     <li>
-                                        <button 
+                                        <button
                                             class="text-xs capitalize"
                                             on:click=move |_| {
                                                 let doc = web_sys::window().unwrap().document().unwrap();
                                                 let _ = doc.document_element().unwrap().set_attribute("data-theme", theme);
-                                                
+
                                                 if let Some(meta) = doc.query_selector("meta[name='theme-color']").unwrap() {
                                                     let window = web_sys::window().unwrap();
                                                     if let Ok(Some(style)) = window.get_computed_style(&doc.body().unwrap()) {
@@ -98,7 +97,7 @@ pub fn StatusBar() -> impl IntoView {
                         }
                     </ul>
                 </div>
-                
+
                  <button class="btn btn-ghost btn-xs btn-square" title="Settings">
                      <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="w-4 h-4">
                         <path stroke-linecap="round" stroke-linejoin="round" d="M9.594 3.94c.09-.542.56-.94 1.11-.94h2.593c.55 0 1.02.398 1.11.94l.213 1.281c.063.374.313.686.645.87.074.04.147.083.22.127.324.196.72.257 1.075.124l1.217-.456a1.125 1.125 0 011.37.49l1.296 2.247a1.125 1.125 0 01-.26 1.431l-1.003.827c-.293.24-.438.613-.431.992a6.759 6.759 0 010 .255c-.007.378.138.75.43.99l1.005.828c.424.35.534.954.26 1.43l-1.298 2.247a1.125 1.125 0 01-1.369.491l-1.217-.456c-.355-.133-.75-.072-1.076.124a6.57 6.57 0 01-.22.128c-.331.183-.581.495-.644.869l-.212 1.28c-.09.543-.56.941-1.11.941h-2.594c-.55 0-1.02-.398-1.11-.94l-.213-1.281c-.062-.374-.312-.686-.644-.87a6.52 6.52 0 01-.22-.127c-.325-.196-.72-.257-1.076-.124l-1.217.456a1.125 1.125 0 01-1.369-.49l-1.297-2.247a1.125 1.125 0 01.26-1.431l1.004-.827c.292-.24.437-.613.43-.992a6.932 6.932 0 010-.255c.007-.378-.138-.75-.43-.99l-1.004-.828a1.125 1.125 0 01-.26-1.43l1.297-2.247a1.125 1.125 0 011.37-.491l1.216.456c.356.133.751.072 1.076-.124.072-.044.146-.087.22-.128.332-.183.582-.495.644-.869l.214-1.281z" />
diff --git a/frontend/src/components/torrent/table.rs b/frontend/src/components/torrent/table.rs
index 3abe579..1ee9147 100644
--- a/frontend/src/components/torrent/table.rs
+++ b/frontend/src/components/torrent/table.rs
@@ -2,16 +2,17 @@ use leptos::*;
 use wasm_bindgen::closure::Closure;
 use wasm_bindgen::JsCast;
 
-
-
-
 fn format_bytes(bytes: i64) -> String {
     const UNITS: [&str; 6] = ["B", "KB", "MB", "GB", "TB", "PB"];
     if bytes < 1024 {
         return format!("{} B", bytes);
     }
     let i = (bytes as f64).log2().div_euclid(10.0) as usize;
-    format!("{:.1} {}", (bytes as f64) / 1024_f64.powi(i as i32), UNITS[i])
+    format!(
+        "{:.1} {}",
+        (bytes as f64) / 1024_f64.powi(i as i32),
+        UNITS[i]
+    )
 }
 
 fn format_speed(bytes_per_sec: i64) -> String {
@@ -41,32 +42,47 @@ enum SortDirection {
 #[component]
 pub fn TorrentTable() -> impl IntoView {
     let store = use_context::<crate::store::TorrentStore>().expect("store not provided");
-    
+
     let sort_col = create_rw_signal(SortColumn::Name);
     let sort_dir = create_rw_signal(SortDirection::Ascending);
 
     let filtered_torrents = move || {
-        let mut torrents = store.torrents.get().into_iter().filter(|t| {
-            let filter = store.filter.get();
-            let search = store.search_query.get().to_lowercase();
-            
-            let matches_filter = match filter {
-                crate::store::FilterStatus::All => true,
-                crate::store::FilterStatus::Downloading => t.status == shared::TorrentStatus::Downloading,
-                crate::store::FilterStatus::Seeding => t.status == shared::TorrentStatus::Seeding,
-                crate::store::FilterStatus::Completed => t.status == shared::TorrentStatus::Seeding || t.status == shared::TorrentStatus::Paused, // Approximate
-                crate::store::FilterStatus::Inactive => t.status == shared::TorrentStatus::Paused || t.status == shared::TorrentStatus::Error,
-                _ => true
-            };
+        let mut torrents = store
+            .torrents
+            .get()
+            .into_iter()
+            .filter(|t| {
+                let filter = store.filter.get();
+                let search = store.search_query.get().to_lowercase();
 
-            let matches_search = if search.is_empty() {
-                true
-            } else {
-                t.name.to_lowercase().contains(&search)
-            };
+                let matches_filter = match filter {
+                    crate::store::FilterStatus::All => true,
+                    crate::store::FilterStatus::Downloading => {
+                        t.status == shared::TorrentStatus::Downloading
+                    }
+                    crate::store::FilterStatus::Seeding => {
+                        t.status == shared::TorrentStatus::Seeding
+                    }
+                    crate::store::FilterStatus::Completed => {
+                        t.status == shared::TorrentStatus::Seeding
+                            || t.status == shared::TorrentStatus::Paused
+                    } // Approximate
+                    crate::store::FilterStatus::Inactive => {
+                        t.status == shared::TorrentStatus::Paused
+                            || t.status == shared::TorrentStatus::Error
+                    }
+                    _ => true,
+                };
 
-            matches_filter && matches_search
-        }).collect::<Vec<_>>();
+                let matches_search = if search.is_empty() {
+                    true
+                } else {
+                    t.name.to_lowercase().contains(&search)
+                };
+
+                matches_filter && matches_search
+            })
+            .collect::<Vec<_>>();
 
         torrents.sort_by(|a, b| {
             let col = sort_col.get();
@@ -74,32 +90,41 @@ pub fn TorrentTable() -> impl IntoView {
             let cmp = match col {
                 SortColumn::Name => a.name.to_lowercase().cmp(&b.name.to_lowercase()),
                 SortColumn::Size => a.size.cmp(&b.size),
-                SortColumn::Progress => a.percent_complete.partial_cmp(&b.percent_complete).unwrap_or(std::cmp::Ordering::Equal),
+                SortColumn::Progress => a
+                    .percent_complete
+                    .partial_cmp(&b.percent_complete)
+                    .unwrap_or(std::cmp::Ordering::Equal),
                 SortColumn::Status => format!("{:?}", a.status).cmp(&format!("{:?}", b.status)),
                 SortColumn::DownSpeed => a.down_rate.cmp(&b.down_rate),
                 SortColumn::UpSpeed => a.up_rate.cmp(&b.up_rate),
                 SortColumn::ETA => {
-                     // ETA 0 means infinity usually, so we need to handle it.
-                     // But for simple sorting, maybe just treat is numeric?
-                     // Let's treat 0 as MAX for ascending, MIN for descending? Or just as is?
-                     // Usually negative or 0 means unknown/inf.
-                     // Let's handle 0 as very large number for sorting purposes if we want it at the end of ascending
-                     let a_eta = if a.eta <= 0 { i64::MAX } else { a.eta };
-                     let b_eta = if b.eta <= 0 { i64::MAX } else { b.eta };
-                     a_eta.cmp(&b_eta)
+                    // ETA 0 means infinity usually, so we need to handle it.
+                    // But for simple sorting, maybe just treat is numeric?
+                    // Let's treat 0 as MAX for ascending, MIN for descending? Or just as is?
+                    // Usually negative or 0 means unknown/inf.
+                    // Let's handle 0 as very large number for sorting purposes if we want it at the end of ascending
+                    let a_eta = if a.eta <= 0 { i64::MAX } else { a.eta };
+                    let b_eta = if b.eta <= 0 { i64::MAX } else { b.eta };
+                    a_eta.cmp(&b_eta)
                 }
             };
-            if dir == SortDirection::Descending { cmp.reverse() } else { cmp }
+            if dir == SortDirection::Descending {
+                cmp.reverse()
+            } else {
+                cmp
+            }
         });
-        
+
         torrents
     };
 
     let handle_sort = move |col: SortColumn| {
         if sort_col.get() == col {
-            sort_dir.update(|d| *d = match d {
-                SortDirection::Ascending => SortDirection::Descending,
-                SortDirection::Descending => SortDirection::Ascending,
+            sort_dir.update(|d| {
+                *d = match d {
+                    SortDirection::Ascending => SortDirection::Descending,
+                    SortDirection::Descending => SortDirection::Ascending,
+                }
             });
         } else {
             sort_col.set(col);
@@ -110,11 +135,16 @@ pub fn TorrentTable() -> impl IntoView {
     let sort_arrow = move |col: SortColumn| {
         if sort_col.get() == col {
             match sort_dir.get() {
-                SortDirection::Ascending => view!{ <span class="ml-1 text-xs">"▲"</span> }.into_view(),
-                SortDirection::Descending => view!{ <span class="ml-1 text-xs">"▼"</span> }.into_view(),
+                SortDirection::Ascending => {
+                    view! { <span class="ml-1 text-xs">"▲"</span> }.into_view()
+                }
+                SortDirection::Descending => {
+                    view! { <span class="ml-1 text-xs">"▼"</span> }.into_view()
+                }
             }
         } else {
-            view!{ <span class="ml-1 text-xs opacity-0 group-hover:opacity-50">"▲"</span> }.into_view()
+            view! { <span class="ml-1 text-xs opacity-0 group-hover:opacity-50">"▲"</span> }
+                .into_view()
         }
     };
 
@@ -134,29 +164,34 @@ pub fn TorrentTable() -> impl IntoView {
         set_menu_visible.set(false); // Close menu immediately
 
         spawn_local(async move {
-            let action_req =  if action == "delete_with_data" { "delete_with_data" } else { &action };
-            
+            let action_req = if action == "delete_with_data" {
+                "delete_with_data"
+            } else {
+                &action
+            };
+
             let req_body = shared::TorrentActionRequest {
                 hash: hash.clone(),
                 action: action_req.to_string(),
             };
 
-            let client = gloo_net::http::Request::post("/api/torrents/action")
-                .json(&req_body);
+            let client = gloo_net::http::Request::post("/api/torrents/action").json(&req_body);
 
             match client {
-                Ok(req) => {
-                    match req.send().await {
-                        Ok(resp) => {
-                            if !resp.ok() {
-                                logging::error!("Failed to execute action: {} {}", resp.status(), resp.status_text());
-                            } else {
-                                logging::log!("Action {} executed successfully", action);
-                            }
+                Ok(req) => match req.send().await {
+                    Ok(resp) => {
+                        if !resp.ok() {
+                            logging::error!(
+                                "Failed to execute action: {} {}",
+                                resp.status(),
+                                resp.status_text()
+                            );
+                        } else {
+                            logging::log!("Action {} executed successfully", action);
                         }
-                        Err(e) => logging::error!("Network error executing action: {}", e),
                     }
-                }
+                    Err(e) => logging::error!("Network error executing action: {}", e),
+                },
                 Err(e) => logging::error!("Failed to serialize request: {}", e),
             }
         });
@@ -202,17 +237,17 @@ pub fn TorrentTable() -> impl IntoView {
                                 shared::TorrentStatus::Error => "text-error",
                                 _ => "text-base-content/50"
                             };
-                            let t_hash = t.hash.clone(); 
+                            let t_hash = t.hash.clone();
                             let t_hash_click = t.hash.clone();
-                            
+
                             let is_selected_fn = move || {
                                 selected_hash.get() == Some(t_hash.clone())
                             };
 
                             view! {
-                                <tr 
+                                <tr
                                     class=move || {
-                                        let base = "hover border-b border-base-200 select-none"; 
+                                        let base = "hover border-b border-base-200 select-none";
                                         if is_selected_fn() {
                                             format!("{} bg-primary/10", base)
                                         } else {
@@ -262,16 +297,14 @@ pub fn TorrentTable() -> impl IntoView {
                     };
                     let t_hash = t.hash.clone();
                     // We don't need t_hash_click separately if we use t_hash, but existing pattern uses clones
-                    let t_hash_click = t.hash.clone(); 
-                    
-                    let is_selected_fn = move || {
-                        selected_hash.get() == Some(t_hash.clone())
-                    };
+                    let t_hash_click = t.hash.clone();
+
+
 
                     // Long press logic
                     let (timer_id, set_timer_id) = create_signal(Option::<i32>::None);
                     let t_hash_long = t.hash.clone();
-                    
+
                     let clear_timer = move || {
                         if let Some(id) = timer_id.get_untracked() {
                             window().clear_timeout_with_handle(id);
@@ -282,38 +315,38 @@ pub fn TorrentTable() -> impl IntoView {
                     let handle_touchstart = {
                          let t_hash = t_hash_long.clone();
                          move |e: web_sys::TouchEvent| {
-                            // Don't prevent default immediately, or we can't scroll. 
+                            // Don't prevent default immediately, or we can't scroll.
                             // But for long press, we might need to if we want to stop iOS menu.
                             // -webkit-touch-callout: none (in CSS) handles the iOS menu suppression usually.
-                            
+
                             clear_timer();
-                            
+
                             if let Some(touch) = e.touches().get(0) {
                                 let x = touch.client_x();
                                 let y = touch.client_y();
                                 let hash = t_hash.clone();
-                                
+
                                 let closure = Closure::wrap(Box::new(move || {
                                     set_menu_position.set((x, y));
                                     set_selected_hash.set(Some(hash.clone()));
                                     set_menu_visible.set(true);
-                                    
+
                                     // Haptic feedback if available
                                     let navigator = window().navigator();
                                     let _ = navigator.vibrate_with_duration(50);
                                 }) as Box<dyn Fn()>);
-                                
+
                                 let id = window()
                                     .set_timeout_with_callback_and_timeout_and_arguments_0(
                                         closure.as_ref().unchecked_ref(),
                                         600 // 600ms long press
                                     )
                                     .unwrap_or(0);
-                                
+
                                 closure.forget(); // Leak memory? effectively yes, but for a simplified timeout it's "okay" in this context or we need to store the closure key.
-                                // In a real app we might want to store the closure to drop it, but `set_timeout` takes a function pointer effectively. 
+                                // In a real app we might want to store the closure to drop it, but `set_timeout` takes a function pointer effectively.
                                 // Actually, `closure.forget()` is standard for one-off callbacks that the JS side consumes.
-                                
+
                                 set_timer_id.set(Some(id));
                             }
                         }
@@ -329,7 +362,7 @@ pub fn TorrentTable() -> impl IntoView {
                     };
 
                     view! {
-                        <div 
+                        <div
                             class=move || {
                                 "card card-compact bg-base-100 shadow-sm border border-base-200 transition-transform active:scale-[0.99] select-none"
                             }
@@ -355,7 +388,7 @@ pub fn TorrentTable() -> impl IntoView {
                                         {status_str}
                                     </div>
                                 </div>
-                                
+
                                 <div class="flex flex-col gap-1">
                                     <div class="flex justify-between text-[10px] opacity-70">
                                         <span>{format_bytes(t.size)}</span>
@@ -383,7 +416,7 @@ pub fn TorrentTable() -> impl IntoView {
                     }
                 }).collect::<Vec<_>>()}
             </div>
-            
+
             <Show when=move || menu_visible.get() fallback=|| ()>
                 <crate::components::context_menu::ContextMenu
                     visible=true