Skip to content

Commit abc9b8f

Browse files
Formatting
1 parent 7eb6bf4 commit abc9b8f

File tree

3 files changed

+10
-8
lines changed

3 files changed

+10
-8
lines changed

src/components/content-presentation/notification-banner/NotificationBanner.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ export interface NotificationBannerProps extends ComponentPropsWithoutRef<'div'>
2626
const NotificationBannerComponent = forwardRef<HTMLDivElement, NotificationBannerProps>(
2727
(props, forwardedRef) => {
2828
const { children, className, title, titleId, success, role, disableAutoFocus, ...rest } = props;
29+
2930
const [moduleRef] = useState(() => forwardedRef || createRef<HTMLDivElement>());
3031
const [instanceError, setInstanceError] = useState<Error>();
3132
const [instance, setInstance] = useState<NotificationBannerModule>();
@@ -47,6 +48,7 @@ const NotificationBannerComponent = forwardRef<HTMLDivElement, NotificationBanne
4748
className: 'nhsuk-notification-banner__title',
4849
}),
4950
);
51+
5052
const titleElementId = titleElement?.props.id || titleId || 'nhsuk-notification-banner-title';
5153

5254
const contentItems = items.filter((child) => child !== titleElement);
@@ -73,12 +75,12 @@ const NotificationBannerComponent = forwardRef<HTMLDivElement, NotificationBanne
7375
{titleElement ? (
7476
<>{titleElement}</>
7577
) : (
76-
<NotificationBannerTitle success={success} id={titleElementId}>
78+
<NotificationBannerTitle id={titleElementId} success={success}>
7779
{title}
7880
</NotificationBannerTitle>
7981
)}
8082
</div>
81-
<div className={'nhsuk-notification-banner__content'}>{contentItems}</div>
83+
<div className="nhsuk-notification-banner__content">{contentItems}</div>
8284
</div>
8385
);
8486
},

src/components/content-presentation/notification-banner/__tests__/NotificationBanner.test.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ describe('NotificationBanner', () => {
3030

3131
it('matches snapshot custom title', async () => {
3232
const { container } = await renderClient(
33-
<NotificationBanner title={'Upcoming maintenance'}>
33+
<NotificationBanner title="Upcoming maintenance">
3434
<NotificationBanner.Heading>
3535
The service will be unavailable from 8pm to 9pm on Thursday 1 January 2025.
3636
</NotificationBanner.Heading>
@@ -194,7 +194,7 @@ describe('NotificationBanner', () => {
194194

195195
it('matches snapshot custom title (via server)', async () => {
196196
const { container } = await renderServer(
197-
<NotificationBanner title={'Upcoming maintenance'}>
197+
<NotificationBanner title="Upcoming maintenance">
198198
<NotificationBanner.Heading>
199199
The service will be unavailable from 8pm to 9pm on Thursday 1 January 2025.
200200
</NotificationBanner.Heading>
@@ -208,7 +208,7 @@ describe('NotificationBanner', () => {
208208
it('matches snapshot custom title html (via server)', async () => {
209209
const { container } = await renderServer(
210210
<NotificationBanner>
211-
<NotificationBanner.Title id={'custom-title'}>
211+
<NotificationBanner.Title id="custom-title">
212212
<strong>Very</strong> important information
213213
</NotificationBanner.Title>
214214
<NotificationBanner.Heading>
@@ -390,8 +390,8 @@ describe('NotificationBanner', () => {
390390

391391
it('prioritises id of title element over provided title id', async () => {
392392
const { modules } = await renderClient(
393-
<NotificationBanner titleId={'wrong-id'}>
394-
<NotificationBanner.Title id={'correct-id'}>Important information</NotificationBanner.Title>
393+
<NotificationBanner titleId="wrong-id">
394+
<NotificationBanner.Title id="correct-id">Important information</NotificationBanner.Title>
395395
<NotificationBanner.Heading>
396396
The service will be unavailable from 8pm to 9pm on Thursday 1 January 2025.
397397
</NotificationBanner.Heading>

stories/Content Presentation/NotificationBanner.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ export const StandardPanelWithCustomTitleElement: Story = {
9393
render: () => (
9494
<NotificationBanner>
9595
<NotificationBanner.Title>
96-
Maintenance <time dateTime={'2025-01-01'}>January 1</time>
96+
Maintenance <time dateTime="2025-01-01">January 1</time>
9797
</NotificationBanner.Title>
9898
<NotificationBanner.Heading>Upcoming maintenance</NotificationBanner.Heading>
9999
<p>The service will be unavailable from 8pm to 9pm on Thursday 1 January 2025.</p>

0 commit comments

Comments
 (0)