-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
Update toast outlines for persistent toasts #371
Conversation
🦋 Changeset detectedLatest commit: 77ae30a The changes in this PR will be included in the next version bump. This PR includes changesets to release 13 packages
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 |
Thank you for submitting your Pull Request, the following links will become available for preview shortly:
|
The preview deployment failed. 🔴 Last updated at: 2024-11-02 12:01:36 CET |
The preview deployment is ready. 🟢 Open Preview | Open Build Logs Last updated at: 2024-11-03 10:10:50 CET |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Nice catch too, that's a very common accessibility miss (not using a border for separation when the BG of the two elements doesn't have enough contrast)
Thank you for submitting your Pull Request, the following links will become available for preview shortly:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @louisescher 😁
Description