Skip to content

feat: replace save toast notifications with header status indicator#6

Merged
j4rviscmd merged 1 commit intomainfrom
feat/save-status-indicator
Mar 22, 2026
Merged

feat: replace save toast notifications with header status indicator#6
j4rviscmd merged 1 commit intomainfrom
feat/save-status-indicator

Conversation

@j4rviscmd
Copy link
Copy Markdown
Owner

Summary

  • Replace intrusive toast.success notifications ("Saved", "Note created") with a subtle header-based status indicator
  • The indicator shows a pulsing dot while saving, a checkmark on success (auto-fades after 3s), and an error icon + text on failure
  • Error toasts (toast.error) are preserved for critical feedback that requires user attention
  • SaveStatus type is exported via the feature's public API (features/editor/index.ts)

Changes

File Change
useAutoSave.ts Added SaveStatus state + type, replaced toast.success with setSaveStatus, consolidated save success callback
Editor.tsx Added onStatusChange prop to propagate save status to parent
App.tsx Added SaveStatusIndicator in header next to ModeToggle
SaveStatusIndicator.tsx New component with pulsing dot (saving), checkmark (saved), error icon (error)
features/editor/index.ts Exported SaveStatus type via public API

Test plan

  • Type a note and verify a pulsing dot appears while saving
  • Verify the checkmark appears after save and fades out after ~3 seconds
  • Verify the checkmark stays visible even when typing resumes before the 3s timer expires
  • Verify error state shows "Save failed" text with warning icon
  • Verify toast.error still appears for save failures
  • Verify no toast appears on successful save or note creation
  • Test in both light and dark themes

🤖 Generated with Claude Code

Replace intrusive toast.success notifications ("Saved", "Note created")
with a subtle header-based status indicator. The indicator shows a
pulsing dot while saving, a checkmark on success (auto-fades after 3s),
and an error icon on failure. Error toasts are preserved for critical
feedback. SaveStatus type is now exported via the feature's public API.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@j4rviscmd j4rviscmd added the enhancement New feature or request label Mar 22, 2026
@j4rviscmd j4rviscmd merged commit d330c35 into main Mar 22, 2026
@j4rviscmd j4rviscmd deleted the feat/save-status-indicator branch March 22, 2026 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant