sinvo/docmost
1
0
Fork 0
mirror of https://github.com/docmost/docmost.git synced 2026-08-27 08:47:06 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(base): mirror filter-clear fix for sorts so deleting the last sort sticks

Browse Source
This commit is contained in:
Philipinho
2026-04-24 03:04:51 +01:00
parent 2db43788d5
commit dcff8d3c53
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
apps/client/src/features/base/hooks/use-view-draft.ts
+8 -2
View File
@@ -91,7 +91,13 @@ export function useViewDraft(args: UseViewDraftArgs): ViewDraftState {
if (!ready) return; if (!ready) return;
const current = storedDraft ?? null; const current = storedDraft ?? null;
const mergedFilter = current?.filter; const mergedFilter = current?.filter;
const mergedSorts = next; // If the baseline has sorts, clearing to `undefined` would fall back
// to the baseline in effectiveSorts. Persist an empty array instead
// so the draft explicitly overrides the baseline with no sorts.
const mergedSorts =
next === undefined && baselineSorts !== undefined && baselineSorts.length > 0
? []
: next;
if (mergedFilter === undefined && (mergedSorts === undefined || mergedSorts === null)) { if (mergedFilter === undefined && (mergedSorts === undefined || mergedSorts === null)) {
setDraft(RESET); setDraft(RESET);
return; return;
@@ -102,7 +108,7 @@ export function useViewDraft(args: UseViewDraftArgs): ViewDraftState {
updatedAt: new Date().toISOString(), updatedAt: new Date().toISOString(),
}); });
}, },
[ready, storedDraft, setDraft], [ready, storedDraft, setDraft, baselineSorts],
); );
const reset = useCallback(() => { const reset = useCallback(() => {