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

Remove error notification banner type #2030

Merged
merged 2 commits into from
Nov 20, 2020

Conversation

vanitabarrett
Copy link
Contributor

What

Remove 'error' type notification banner

Why

We struggled to find a use case for the error banner that wasn't already covered by the error summary, an information notification banner or another component/pattern. For now, we're going to release the success and information banner and add the error banner later if we identify a strong use case.

We struggled to find a use case for the error banner that wasn't already covered by the error summary, an information notification banner or another component/pattern. For now, we're going to release the success and information banner and add the error banner later if we identify a strong use case.
@vanitabarrett vanitabarrett added this to Needs review in Design System Sprint Board via automation Nov 20, 2020
@vanitabarrett vanitabarrett added this to the v3.10.0 milestone Nov 20, 2020
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-2030 November 20, 2020 11:11 Inactive
@vanitabarrett vanitabarrett marked this pull request as ready for review November 20, 2020 11:19
Copy link
Contributor

@EoinShaughnessy EoinShaughnessy left a comment

Choose a reason for hiding this comment

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

Text changes look good!

@vanitabarrett vanitabarrett merged commit 0ac715b into master Nov 20, 2020
Design System Sprint Board automation moved this from Needs review to Done Nov 20, 2020
@vanitabarrett vanitabarrett deleted the remove-error-option-notif-banner branch November 20, 2020 16:02
@vanitabarrett vanitabarrett mentioned this pull request Nov 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants