improvement(toast): match notification styling with countdown ring and consistent design#3688
Conversation
…d consistent design
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Adds an animated SVG countdown ring that appears for timed toasts and removes the Written by Cursor Bugbot for commit eeebcb8. Configure here. |
Greptile SummaryThis PR reskins the toast component to visually match the Key changes:
Confidence Score: 4/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant Caller
participant toast as toast() / useToast
participant ToastProvider
participant ToastItem
participant CountdownRing
Caller->>toast: toast.success("msg", { duration: 5000 })
toast->>ToastProvider: addToast({ message, variant, duration })
ToastProvider->>ToastProvider: append ToastData to state
ToastProvider->>ToastItem: render <ToastItem toast={t} />
ToastItem->>CountdownRing: render (if t.duration > 0)
CountdownRing-->>ToastItem: SVG with notification-countdown CSS animation
ToastItem->>ToastItem: useEffect → setTimeout(dismiss, duration)
Note over ToastItem,CountdownRing: Ring animation & dismiss timer start together
ToastItem->>ToastItem: setExiting(true) → toast-exit animation
ToastItem->>ToastProvider: onDismiss(id) after EXIT_ANIMATION_MS
ToastProvider->>ToastProvider: filter toast from state
Last reviewed commit: "fix(toast): add succ..." |
|
@greptile |
|
@cursor review |
Summary
notification-countdownkeyframe)descriptionfield from toast typesType of Change
Testing
Tested manually
Checklist