hanh.tonguyen/paperclip
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: add severity-based TTL defaults for toast notifications

Browse Source 
Info/success toasts auto-dismiss faster (3.5-4s) while warn/error
toasts persist longer (8-10s). Callers can still override with
explicit ttlMs. Dedupe, cooldown window, and MAX_TOASTS cap were
already in place from Phase 1.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Forgotten
2026-02-20 13:53:51 -06:00
parent 45f67ae311
commit 5b06118ec8
1 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
ui/src/context/ToastContext.tsx
View File

@@ -43,7 +43,12 @@ interface ToastContextValue {
clearToasts: () => void; clearToasts: () => void;
} }
const DEFAULT_TTL_MS = 5000; const DEFAULT_TTL_BY_TONE: Record<ToastTone, number> = {
info: 4000,
success: 3500,
warn: 8000,
error: 10000,
};
const MIN_TTL_MS = 1500; const MIN_TTL_MS = 1500;
const MAX_TTL_MS = 15000; const MAX_TTL_MS = 15000;
const MAX_TOASTS = 5; const MAX_TOASTS = 5;
@@ -52,8 +57,8 @@ const DEDUPE_MAX_AGE_MS = 20000;
const ToastContext = createContext<ToastContextValue | null>(null); const ToastContext = createContext<ToastContextValue | null>(null);
function normalizeTtl(value: number | undefined) { function normalizeTtl(value: number | undefined, tone: ToastTone) {
const fallback = DEFAULT_TTL_MS; const fallback = DEFAULT_TTL_BY_TONE[tone];
if (typeof value !== "number" || !Number.isFinite(value)) return fallback; if (typeof value !== "number" || !Number.isFinite(value)) return fallback;
return Math.max(MIN_TTL_MS, Math.min(MAX_TTL_MS, Math.floor(value))); return Math.max(MIN_TTL_MS, Math.min(MAX_TTL_MS, Math.floor(value)));
} }
@@ -95,7 +100,7 @@ export function ToastProvider({ children }: { children: ReactNode }) {
(input: ToastInput) => { (input: ToastInput) => {
const now = Date.now(); const now = Date.now();
const tone = input.tone ?? "info"; const tone = input.tone ?? "info";
const ttlMs = normalizeTtl(input.ttlMs); const ttlMs = normalizeTtl(input.ttlMs, tone);
const dedupeKey = const dedupeKey =
input.dedupeKey ?? input.id ?? `${tone}|${input.title}|${input.body ?? ""}|${input.action?.href ?? ""}`; input.dedupeKey ?? input.id ?? `${tone}|${input.title}|${input.body ?? ""}|${input.action?.href ?? ""}`;