Fix mobile dropdown interaction: Revert to pointerdown with stop_propagation and use overlay for reliable closing

.github/workflows/build-mips.yml

@@ -1,80 +0,0 @@
-name: Build MIPS Binary
-
-on:
-  push:
-    branches: [ "main" ]
-  workflow_dispatch:
-
-permissions:
-  contents: write
-
-env:
-  CARGO_TERM_COLOR: always
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-
-    steps:
-    - uses: actions/checkout@v4
-
-    - name: Debug - List Files (Pre-Build)
-      run: ls -R
-
-    - name: Setup Node.js
-      uses: actions/setup-node@v4
-      with:
-        node-version: '20'
-
-    - name: Install Rust
-      uses: dtolnay/rust-toolchain@nightly
-      with:
-        targets: wasm32-unknown-unknown
-        components: rust-src
-
-    - name: Rust Cache
-      uses: Swatinem/rust-cache@v2
-
-    - name: Install Trunk
-      uses: jetli/trunk-action@v0.5.0
-      with:
-        version: 'latest'
-
-    - name: Build Frontend
-      run: |
-        cd frontend
-        npm install
-        trunk build --release
-
-    - name: Install Cross
-      run: cargo install cross
-
-    - name: Build Backend (MIPS)
-      env:
-        RUSTUP_TOOLCHAIN: nightly
-        CROSS_NO_WARNINGS: 0
-      run: |
-        cd backend
-        cross build --target mips-unknown-linux-musl --release -Z build-std=std,panic_abort
-
-    - name: Debug - List Files
-      run: |
-        echo "Listing target directory..."
-        find target -maxdepth 5 || true
-        ls -R target/mips-unknown-linux-musl/release || true
-
-    - name: Rename Binary
-      run: mv target/mips-unknown-linux-musl/release/backend target/mips-unknown-linux-musl/release/vibetorrent-mips
-
-    - name: Generate Tag
-      id: tag
-      run: echo "release_tag=release-$(date +'%Y%m%d-%H%M')" >> $GITHUB_OUTPUT
-
-    - name: Create Release
-      uses: softprops/action-gh-release@v1
-      with:
-        tag_name: ${{ steps.tag.outputs.release_tag }}
-        name: Release ${{ steps.tag.outputs.release_tag }}
-        files: target/mips-unknown-linux-musl/release/vibetorrent-mips
-        draft: false
-        prerelease: false

docker/runner/Dockerfile

@@ -16,6 +16,8 @@ RUN apt-get update && apt-get install -y \
     xz-utils \
     libssl-dev \
     pkg-config \
