Skip to content

Conversation

@FilisLiu
Copy link
Contributor

@FilisLiu FilisLiu commented Feb 4, 2026

What?

Support ds_notice with success banner (rendered in green) and custom banner text

Why?

Use a success banner to confirm an action on a previous page was successful. For example, for steps in a long journey or small repeated tasks.
Both GOVUK and NHSUK have this so we should probably have it too

How?

ds_notice takes two more keyword arguments now - 'header' and 'type'
available types: :information and :success (default to information), each coming with their own default text which could be overrode

Testing?

All passed

Screenshots (optional)

GOVUK:
image

NHSUK:
image

Anything Else?

No

@FilisLiu FilisLiu changed the title Feature/notification type ds_notice with success banner and custom banner text Feb 4, 2026
@FilisLiu FilisLiu requested a review from timgentry February 4, 2026 14:01
@FilisLiu FilisLiu marked this pull request as ready for review February 4, 2026 14:01
Copy link
Contributor

@shilpigoeldev shilpigoeldev left a comment

Choose a reason for hiding this comment

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

comment/query inline

@FilisLiu FilisLiu merged commit 58d511c into main Feb 5, 2026
1 of 3 checks passed
@FilisLiu FilisLiu deleted the feature/notification-type branch February 5, 2026 09:35
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.

3 participants