Skip to content

Commit 6ded996

Browse files
committed
CCM-11496: use summary list
1 parent 4cba150 commit 6ded996

File tree

3 files changed

+26
-53
lines changed

3 files changed

+26
-53
lines changed

frontend/src/__tests__/app/message-plans/get-ready-to-move/__snapshots__/page.test.tsx.snap

Lines changed: 15 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -23,28 +23,24 @@ exports[`matches snapshot 1`] = `
2323
>
2424
Get ready to move message plan to production
2525
</h1>
26-
<table
27-
class="nhsuk-u-margin-bottom-6 table"
26+
<dl
27+
class="nhsuk-summary-list"
2828
>
29-
<tbody
30-
class="nhsuk-table__body"
29+
<div
30+
class="nhsuk-summary-list__row"
3131
>
32-
<tr
33-
class="nhsuk-table__row"
32+
<dt
33+
class="nhsuk-summary-list__key"
3434
>
35-
<th
36-
class="nhsuk-table__cell"
37-
>
38-
Name
39-
</th>
40-
<td
41-
class="nhsuk-table__cell"
42-
>
43-
My Routing Config
44-
</td>
45-
</tr>
46-
</tbody>
47-
</table>
35+
Name
36+
</dt>
37+
<dd
38+
class="nhsuk-summary-list__value"
39+
>
40+
My Routing Config
41+
</dd>
42+
</div>
43+
</dl>
4844
<p>
4945
Moving message plans from draft to production means they are ready to send.
5046
</p>

frontend/src/app/message-plans/get-ready-to-move/[routingConfigId]/page.module.scss

Lines changed: 0 additions & 22 deletions
This file was deleted.

frontend/src/app/message-plans/get-ready-to-move/[routingConfigId]/page.tsx

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import classNames from 'classnames';
21
import type { Metadata } from 'next';
32
import { redirect, RedirectType } from 'next/navigation';
43
import type { MessagePlanPageProps } from 'nhs-notify-web-template-management-utils';
@@ -7,7 +6,6 @@ import { NHSNotifyMain } from '@atoms/NHSNotifyMain/NHSNotifyMain';
76
import content from '@content/content';
87
import { ContentRenderer } from '@molecules/ContentRenderer/ContentRenderer';
98
import { getRoutingConfig } from '@utils/message-plans';
10-
import styles from './page.module.scss';
119

1210
const pageContent = content.pages.messagePlanGetReadyToMoveToProduction();
1311

@@ -32,17 +30,18 @@ export default async function MessagePlanGetReadyToMoveToProductionPage({
3230
<div className='nhsuk-grid-column-two-thirds'>
3331
<span className='nhsuk-caption-xl'>{pageContent.stepCounter}</span>
3432
<h1 className='nhsuk-heading-xl'>{pageContent.heading}</h1>
35-
<table
36-
className={classNames('nhsuk-u-margin-bottom-6', styles.table)}
37-
>
38-
<tbody className='nhsuk-table__body'>
39-
<tr className='nhsuk-table__row'>
40-
<th className='nhsuk-table__cell'>Name</th>
41-
<td className='nhsuk-table__cell'>{routingConfig.name}</td>
42-
</tr>
43-
</tbody>
44-
</table>
33+
34+
<dl className='nhsuk-summary-list'>
35+
<div className='nhsuk-summary-list__row'>
36+
<dt className='nhsuk-summary-list__key'>Name</dt>
37+
<dd className='nhsuk-summary-list__value'>
38+
{routingConfig.name}
39+
</dd>
40+
</div>
41+
</dl>
42+
4543
<ContentRenderer content={pageContent.content} />
44+
4645
<div className='nhsuk-warning-callout'>
4746
<h3 className='nhsuk-warning-callout__label'>
4847
{pageContent.callout.label}

0 commit comments

Comments
 (0)