-
Notifications
You must be signed in to change notification settings - Fork 127
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add remove toast callback #2823
Conversation
🦋 Changeset detectedLatest commit: 9830fe5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Size Change: +23 B (0%) Total Size: 670 kB
ℹ️ View Unchanged
|
422a1d3
to
9830fe5
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2823 +/- ##
=======================================
Coverage 87.91% 87.91%
=======================================
Files 218 218
Lines 12725 12729 +4
Branches 1726 1727 +1
=======================================
+ Hits 11187 11191 +4
Misses 1486 1486
Partials 52 52
|
There are concerns that the behaviour might introduce UX issues, will close the PR |
Purpose
Allow users to dismiss a Toast programmatically. Sometimes this can be beneficial to avoid showing too many toasts or dismissing toasts after navigating away.
Approach and changes
Return a callback function from the
setToast
function that removes the toast from the context.Definition of done