fix: ios safari mobile dropdown and click bubbling issues resolved with global closer and native details

fix: ios safari mobile dropdown and click bubbling issues resolved with cursor-pointer and css fixes
fix: incorrect import of on_click_outside in statusbar.rs
2026-02-08 21:49:00 +03:00 · 2026-02-08 21:28:10 +03:00 · 2026-02-08 21:00:32 +03:00 · 2026-02-08 20:58:14 +03:00 · 2026-02-08 20:47:42 +03:00
4 changed files with 75 additions and 118 deletions
--- a/frontend/Cargo.toml
+++ b/frontend/Cargo.toml
@@ -22,33 +22,7 @@ wasm-bindgen-futures = "0.4"
 uuid = { version = "1", features = ["v4", "js"] }
 futures = "0.3"
 chrono = { version = "0.4", features = ["serde", "wasm-bindgen"] }
-web-sys = { version = "0.3", features = [
+web-sys = { version = "0.3", features = ["HtmlDivElement", "HtmlUListElement", "HtmlLiElement", "HtmlAnchorElement", "MouseEvent", "Event", "Window", "Document", "Element", "DomTokenList", "CssStyleDeclaration", "Storage", "TouchEvent", "TouchList", "Touch", "Navigator", "Notification", "NotificationOptions", "NotificationPermission", "ServiceWorkerContainer", "ServiceWorkerRegistration", "PushManager", "PushSubscription", "PushSubscriptionOptions", "PushSubscriptionOptionsInit", "HtmlDetailsElement"] }
    "HtmlDivElement",
    "HtmlUListElement",
    "HtmlLiElement",
    "HtmlAnchorElement",
    "MouseEvent",
    "Event",
    "Window",
    "Document",
    "Element",
    "DomTokenList",
    "CssStyleDeclaration",
    "Storage",
    "TouchEvent",
    "TouchList",
    "Touch",
    "Navigator",
    "Notification",
    "NotificationOptions",
    "NotificationPermission",
    "ServiceWorkerContainer",
    "ServiceWorkerRegistration",
    "PushManager",
    "PushSubscription",
    "PushSubscriptionOptions",
    "PushSubscriptionOptionsInit"
 ] }
 shared = { path = "../shared" }
 tailwind_fuse = "0.3.2"
 js-sys = "0.3.85"
--- a/frontend/index.html
+++ b/frontend/index.html
@@ -4,7 +4,7 @@
        <meta charset="utf-8" />
        <meta
            name="viewport"
-            content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no, viewport-fit=cover"
+            content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no, viewport-fit=cover"
        />
        <title>VibeTorrent</title>
@@ -81,7 +81,7 @@
        </script>
    </head>
-    <body>
+    <body style="cursor: pointer;">
        <div
            id="app-loading"
            style="
@@ -142,6 +142,19 @@
            body.app-loaded #app-loading {
                display: none !important;
            }
            /* iOS Safari Click Fixes */
            body {
                cursor: pointer;
                -webkit-tap-highlight-color: transparent;
            }
            summary {
                list-style: none;
            }
            summary::-webkit-details-marker {
                display: none;
            }
        </style>
        <script>