+    file \
+    jq \
     # Needed for some crate compilations
     protobuf-compiler \
     && rm -rf /var/lib/apt/lists/*

frontend/src/components/layout/statusbar.rs

@@ -111,8 +111,6 @@ pub fn StatusBar() -> impl IntoView {
 
         // Signal-based dropdown state: 0=none, 1=download, 2=upload, 3=theme
         let (active_dropdown, set_active_dropdown) = create_signal(0u8);
-    // Guard to prevent global close from firing right after toggle opens
-    let skip_next_close = store_value(false);
 
         // Toggle a specific dropdown
         let toggle = move |id: u8| {
@@ -121,8 +119,6 @@ pub fn StatusBar() -> impl IntoView {
                 set_active_dropdown.set(0);
             } else {
                 set_active_dropdown.set(id);
-            // Mark that the next global close should be skipped
-            skip_next_close.set_value(true);
             }
         };
 
@@ -131,16 +127,15 @@ pub fn StatusBar() -> impl IntoView {
             set_active_dropdown.set(0);
         };
 
-    // Close dropdowns when tapping outside — uses click (fires after pointerdown)
-    let _ = window_event_listener(ev::click, move |_| {
-        if skip_next_close.get_value() {
-            skip_next_close.set_value(false);
-            return;
-        }
-        set_active_dropdown.set(0);
-    });
-
         view! {
+            // Transparent overlay to close dropdowns when clicking outside
+            <Show when=move || active_dropdown.get() != 0>
+                <div
+                    class="fixed inset-0 z-[98] cursor-default"
+                    on:pointerdown=move |_| close_all()
+                ></div>
+            </Show>
+
             <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]">
 
                 // --- DOWNLOAD SPEED DROPDOWN ---
@@ -167,7 +162,6 @@ pub fn StatusBar() -> impl IntoView {
                     <ul
                         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" }
-                    on:pointerdown=move |e| e.stop_propagation()
                     >
                         {
                             limits.clone().into_iter().map(|(val, label)| {
@@ -221,7 +215,6 @@ pub fn StatusBar() -> impl IntoView {
                     <ul
                         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" }
-                    on:pointerdown=move |e| e.stop_propagation()
                     >
                         {
                             limits.clone().into_iter().map(|(val, label)| {
@@ -269,7 +262,6 @@ pub fn StatusBar() -> impl IntoView {
                         <ul
                             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" }
-                        on:pointerdown=move |e| e.stop_propagation()
                         >
                             {
                                 let themes = vec![
@@ -302,7 +294,6 @@ pub fn StatusBar() -> impl IntoView {
                             }
                         </ul>
                     </div>
-
                 <button
                     class="btn btn-ghost btn-xs btn-square"
                     title="Settings & Notification Permissions"

frontend/src/components/torrent/table.rs

@@ -154,14 +154,6 @@ pub fn TorrentTable() -> impl IntoView {
 
     // Signal-based sort dropdown for mobile
     let (sort_open, set_sort_open) = create_signal(false);
-    let sort_skip_close = store_value(false);
-    let _ = window_event_listener(ev::click, move |_| {
-        if sort_skip_close.get_value() {
-            sort_skip_close.set_value(false);
-            return;
-        }
-        set_sort_open.set(false);
-    });
 
     let sort_arrow = move |col: SortColumn| {
         if sort_col.get() == col {
@@ -332,7 +324,15 @@ pub fn TorrentTable() -> impl IntoView {
                 </table>
             </div>
 
-            <div class="md:hidden flex flex-col h-full bg-base-200">
+                        <div class="md:hidden flex flex-col h-full bg-base-200 relative">
+                            // Transparent overlay to close sort dropdown
+                            <Show when=move || sort_open.get()>
+                                <div
+                                    class="fixed inset-0 z-[98] cursor-default"
+                                    on:pointerdown=move |_| set_sort_open.set(false)
+                                ></div>
+                            </Show>
+
                             <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">
                                 <span class="text-xs font-bold opacity-50 uppercase tracking-wider">"Torrents"</span>
 
@@ -343,12 +343,7 @@ pub fn TorrentTable() -> impl IntoView {
                                         on:pointerdown=move |e| {
                                             e.stop_propagation();
                                             let cur = sort_open.get_untracked();
-                                if cur {
-                                    set_sort_open.set(false);
-                                } else {
-                                    set_sort_open.set(true);
-                                    sort_skip_close.set_value(true);
-                                }
+                                            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">
@@ -405,8 +400,7 @@ pub fn TorrentTable() -> impl IntoView {
                                 </div>
                             </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">                {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 {

frontend/sw.js

@@ -1,69 +1,99 @@
-const CACHE_NAME = 'vibetorrent-v1';
+const CACHE_NAME = "vibetorrent-v2";
 const ASSETS_TO_CACHE = [
-    '/',
-    '/index.html',
-    '/manifest.json',
-    '/icon-192.png',
-    '/icon-512.png'
+  "/",
+  "/index.html",
+  "/manifest.json",
+  "/icon-192.png",
+  "/icon-512.png",
 ];
 
 // Install event - cache assets
-self.addEventListener('install', (event) => {
-    console.log('[Service Worker] Installing...');
+self.addEventListener("install", (event) => {
+  console.log("[Service Worker] Installing...");
   event.waitUntil(
-        caches.open(CACHE_NAME).then((cache) => {
-            console.log('[Service Worker] Caching static assets');
+    caches
+      .open(CACHE_NAME)
+      .then((cache) => {
+        console.log("[Service Worker] Caching static assets");
         return cache.addAll(ASSETS_TO_CACHE);
-        }).then(() => {
-            console.log('[Service Worker] Skip waiting');
-            return self.skipWaiting();
       })
+      .then(() => {
+        console.log("[Service Worker] Skip waiting");
+        return self.skipWaiting();
+      }),
   );
 });
 
 // Activate event - clean old caches
-self.addEventListener('activate', (event) => {
-    console.log('[Service Worker] Activating...');
+self.addEventListener("activate", (event) => {
+  console.log("[Service Worker] Activating...");
   event.waitUntil(
-        caches.keys().then((cacheNames) => {
+    caches
+      .keys()
+      .then((cacheNames) => {
         return Promise.all(
           cacheNames.map((key) => {
             if (key !== CACHE_NAME) {
-                        console.log('[Service Worker] Deleting old cache:', key);
+              console.log("[Service Worker] Deleting old cache:", key);
               return caches.delete(key);
             }
-                })
+          }),
         );
-        }).then(() => {
-            console.log('[Service Worker] Claiming clients');
-            return self.clients.claim();
       })
+      .then(() => {
+        console.log("[Service Worker] Claiming clients");
+        return self.clients.claim();
+      }),
   );
 });
 
-// Fetch event - network first, cache fallback for API calls
-self.addEventListener('fetch', (event) => {
+// Fetch event - network first for HTML, cache fallback for API calls
+self.addEventListener("fetch", (event) => {
   const url = new URL(event.request.url);
 
   // Network-first strategy for API calls
-    if (url.pathname.startsWith('/api/')) {
+  if (url.pathname.startsWith("/api/")) {
     event.respondWith(
-            fetch(event.request)
-                .catch(() => {
+      fetch(event.request).catch(() => {
         // Could return cached API response or offline page
-                    return new Response(
-                        JSON.stringify({ error: 'Offline' }),
-                        { headers: { 'Content-Type': 'application/json' } }
-                    );
-                })
+        return new Response(JSON.stringify({ error: "Offline" }), {
+          headers: { "Content-Type": "application/json" },
+        });
+      }),
     );
     return;
   }
 
-    // Cache-first strategy for static assets
+  // Network-first strategy for HTML pages (entry points)
+  // This ensures users always get the latest version of the app
+  if (
+    event.request.mode === "navigate" ||
+    url.pathname.endsWith("index.html") ||
+    url.pathname === "/"
+  ) {
+    event.respondWith(
+      fetch(event.request)
+        .then((response) => {
+          // Cache the latest version of the HTML
+          const responseToCache = response.clone();
+          caches.open(CACHE_NAME).then((cache) => {
+            cache.put(event.request, responseToCache);
+          });
+          return response;
+        })
+        .catch(() => {
+          return caches.match(event.request);
+        }),
+    );
+    return;
+  }
+
+  // Cache-first strategy for static assets (JS, CSS, Images)
   event.respondWith(
     caches.match(event.request).then((response) => {
-            return response || fetch(event.request).then((fetchResponse) => {
+      return (
+        response ||
+        fetch(event.request).then((fetchResponse) => {
           // Optionally cache new requests
           if (fetchResponse && fetchResponse.status === 200) {
             const responseToCache = fetchResponse.clone();
@@ -72,56 +102,57 @@ self.addEventListener('fetch', (event) => {
             });
           }
           return fetchResponse;
-            });
         })
       );
+    }),
+  );
 });
 
 // Notification click event - focus or open app
-self.addEventListener('notificationclick', (event) => {
-    console.log('[Service Worker] Notification clicked:', event.notification.tag);
+self.addEventListener("notificationclick", (event) => {
+  console.log("[Service Worker] Notification clicked:", event.notification.tag);
   event.notification.close();
 
   event.waitUntil(
-        clients.matchAll({ type: 'window', includeUncontrolled: true }).then((clientList) => {
+    clients
+      .matchAll({ type: "window", includeUncontrolled: true })
+      .then((clientList) => {
         // If app is already open, focus it
         for (let client of clientList) {
-                if (client.url === '/' && 'focus' in client) {
+          if (client.url === "/" && "focus" in client) {
             return client.focus();
           }
         }
         // Otherwise open new window
         if (clients.openWindow) {
-                return clients.openWindow('/');
+          return clients.openWindow("/");
         }
-        })
+      }),
   );
 });
 
 // Push notification event
-self.addEventListener('push', (event) => {
-    console.log('[Service Worker] Push notification received');
+self.addEventListener("push", (event) => {
+  console.log("[Service Worker] Push notification received");
   const data = event.data ? event.data.json() : {};
 
-    const title = data.title || 'VibeTorrent';
+  const title = data.title || "VibeTorrent";
   const options = {
-        body: data.body || 'New notification',
-        icon: data.icon || '/icon-192.png',
-        badge: data.badge || '/icon-192.png',
-        tag: data.tag || 'vibetorrent-notification',
+    body: data.body || "New notification",
+    icon: data.icon || "/icon-192.png",
+    badge: data.badge || "/icon-192.png",
+    tag: data.tag || "vibetorrent-notification",
     requireInteraction: false,
     // iOS-specific: vibrate pattern (if supported)
     vibrate: [200, 100, 200],
     // Add data for notification click handling
     data: {
-            url: data.url || '/',
-            timestamp: Date.now()
-        }
+      url: data.url || "/",
+      timestamp: Date.now(),
+    },
   };
 
-    console.log('[Service Worker] Showing notification:', title, options);
+  console.log("[Service Worker] Showing notification:", title, options);
 
-    event.waitUntil(
-        self.registration.showNotification(title, options)
-    );
+  event.waitUntil(self.registration.showNotification(title, options));
 });