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

Tweak toast docs #33810

Merged
merged 3 commits into from
May 4, 2021
Merged

Tweak toast docs #33810

merged 3 commits into from
May 4, 2021

Conversation

patrickhlauke
Copy link
Member

  • a few rewordings/tweaks
  • add info about toasts with focusable/actionable controls (shouldn't autohide them)

- a few rewordings/tweaks
- add info about toasts with focusable/actionable controls (shouldn't autohide them)
Copy link
Member

@mdo mdo left a comment

Choose a reason for hiding this comment

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

Couple copy tweaks

site/content/docs/5.0/components/toasts.md Outdated Show resolved Hide resolved
site/content/docs/5.0/components/toasts.md Outdated Show resolved Hide resolved
@XhmikosR XhmikosR merged commit cb38744 into main May 4, 2021
@XhmikosR XhmikosR deleted the patrichlauke-toast-docs-tweak branch May 4, 2021 12:37
@XhmikosR
Copy link
Member

XhmikosR commented Oct 8, 2021

@patrickhlauke I'm going to remove this from the 4.6.1 project, but if you backport it we can include it.

@patrickhlauke patrickhlauke mentioned this pull request Jan 2, 2022
patrickhlauke added a commit that referenced this pull request Jan 2, 2022
- a few rewordings/tweaks
- add info about toasts with focusable/actionable controls (shouldn't autohide them)

manual backport of #33810
XhmikosR pushed a commit that referenced this pull request Jan 3, 2022
- a few rewordings/tweaks
- add info about toasts with focusable/actionable controls (shouldn't autohide them)

manual backport of #33810
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants