From 761a203d7c7630fc161fc6c64b32f3d22354cb8d Mon Sep 17 00:00:00 2001
From: Edouard Vanbelle <opensource@edouard.vanbelle.fr>
Date: Sun, 26 Jul 2026 13:27:17 +0200
Subject: [PATCH] feat(drag&drop): dropzone is the while page

---
 .../src/lib/components/ResourceList.svelte    | 17 +++-
 .../src/routes/files/[...path]/+page.svelte   | 78 ++++++++++++++++++-
 2 files changed, 93 insertions(+), 2 deletions(-)

diff --git a/frontend/src/lib/components/ResourceList.svelte b/frontend/src/lib/components/ResourceList.svelte
index 3da54ec4..956b7a43 100644
--- a/frontend/src/lib/components/ResourceList.svelte
+++ b/frontend/src/lib/components/ResourceList.svelte
@@ -297,6 +297,20 @@
 		 * `enableSystemDrop` is false.
 		 */
 		onsystemdrop?: (e: DragEvent) => void;
+		/**
+		 * Forces the OS-file drop overlay on regardless of whether the
+		 * pointer is currently over `.rl-root`. Only takes effect when
+		 * `enableSystemDrop` is true (otherwise there's no drop route
+		 * behind the overlay). Meant for pages that track drag state
+		 * at window level and want the overlay to react anywhere on
+		 * the viewport — not just over the list — so drops on the
+		 * page's title strip / margins get the same "drop to upload"
+		 * cue. `/files/[...path]/+page.svelte` uses this. The overlay
+		 * itself is already `position: fixed`, so activating it from a
+		 * broader trigger costs no visual change — it OR's with the
+		 * internal `systemDropOver` counter.
+		 */
+		systemDropOverlayActive?: boolean;
 		/**
 		 * Render `<img>` thumbnails on file rows and fall back to
 		 * client-side generation when the server doesn't have one
@@ -394,6 +408,7 @@
 		breadcrumb,
 		enableSystemDrop = false,
 		onsystemdrop,
+		systemDropOverlayActive = false,
 		enableThumbnails = true,
 		isDraggable,
 		isDropTarget,
@@ -1419,7 +1434,7 @@
 	     border alone read as decoration on busy folders. `pointer-events:
 	     none` on the container keeps it inert (drag events still hit
 	     `.rl-root` underneath so `dragleave`/`drop` fire correctly). -->
-	{#if systemDropOver && enableSystemDrop}
+	{#if enableSystemDrop && (systemDropOver || systemDropOverlayActive)}
 		<div class="rl-drop-overlay" aria-hidden="true">
 			<div class="rl-drop-overlay__inner">
 				<Icon name="cloud-arrow-up" class="rl-drop-overlay__icon" />
diff --git a/frontend/src/routes/files/[...path]/+page.svelte b/frontend/src/routes/files/[...path]/+page.svelte
index 36b3c387..87f36152 100644
--- a/frontend/src/routes/files/[...path]/+page.svelte
+++ b/frontend/src/routes/files/[...path]/+page.svelte
@@ -775,6 +775,74 @@
 		else if (dt.files?.length) await uploadBatch(Array.from(dt.files));
 	}
 
+	/**
+	 * Page-wide OS drag/drop safety net.
+	 *
+	 * `ResourceList` wires system-drop handlers on `.rl-root`, so a drop
+	 * on the list surface hits `onDrop` correctly. But `.rl-root` doesn't
+	 * cover the whole viewport — dropping OS files on the title strip,
+	 * the outer margin, or any gap around the list would either be
+	 * ignored (bad UX) or opened as a top-level navigation by the
+	 * browser (data-loss-esque: the tab replaces the app with the
+	 * dropped file). Both `dragover` and `drop` are captured at window
+	 * level here so anywhere on the `/files` page accepts the drop.
+	 *
+	 * The two `defaultPrevented` guards below defer to `.rl-root`'s
+	 * handler when the drop lands there: ResourceList's handlers
+	 * already `preventDefault`, so a bubbling window listener sees
+	 * `defaultPrevented === true` and skips — no double-upload.
+	 */
+	function onWindowDragOver(e: DragEvent) {
+		if (!e.dataTransfer?.types?.includes('Files')) return;
+		if (e.defaultPrevented) return;
+		// preventDefault on `dragover` is what tells the browser "this
+		// element accepts drops" — without it, `drop` never fires and
+		// the OS shows the "no-drop" cursor over the page's margins.
+		e.preventDefault();
+		if (e.dataTransfer) e.dataTransfer.dropEffect = 'copy';
+	}
+	function onWindowDrop(e: DragEvent) {
+		if (!e.dataTransfer?.types?.includes('Files')) return;
+		// Force-clear the overlay counter regardless of who handles the
+		// drop — .rl-root's inner handler doesn't touch the page-level
+		// counter and a stray unbalanced dragenter would otherwise
+		// leave the overlay stuck on.
+		pageSystemDragDepth = 0;
+		if (e.defaultPrevented) return;
+		onDrop(e);
+	}
+
+	/**
+	 * Page-wide drop-overlay trigger. The overlay itself lives in
+	 * `ResourceList` (`position: fixed`, already viewport-sized), but its
+	 * default trigger is a counter on `.rl-root` — so dragging OS files
+	 * over the page's title strip or the outer margins wouldn't light
+	 * up the overlay even though the drop WOULD be accepted (see
+	 * `onWindowDragOver`/`onWindowDrop` above). This counter tracks
+	 * the drag at window level and drives ResourceList via the
+	 * `systemDropOverlayActive` prop so the visual matches the
+	 * accepts-drop area exactly.
+	 *
+	 * Same enter/leave-counter shape ResourceList uses internally —
+	 * dragenter/leave chatter as the pointer moves between child
+	 * elements would flicker a naive boolean, so we count.
+	 * `drop`/`dragend` force-zero the counter to survive an
+	 * unbalanced enter (e.g. drag cancelled by the user pressing Esc).
+	 */
+	let pageSystemDragDepth = $state(0);
+	const pageSystemDropOver = $derived(pageSystemDragDepth > 0);
+	function onWindowDragEnter(e: DragEvent) {
+		if (!e.dataTransfer?.types?.includes('Files')) return;
+		pageSystemDragDepth++;
+	}
+	function onWindowDragLeave(e: DragEvent) {
+		if (!e.dataTransfer?.types?.includes('Files')) return;
+		if (pageSystemDragDepth > 0) pageSystemDragDepth--;
+	}
+	function onWindowDragEnd() {
+		pageSystemDragDepth = 0;
+	}
+
 	/**
 	 * Expand dropped OS entries into `{file, relativePath}` rows, walking any
 	 * directory tree via the (non-standard but ubiquitous) `webkitGetAsEntry` /
@@ -1747,7 +1815,14 @@
 
 <svelte:head><title>{t('nav.files', 'Files')} · OxiCloud</title></svelte:head>
 
-<svelte:window onkeydown={onKeydown} />
+<svelte:window
+	onkeydown={onKeydown}
+	ondragenter={onWindowDragEnter}
+	ondragover={onWindowDragOver}
+	ondragleave={onWindowDragLeave}
+	ondragend={onWindowDragEnd}
+	ondrop={onWindowDrop}
+/>
 
 <div class="files-page" data-testid="files-dropzone">
 	<!-- Read-only freeze banner, placed inside the listing container so it
@@ -1808,6 +1883,7 @@
 		showDotfileToggle
 		enableSystemDrop
 		onsystemdrop={onDrop}
+		systemDropOverlayActive={pageSystemDropOver}
 		groupBys={rlGroupBys}
 		bind:groupBy
 		bind:reversed