@@ -174,6 +187,17 @@
        <!-- Service Worker Registration & PWA Setup -->
        <script>
            // Global Dropdown Closer for iOS/Mobile
            document.addEventListener('click', function(event) {
                const details = document.querySelectorAll('details[open]');
                details.forEach(detail => {
                    // Eğer tıklanan yer bu details'in içinde değilse kapat
                    if (!detail.contains(event.target)) {
                        detail.removeAttribute('open');
                    }
                });
            }, true); // Use capture phase for better mobile support
            if ("serviceWorker" in navigator) {
                window.addEventListener("load", () => {
                    navigator.serviceWorker
--- a/frontend/src/components/layout/statusbar.rs
+++ b/frontend/src/components/layout/statusbar.rs
@@ -97,43 +97,24 @@ pub fn StatusBar() -> impl IntoView {
        });
    };
-        // Signal-based dropdown state: 0=none, 1=download, 2=upload, 3=theme
+        // Refs for click outside detection (Handled globally via JS in index.html for better iOS support)
-        let (active_dropdown, set_active_dropdown) = create_signal(0u8);
+        let down_details_ref = create_node_ref::<html::Details>();
        let up_details_ref = create_node_ref::<html::Details>();
        let theme_details_ref = create_node_ref::<html::Details>();
-        // Toggle a specific dropdown
+        // Helper to close a details element
-        let toggle = move |id: u8| {
+        let close_details = |node_ref: NodeRef<html::Details>| {
-            let current = active_dropdown.get_untracked();
+            if let Some(el) = node_ref.get_untracked() {
-            if current == id {
+                el.set_open(false);
                set_active_dropdown.set(0);
            } else {
                set_active_dropdown.set(id);
            }
        };
        // Close all dropdowns
        let close_all = move || {
            set_active_dropdown.set(0);
        };
        // Refs for click outside detection
        let down_ref = create_node_ref::<html::Div>();
        let up_ref = create_node_ref::<html::Div>();
        let theme_ref = create_node_ref::<html::Div>();
        let _ = on_click_outside(down_ref, move |_| if active_dropdown.get_untracked() == 1 { close_all() });
        let _ = on_click_outside(up_ref, move |_| if active_dropdown.get_untracked() == 2 { close_all() });
        let _ = on_click_outside(theme_ref, move |_| if active_dropdown.get_untracked() == 3 { close_all() });
        view! {
-            <div class="fixed bottom-0 left-0 right-0 h-8 min-h-8 bg-base-200 border-t border-base-300 flex items-center px-4 text-xs gap-4 text-base-content/70 z-[99]">
+            <div class="fixed bottom-0 left-0 right-0 h-8 min-h-8 bg-base-200 border-t border-base-300 flex items-center px-4 text-xs gap-4 text-base-content/70 z-[99] cursor-pointer">
                // --- DOWNLOAD SPEED DROPDOWN ---
-                <div class="relative" node_ref=down_ref>
+                <details class="dropdown dropdown-top" node_ref=down_details_ref>
-                    <div
+                    <summary class="flex items-center gap-2 cursor-pointer hover:text-primary transition-colors select-none list-none [&::-webkit-details-marker]:hidden outline-none">
                        class="flex items-center gap-2 cursor-pointer hover:text-primary transition-colors select-none"
                        title="Global Download Speed - Click to Set Limit"
                        on:click=move |_| toggle(1)
                    >
                        <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="M19.5 13.5L12 21m0 0l-7.5-7.5M12 21V3" />
                        </svg>
@@ -143,12 +124,9 @@ pub fn StatusBar() -> impl IntoView {
                                {move || format!("(Limit: {})", format_speed(stats.get().down_limit.unwrap_or(0)))}
                            </span>
                        </Show>
-                    </div>
+                    </summary>
-                    <ul
+                    <ul class="dropdown-content z-[100] menu p-2 shadow bg-base-200 rounded-box w-40 mb-2 border border-base-300">
                        class="absolute bottom-full left-0 z-[100] menu p-2 shadow bg-base-200 rounded-box w-40 mb-2 border border-base-300"
                        style=move || if active_dropdown.get() == 1 { "display: block" } else { "display: none" }
                    >
                        {
                            limits.clone().into_iter().map(|(val, label)| {
                                let is_active = move || {
@@ -161,7 +139,7 @@ pub fn StatusBar() -> impl IntoView {
                                            class=move || if is_active() { "bg-primary/10 text-primary font-bold text-xs flex justify-between" } else { "text-xs flex justify-between" }
                                            on:click=move |_| {
                                                set_limit("down", val);
-                                                close_all();
+                                                close_details(down_details_ref);
                                            }
                                        >
                                            {label}
@@ -174,15 +152,11 @@ pub fn StatusBar() -> impl IntoView {
                            }).collect::<Vec<_>>()
                        }
                    </ul>
-                </div>
+                </details>
                // --- UPLOAD SPEED DROPDOWN ---
-                <div class="relative" node_ref=up_ref>
+                <details class="dropdown dropdown-top" node_ref=up_details_ref>
-                    <div
+                    <summary class="flex items-center gap-2 cursor-pointer hover:text-primary transition-colors select-none list-none [&::-webkit-details-marker]:hidden outline-none">
                        class="flex items-center gap-2 cursor-pointer hover:text-primary transition-colors select-none"
                        title="Global Upload Speed - Click to Set Limit"
                        on:click=move |_| toggle(2)
                    >
                        <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="M4.5 10.5L12 3m0 0l7.5 7.5M12 3v18" />
                        </svg>
@@ -192,12 +166,9 @@ pub fn StatusBar() -> impl IntoView {
                                {move || format!("(Limit: {})", format_speed(stats.get().up_limit.unwrap_or(0)))}
                            </span>
                        </Show>
-                    </div>
+                    </summary>
-                    <ul
+                    <ul class="dropdown-content z-[100] menu p-2 shadow bg-base-200 rounded-box w-40 mb-2 border border-base-300">
                        class="absolute bottom-full left-0 z-[100] menu p-2 shadow bg-base-200 rounded-box w-40 mb-2 border border-base-300"
                        style=move || if active_dropdown.get() == 2 { "display: block" } else { "display: none" }
                    >
                        {
                            limits.clone().into_iter().map(|(val, label)| {
                                let is_active = move || {
@@ -210,7 +181,7 @@ pub fn StatusBar() -> impl IntoView {
                                            class=move || if is_active() { "bg-primary/10 text-primary font-bold text-xs flex justify-between" } else { "text-xs flex justify-between" }
                                            on:click=move |_| {
                                                set_limit("up", val);
-                                                close_all();
+                                                close_details(up_details_ref);
                                            }
                                        >
                                            {label}
@@ -223,24 +194,17 @@ pub fn StatusBar() -> impl IntoView {
                            }).collect::<Vec<_>>()
                        }
                    </ul>
-                </div>
+                </details>
                <div class="ml-auto flex items-center gap-4">
-                    <div class="relative" node_ref=theme_ref>
+                    <details class="dropdown dropdown-top dropdown-end" node_ref=theme_details_ref>
-                        <div
+                        <summary class="btn btn-ghost btn-xs btn-square cursor-pointer outline-none list-none [&::-webkit-details-marker]:hidden">
                            class="btn btn-ghost btn-xs btn-square"
                            title="Change Theme"
                            on:click=move |_| toggle(3)
                        >
                            <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.53 16.122a3 3 0 0 0-5.78 1.128 2.25 2.25 0 0 1-2.4 2.245 4.5 4.5 0 0 0 8.4-2.245c0-.399-.078-.78-.22-1.128Zm0 0a15.998 15.998 0 0 0 3.388-1.62m-5.043-.025a15.994 15.994 0 0 1 1.622-3.395m3.42 3.42a15.995 15.995 0 0 0 4.764-4.648l3.876-5.814a1.151 1.151 0 0 0-1.597-1.597L14.146 6.32a15.996 15.996 0 0 0-4.649 4.763m3.42 3.42a6.776 6.776 0 0 0-3.42-3.42" />
                            </svg>
-                        </div>
+                        </summary>
-                        <ul
+                        <ul 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">
                            class="absolute bottom-full right-0 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"
                            style=move || if active_dropdown.get() == 3 { "display: block" } else { "display: none" }
                        >
                            {
                                let themes = vec![
                                    "light", "dark", "dim", "nord", "cupcake", "dracula", "cyberpunk", "emerald", "sunset", "abyss"
@@ -252,7 +216,7 @@ pub fn StatusBar() -> impl IntoView {
                                                class=move || if current_theme.get() == theme { "bg-primary/10 text-primary font-bold text-xs capitalize" } else { "text-xs capitalize" }
                                                on:click=move |_| {
                                                    set_current_theme.set(theme.to_string());
-                                                    close_all();
+                                                    close_details(theme_details_ref);
                                                }
                                            >
                                                {theme}
@@ -262,7 +226,7 @@ pub fn StatusBar() -> impl IntoView {
                                }).collect::<Vec<_>>()
                            }
                        </ul>
-                    </div>
+                    </details>
                <button
                    class="btn btn-ghost btn-xs btn-square"
                    title="Settings & Notification Permissions"
--- a/frontend/src/components/torrent/table.rs
+++ b/frontend/src/components/torrent/table.rs
@@ -164,10 +164,13 @@ pub fn TorrentTable() -> impl IntoView {
        }
    };
-    // Signal-based sort dropdown for mobile
+    // Refs for click outside detection
-    let (sort_open, set_sort_open) = create_signal(false);
+    let sort_details_ref = create_node_ref::<html::Details>();
-    let sort_menu_ref = create_node_ref::<html::Div>();
+    let _ = on_click_outside(sort_details_ref, move |_| {
-    let _ = on_click_outside(sort_menu_ref, move |_| set_sort_open.set(false));
+        if let Some(el) = sort_details_ref.get_untracked() {
            el.set_open(false);
        }
    });
    let sort_arrow = move |col: SortColumn| {
        if sort_col.get() == col {
@@ -342,28 +345,18 @@ pub fn TorrentTable() -> impl IntoView {
                </table>
            </div>
-                        <div class="md:hidden flex flex-col h-full bg-base-200 relative">
+                        <div class="md:hidden flex flex-col h-full bg-base-200 relative cursor-pointer">
-                            <div class="px-3 py-2 border-b border-base-200 flex justify-between items-center bg-base-100/95 backdrop-blur z-10 shrink-0">
+                            <div class="px-3 py-2 border-b border-base-200 flex justify-between items-center bg-base-100/95 backdrop-blur z-10 shrink-0 cursor-default">
                                <span class="text-xs font-bold opacity-50 uppercase tracking-wider">"Torrents"</span>
-                                <div class="relative" node_ref=sort_menu_ref>
+                                <details class="dropdown dropdown-end" node_ref=sort_details_ref>
-                                    <div
+                                    <summary class="btn btn-ghost btn-xs gap-1 opacity-70 font-normal list-none [&::-webkit-details-marker]:hidden cursor-pointer">
-                                        role="button"
+                                        <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 pointer-events-none">
                                        class="btn btn-ghost btn-xs gap-1 opacity-70 font-normal"
                                        on:click=move |_| {
                                            let cur = sort_open.get_untracked();
                                            set_sort_open.set(!cur);
                                        }
                                    >
                                        <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="M3 7.5L7.5 3m0 0L12 7.5M7.5 3v13.5m13.5 0L16.5 21m0 0L12 16.5m4.5 4.5V7.5" />
                                        </svg>
-                                        "Sort"
+                                        <span class="pointer-events-none">"Sort"</span>
-                                    </div>
+                                    </summary>
-                                    <ul
+                                    <ul class="dropdown-content z-[100] menu p-2 shadow bg-base-100 rounded-box w-48 mt-1 border border-base-200 text-xs cursor-default">
                                        class="absolute top-full right-0 z-[100] menu p-2 shadow bg-base-100 rounded-box w-48 mt-1 border border-base-200 text-xs"
                                        style=move || if sort_open.get() { "display: block" } else { "display: none" }
                                    >
                                         <li class="menu-title px-2 py-1 opacity-50 text-[10px] uppercase font-bold">"Sort By"</li>
                                         {
                                              let columns = vec![
@@ -388,7 +381,9 @@ pub fn TorrentTable() -> impl IntoView {
                                                              class=move || if is_active() { "bg-primary/10 text-primary font-bold flex justify-between" } else { "flex justify-between" }
                                                              on:click=move |_| {
                                                                  handle_sort(col);
-                                                                  set_sort_open.set(false);
+                                                                  if let Some(el) = sort_details_ref.get_untracked() {
                                                                      el.set_open(false);
                                                                  }
                                                              }
                                                          >
                                                             {label}
@@ -406,10 +401,10 @@ pub fn TorrentTable() -> impl IntoView {
                                             }).collect::<Vec<_>>()
                                         }
                                    </ul>
-                                </div>
+                                </details>
                            </div>
-                            <div class="overflow-y-auto p-3 pb-20 flex-1 grid grid-cols-1 content-start gap-3">                {move || filtered_torrents().into_iter().map(|t| {
+                            <div class="overflow-y-auto p-3 pb-20 flex-1 grid grid-cols-1 content-start gap-3 cursor-pointer">                {move || filtered_torrents().into_iter().map(|t| {
                    let progress_class = if t.percent_complete >= 100.0 { "progress-success" } else { "progress-primary" };
                    let status_str = format!("{:?}", t.status);
                    let status_badge_class = match t.status {
@@ -464,7 +459,7 @@ pub fn TorrentTable() -> impl IntoView {
                    view! {
                        <div
                            class=move || {
-                                "card card-compact bg-base-100 shadow-sm border border-base-200 transition-transform active:scale-[0.99] select-none"
+                                "card card-compact bg-base-100 shadow-sm border border-base-200 transition-transform active:scale-[0.99] select-none cursor-pointer"
                            }
                            style="user-select: none; -webkit-user-select: none; -webkit-touch-callout: none;"
                            on:contextmenu={
Author	SHA1	Message	Date
spinline	d3792e78e0	fix: ios safari mobile dropdown and click bubbling issues resolved with global closer and native details All checks were successful Build MIPS Binary / build (push) Successful in 5m45s Details	2026-02-08 21:49:00 +03:00
spinline	384165a958	fix: ios safari mobile dropdown and click bubbling issues resolved with cursor-pointer and css fixes All checks were successful Build MIPS Binary / build (push) Successful in 4m27s Details	2026-02-08 21:28:10 +03:00
spinline	7169e44f4e	fix: incorrect import of on_click_outside in statusbar.rs All checks were successful Build MIPS Binary / build (push) Successful in 4m28s Details	2026-02-08 21:00:32 +03:00
spinline	51fb85c2d8	fix: iOS Safari uyumluluğu için menü yapıları ve click bubbling sorunları giderildi Some checks failed Build MIPS Binary / build (push) Failing after 1m8s Details	2026-02-08 20:58:14 +03:00
spinline	46882caea0	fix: mobilde menülerin açılmama sorunu HTML details/summary yapısı ile kökten çözüldü All checks were successful Build MIPS Binary / build (push) Successful in 4m31s Details	2026-02-08 20:47:42 +03:00