fix(files): trigger refetch only after set_priority completes on server

frontend/src/components/torrent/files.rs

@@ -15,10 +15,9 @@ pub fn TorrentFilesTab(hash: String) -> impl IntoView {
         |h| async move { shared::server_fns::torrent::get_files(h).await.unwrap_or_default() }
     );
 
-    // Refresh action
-    let refresh_files = Action::new(|h: &String| {
-        let h = h.clone();
-        async move { shared::server_fns::torrent::get_files(h).await.unwrap_or_default() }
+    // Callback to trigger a refetch — safe, doesn't destroy existing components
+    let on_refresh = Callback::new(move |_: ()| {
+        files_resource.refetch();
     });
 
     let stored_hash = StoredValue::new(hash);
@@ -26,10 +25,7 @@ pub fn TorrentFilesTab(hash: String) -> impl IntoView {
     view! {
         <Suspense fallback=move || view! { <FilesFallback /> }>
             {move || {
-                let files = match refresh_files.value().get() {
-                    Some(f) => f,
-                    None => files_resource.get().unwrap_or_default(),
-                };
+                let files = files_resource.get().unwrap_or_default();
 
                 if files.is_empty() {
                     return view! {
@@ -61,12 +57,11 @@ pub fn TorrentFilesTab(hash: String) -> impl IntoView {
                                         key=|f| f.index
                                         children={move |f| {
                                             let p_hash = stored_hash.get_value();
-                                            let r_files = refresh_files.clone();
                                             view! {
                                                 <FileRow 
                                                     file=f 
                                                     hash=p_hash
-                                                    refresh_action=r_files
+                                                    on_refresh=on_refresh.clone()
                                                 />
                                             }
                                         }}
@@ -86,12 +81,14 @@ pub fn TorrentFilesTab(hash: String) -> impl IntoView {
 }
 
 #[component]
-fn FileRow(file: TorrentFile, hash: String, refresh_action: Action<String, Vec<TorrentFile>>) -> impl IntoView {
+fn FileRow(file: TorrentFile, hash: String, on_refresh: Callback<()>) -> impl IntoView {
     let f_idx = file.index;
-    let context_id = format!("file-context-{}-{}", hash, f_idx);
     let path_clone = file.path.clone();
 
-    let set_priority = Action::new(|req: &(String, u32, u8)| {
+    // on_refresh is called AFTER the server responds, not before
+    let on_refresh_stored = StoredValue::new(on_refresh);
+
+    let set_priority = Action::new(move |req: &(String, u32, u8)| {
         let (h, idx, p) = req.clone();
         async move {
             let res = shared::server_fns::torrent::set_file_priority(h, idx, p).await;
@@ -99,6 +96,8 @@ fn FileRow(file: TorrentFile, hash: String, refresh_action: Action<String, Vec<T
                 crate::store::show_toast(shared::NotificationLevel::Error, format!("Öncelik değiştirilemedi: {:?}", e));
             } else {
                 crate::store::show_toast(shared::NotificationLevel::Success, "Dosya önceliği güncellendi.".to_string());
+                // Refetch AFTER the server has saved the priority
+                on_refresh_stored.get_value().run(());
             }
             res
         }
@@ -108,7 +107,6 @@ fn FileRow(file: TorrentFile, hash: String, refresh_action: Action<String, Vec<T
         <FileContextMenu 
             torrent_hash=hash 
             file_index=f_idx 
-            refresh_action=refresh_action 
             set_priority=set_priority
         >
             <TableRow class="hover:bg-muted/50 transition-colors group">
@@ -142,7 +140,6 @@ fn FileContextMenu(
     children: Children,
     torrent_hash: String,
     file_index: u32,
-    refresh_action: Action<String, Vec<TorrentFile>>,
     set_priority: Action<(String, u32, u8), Result<(), ServerFnError>>,
 ) -> impl IntoView {
     let hash_c1 = torrent_hash.clone();
@@ -160,11 +157,9 @@ fn FileContextMenu(
                 <ContextMenuGroup>
                     <ContextMenuItem on:click={
                         let h = hash_c1;
-                        let ra = refresh_action.clone();
                         let sp = set_priority.clone();
                         move |_| {
                             sp.dispatch((h.clone(), file_index, 2));
-                            ra.dispatch(h.clone());
                             crate::components::ui::context_menu::close_context_menu();
                         }
                     }>
@@ -174,11 +169,9 @@ fn FileContextMenu(
                     
                     <ContextMenuItem on:click={
                         let h = hash_c2;
-                        let ra = refresh_action.clone();
                         let sp = set_priority.clone();
                         move |_| {
                             sp.dispatch((h.clone(), file_index, 1));
-                            ra.dispatch(h.clone());
                             crate::components::ui::context_menu::close_context_menu();
                         }
                     }>
@@ -188,11 +181,9 @@ fn FileContextMenu(
                     
                     <ContextMenuItem class="text-destructive focus:bg-destructive/10" on:click={
                         let h = hash_c3;
-                        let ra = refresh_action.clone();
                         let sp = set_priority.clone();
                         move |_| {
                             sp.dispatch((h.clone(), file_index, 0));
-                            ra.dispatch(h.clone());
                             crate::components::ui::context_menu::close_context_menu();
                         }
                     }>

frontend/src/components/ui/context_menu.rs

@@ -215,12 +215,20 @@ pub fn ContextMenuContent(
 
                             // Adjust if menu would go off right edge
                             if (x + menuRect.width > viewportWidth) {{
-                                left = x - menuRect.width;
+                                left = Math.max(0, x - menuRect.width);
                             }}
 
                             // Adjust if menu would go off bottom edge
                             if (y + menuRect.height > viewportHeight) {{
-                                top = y - menuRect.height;
+                                top = Math.max(0, y - menuRect.height);
+                            }}
+                            
+                            // Adjust for CSS transformed containing block
+                            const offsetParent = menu.offsetParent;
+                            if (offsetParent && offsetParent !== document.body && offsetParent !== document.documentElement) {{
+                                const parentRect = offsetParent.getBoundingClientRect();
+                                left -= parentRect.left;
+                                top -= parentRect.top;
                             }}
 
                             menu.style.left = `${{left}}px`;
@@ -228,105 +236,105 @@ pub fn ContextMenuContent(
                             menu.style.transformOrigin = 'top left';
                         }};
 
-                        const openMenu = (x, y) => {{
-                            isOpen = true;
+                            const openMenu = (x, y) => {{
+                                isOpen = true;
 
-                            // Close any other open context menus
-                            const allMenus = document.querySelectorAll('[data-target="target__context"]');
-                            allMenus.forEach(m => {{
-                                if (m !== menu && m.getAttribute('data-state') === 'open') {{
-                                    m.setAttribute('data-state', 'closed');
-                                    m.style.pointerEvents = 'none';
+                                // Close any other open context menus
+                                const allMenus = document.querySelectorAll('[data-target="target__context"]');
+                                allMenus.forEach(m => {{
+                                    if (m !== menu && m.getAttribute('data-state') === 'open') {{
+                                        m.setAttribute('data-state', 'closed');
+                                        m.style.pointerEvents = 'none';
+                                    }}
+                                }});
+
+                                menu.setAttribute('data-state', 'open');
+                                menu.style.visibility = 'hidden';
+                                menu.style.pointerEvents = 'auto';
+
+                                // Force reflow
+                                menu.offsetHeight;
+
+                                updatePosition(x, y);
+                                menu.style.visibility = 'visible';
+
+                                // Lock scroll
+                                if (window.ScrollLock) {{
+                                    window.ScrollLock.lock();
+                                }}
+
+                                setTimeout(() => {{
+                                    document.addEventListener('click', handleClickOutside);
+                                    document.addEventListener('contextmenu', handleContextOutside);
+                                }}, 0);
+                            }};
+
+                            const closeMenu = () => {{
+                                isOpen = false;
+                                menu.setAttribute('data-state', 'closed');
+                                menu.style.pointerEvents = 'none';
+                                document.removeEventListener('click', handleClickOutside);
+                                document.removeEventListener('contextmenu', handleContextOutside);
+
+                                // Dispatch custom event for Leptos to listen to
+                                menu.dispatchEvent(new CustomEvent('contextmenuclose', {{ bubbles: false }}));
+
+                                if (window.ScrollLock) {{
+                                    window.ScrollLock.unlock(200);
+                                }}
+                            }};
+
+                            const handleClickOutside = (e) => {{
+                                if (!menu.contains(e.target)) {{
+                                    closeMenu();
+                                }}
+                            }};
+
+                            const handleContextOutside = (e) => {{
+                                if (!trigger.contains(e.target)) {{
+                                    closeMenu();
+                                }}
+                            }};
+
+                            // Right-click on trigger
+                            trigger.addEventListener('contextmenu', (e) => {{
+                                e.preventDefault();
+                                e.stopPropagation();
+
+                                if (isOpen) {{
+                                    closeMenu();
+                                }}
+                                openMenu(e.clientX, e.clientY);
+                            }});
+
+                            // Close when action is clicked
+                            const actions = menu.querySelectorAll('[data-context-close]');
+                            actions.forEach(action => {{
+                                action.addEventListener('click', () => {{
+                                    closeMenu();
+                                }});
+                            }});
+
+                            // Handle ESC key
+                            document.addEventListener('keydown', (e) => {{
+                                if (e.key === 'Escape' && isOpen) {{
+                                    e.preventDefault();
+                                    closeMenu();
                                 }}
                             }});
-
-                            menu.setAttribute('data-state', 'open');
-                            menu.style.visibility = 'hidden';
-                            menu.style.pointerEvents = 'auto';
-
-                            // Force reflow
-                            menu.offsetHeight;
-
-                            updatePosition(x, y);
-                            menu.style.visibility = 'visible';
-
-                            // Lock scroll
-                            if (window.ScrollLock) {{
-                                window.ScrollLock.lock();
-                            }}
-
-                            setTimeout(() => {{
-                                document.addEventListener('click', handleClickOutside);
-                                document.addEventListener('contextmenu', handleContextOutside);
-                            }}, 0);
                         }};
 
-                        const closeMenu = () => {{
-                            isOpen = false;
-                            menu.setAttribute('data-state', 'closed');
-                            menu.style.pointerEvents = 'none';
-                            document.removeEventListener('click', handleClickOutside);
-                            document.removeEventListener('contextmenu', handleContextOutside);
-
-                            // Dispatch custom event for Leptos to listen to
-                            menu.dispatchEvent(new CustomEvent('contextmenuclose', {{ bubbles: false }}));
-
-                            if (window.ScrollLock) {{
-                                window.ScrollLock.unlock(200);
-                            }}
-                        }};
-
-                        const handleClickOutside = (e) => {{
-                            if (!menu.contains(e.target)) {{
-                                closeMenu();
-                            }}
-                        }};
-
-                        const handleContextOutside = (e) => {{
-                            if (!trigger.contains(e.target)) {{
-                                closeMenu();
-                            }}
-                        }};
-
-                        // Right-click on trigger
-                        trigger.addEventListener('contextmenu', (e) => {{
-                            e.preventDefault();
-                            e.stopPropagation();
-
-                            if (isOpen) {{
-                                closeMenu();
-                            }}
-                            openMenu(e.clientX, e.clientY);
-                        }});
-
-                        // Close when action is clicked
-                        const actions = menu.querySelectorAll('[data-context-close]');
-                        actions.forEach(action => {{
-                            action.addEventListener('click', () => {{
-                                closeMenu();
-                            }});
-                        }});
-
-                        // Handle ESC key
-                        document.addEventListener('keydown', (e) => {{
-                            if (e.key === 'Escape' && isOpen) {{
-                                e.preventDefault();
-                                closeMenu();
-                            }}
-                        }});
-                    }};
-
-                    if (document.readyState === 'loading') {{
-                        document.addEventListener('DOMContentLoaded', setupContextMenu);
-                    }} else {{
-                        setupContextMenu();
-                    }}
-                }})();
-                "#,
-                target_id_for_script,
-                target_id_for_script,
-            )}
-        </script>
+                        if (document.readyState === 'loading') {{
+                            document.addEventListener('DOMContentLoaded', setupContextMenu);
+                        }} else {{
+                            setupContextMenu();
+                        }}
+                    }})();
+                    "#,
+                    target_id_for_script,
+                    target_id_for_script,
+                )}
+            </script>
     }
 }
 

shared/src/server_fns/torrent.rs

@@ -225,6 +225,7 @@ pub async fn set_file_priority(
     let ctx = expect_context::<crate::ServerContext>();
     let client = RtorrentClient::new(&ctx.scgi_socket_path);
 
+    // rTorrent f.set_priority takes: target = "HASH:fINDEX", value = priority
     let target = format!("{}:f{}", hash, file_index);
     let params = vec![
         RpcParam::from(target.as_str()),
@@ -234,8 +235,9 @@ pub async fn set_file_priority(
     client
         .call("f.set_priority", &params)
         .await
-        .map_err(|e| ServerFnError::new(format!("RPC error: {}", e)))?;
+        .map_err(|e| ServerFnError::new(format!("RPC error setting priority: {}", e)))?;
 
+    // Notify rTorrent to update its internal priority state
     let _ = client
         .call("d.update_priorities", &[RpcParam::from(hash.as_str())])
         .await;