#602: clear stale search text on route/folder change (fix A) #624

Merged

Cordy merged 6 commits from fix-602 into main

2026-09-18 19:20:31 +00:00

Cordy commented

2026-09-18 19:17:13 +00:00

Owner

Fix A per Nikola's pick: search is per-view.

New pure:search-reset block: searchResets(nextPath, curPath, curView) — true on any path change or when leaving the dedicated search-results view; false on a same-path reload, so the post-rename/delete/upload refresh keeps the user's active filter (which is what the existing end-of-load() reapply was there for).
load() clears the searchbox, cancels the debounce timer and bumps searchSeq (kills in-flight header searches) whenever searchResets says so, before cwd updates.
The repro path is covered: sidebar navigation, opening folders (including shared ones), and opening a folder from search results all clear; the footer can never disagree with an invisible filter again.

Red witnessed (search-reset block absent), then green: extract fence + full web suite.

Fix A per Nikola's pick: search is per-view. - New `pure:search-reset` block: `searchResets(nextPath, curPath, curView)` — true on any path change or when leaving the dedicated search-results view; false on a same-path reload, so the post-rename/delete/upload refresh keeps the user's active filter (which is what the existing end-of-`load()` reapply was there for). - `load()` clears the searchbox, cancels the debounce timer and bumps `searchSeq` (kills in-flight header searches) whenever `searchResets` says so, before `cwd` updates. - The repro path is covered: sidebar navigation, opening folders (including shared ones), and opening a folder from search results all clear; the footer can never disagree with an invisible filter again. Red witnessed (search-reset block absent), then green: extract fence + full web suite.

Cordy added 6 commits

2026-09-18 19:17:13 +00:00