Skip to content
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

Updated width of toast to a set size #594

Merged
merged 3 commits into from
Sep 25, 2022
Merged

Updated width of toast to a set size #594

merged 3 commits into from
Sep 25, 2022

Conversation

karnikaavelumani
Copy link
Collaborator

Previously, the width of the toast was set to the length of the message. This caused the appearance of the notification to result in different sizes when copying to the outlook calendar versus copying to the google calendar. It has now updated to a set size.

toastwidth

Fixes #408.

@vercel
Copy link

vercel bot commented Sep 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
acm-csuf-site ✅ Ready (Inspect) Visit Preview Sep 25, 2022 at 1:02AM (UTC)

@vercel vercel bot temporarily deployed to Preview September 25, 2022 01:02 Inactive
Copy link
Owner

@EthanThatOneKid EthanThatOneKid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet! LGTM!

Copy link
Contributor

@jaasonw jaasonw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👎

@karnikaavelumani karnikaavelumani merged commit 2e1bc50 into main Sep 25, 2022
@jaasonw jaasonw deleted the fix/408 branch October 5, 2022 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

toast is not max width'd
3 participants