@ukhomeoffice/cop-react-form-renderer 1.0.0 → 2.2.0

package/dist/components/FormRenderer/helpers/getNextPageId.test.js

@@ -11,11 +11,87 @@ describe('components', function () {
   describe('FormRenderer', function () {
     describe('helpers', function () {
       describe('getNextPageId', function () {
+        var OPTIONS = [{
+          value: 'romeo',
+          label: 'Romeo'
+        }, {
+          value: 'juliet',
+          label: 'Juliet'
+        }];
+        var RADIOS = {
+          id: 'radios',
+          fieldId: 'radios',
+          type: _models.ComponentTypes.RADIOS,
+          data: {
+            options: OPTIONS
+          }
+        };
+        var TEXT = {
+          id: 'text',
+          fieldId: 'text',
+          type: _models.ComponentTypes.TEXT
+        };
+        var HTML = {
+          type: _models.ComponentTypes.HTML,
+          content: 'HTML'
+        };
+        var SHOW_WHEN_JULIET = {
+          field: RADIOS.fieldId,
+          op: '=',
+          value: OPTIONS[1].value
+        };
+        var SHOW_WHEN_ROMEO = {
+          field: RADIOS.fieldId,
+          op: '=',
+          value: OPTIONS[0].value
+        };
+        var FORM_DATA = {
+          radios: OPTIONS[0].value
+        }; // 'romeo'
+
         var PAGES = [{
           id: 'alpha'
         }, {
           id: 'bravo'
+        }, {
+          id: 'charlie',
+          components: [RADIOS]
+        }, {
+          id: 'delta',
+          components: [TEXT],
+          show_when: SHOW_WHEN_JULIET
+        }, {
+          id: 'echo',
+          components: [HTML]
+        }, {
+          id: 'foxtrot',
+          components: [TEXT],
+          show_when: SHOW_WHEN_ROMEO
         }];
+        describe("when the action type is '".concat(_models.PageAction.TYPES.NAVIGATE, "'"), function () {
+          it('should return undefined if the action has no page property', function () {
+            var ACTION = {
+              type: _models.PageAction.TYPES.NAVIGATE
+            };
+            expect((0, _getNextPageId.default)(_models.FormTypes.HUB, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toBeUndefined();
+          });
+          it('should return the page on the action if it exists among the pages', function () {
+            var PAGE = PAGES[1].id;
+            var ACTION = {
+              type: _models.PageAction.TYPES.NAVIGATE,
+              page: PAGE
+            };
+            expect((0, _getNextPageId.default)(_models.FormTypes.HUB, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toEqual(PAGE);
+          });
+          it('should return undefined if the page on the action does not exist among the pages', function () {
+            var PAGE = 'golf';
+            var ACTION = {
+              type: _models.PageAction.TYPES.NAVIGATE,
+              page: PAGE
+            };
+            expect((0, _getNextPageId.default)(_models.FormTypes.HUB, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toBeUndefined();
+          });
+        });
         describe("when the form type is '".concat(_models.FormTypes.HUB, "'"), function () {
           var FORM_TYPE = _models.FormTypes.HUB;
           it("should return '".concat(_models.FormPages.HUB, "' by default"), function () {
@@ -23,13 +99,13 @@ describe('components', function () {
           });
           it("should return '".concat(_models.FormPages.HUB, "' if the action is '").concat(_models.PageAction.TYPES.SAVE_AND_RETURN, "'"), function () {
             var ACTION = _models.PageAction.DEFAULTS.saveAndReturn;
-            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, PAGES[0].id, ACTION)).toEqual(_models.FormPages.HUB);
+            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toEqual(_models.FormPages.HUB);
           });
-          it('should return action.nextPageId if specified', function () {
+          it('should return action.page if specified', function () {
             var ACTION = {
-              nextPageId: 'bob'
+              page: 'bob'
             };
-            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, undefined, ACTION)).toEqual(ACTION.nextPageId);
+            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, undefined, ACTION, FORM_DATA)).toEqual(ACTION.page);
           });
         });
         describe("when the form type is '".concat(_models.FormTypes.CYA, "'"), function () {
@@ -40,12 +116,18 @@ describe('components', function () {
           it('should return the second page when on the first page', function () {
             expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, PAGES[0].id)).toEqual(PAGES[1].id);
           });
-          it("should return '".concat(_models.FormPages.CYA, "' when on the last page"), function () {
-            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, PAGES[PAGES.length - 1].id)).toEqual(_models.FormPages.CYA);
+          it('should return the third page when on the second page', function () {
+            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, PAGES[1].id)).toEqual(PAGES[2].id);
+          });
+          it("should return the fifth page when on the third page because the fourth page show_when is not met", function () {
+            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, PAGES[2].id, _models.PageAction.DEFAULTS.saveAndContinue, FORM_DATA)).toEqual(PAGES[4].id);
+          });
+          it("should return the sixth page when on the fifth page because the sixth page show_when is met", function () {
+            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, PAGES[4].id, _models.PageAction.DEFAULTS.saveAndContinue, FORM_DATA)).toEqual(PAGES[5].id);
           });
           it("should return undefined if the action is '".concat(_models.PageAction.TYPES.SAVE_AND_RETURN, "'"), function () {
             var ACTION = _models.PageAction.DEFAULTS.saveAndReturn;
-            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, PAGES[0].id, ACTION)).toBeUndefined();
+            expect((0, _getNextPageId.default)(FORM_TYPE, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toBeUndefined();
           });
         });
         describe("when the form type is '".concat(_models.FormTypes.WIZARD, "'"), function () {

package/dist/components/FormRenderer/helpers/getSubmissionStatus.js

@@ -0,0 +1,27 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = void 0;
+
+var _models = require("../../../models");
+
+var _getNextPageId = _interopRequireDefault(require("./getNextPageId"));
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+
+var getSubmissionStatus = function getSubmissionStatus(formType, pages, currentPageId, action, formData) {
+  if ((action === null || action === void 0 ? void 0 : action.type) === _models.PageAction.TYPES.SAVE_AND_RETURN) {
+    return {
+      page: currentPageId
+    };
+  }
+
+  return {
+    page: (0, _getNextPageId.default)(formType, pages, currentPageId, action, formData)
+  };
+};
+
+var _default = getSubmissionStatus;
+exports.default = _default;

package/dist/components/FormRenderer/helpers/getSubmissionStatus.test.js

@@ -0,0 +1,209 @@
+"use strict";
+
+var _models = require("../../../models");
+
+var _getSubmissionStatus = _interopRequireDefault(require("./getSubmissionStatus"));
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+
+// Local imports
+describe('components', function () {
+  describe('FormRenderer', function () {
+    describe('helpers', function () {
+      describe('getSubmissionStatus', function () {
+        var OPTIONS = [{
+          value: 'romeo',
+          label: 'Romeo'
+        }, {
+          value: 'juliet',
+          label: 'Juliet'
+        }];
+        var RADIOS = {
+          id: 'radios',
+          fieldId: 'radios',
+          type: _models.ComponentTypes.RADIOS,
+          data: {
+            options: OPTIONS
+          }
+        };
+        var TEXT = {
+          id: 'text',
+          fieldId: 'text',
+          type: _models.ComponentTypes.TEXT
+        };
+        var HTML = {
+          type: _models.ComponentTypes.HTML,
+          content: 'HTML'
+        };
+        var SHOW_WHEN_JULIET = {
+          field: RADIOS.fieldId,
+          op: '=',
+          value: OPTIONS[1].value
+        };
+        var SHOW_WHEN_ROMEO = {
+          field: RADIOS.fieldId,
+          op: '=',
+          value: OPTIONS[0].value
+        };
+        var FORM_DATA = {
+          radios: OPTIONS[0].value
+        }; // 'romeo'
+
+        var PAGES = [{
+          id: 'alpha'
+        }, {
+          id: 'bravo'
+        }, {
+          id: 'charlie',
+          components: [RADIOS]
+        }, {
+          id: 'delta',
+          components: [TEXT],
+          show_when: SHOW_WHEN_JULIET
+        }, {
+          id: 'echo',
+          components: [HTML]
+        }, {
+          id: 'foxtrot',
+          components: [TEXT],
+          show_when: SHOW_WHEN_ROMEO
+        }];
+        describe("when the action type is '".concat(_models.PageAction.TYPES.NAVIGATE, "'"), function () {
+          it('should return undefined if the action has no page property', function () {
+            var ACTION = {
+              type: _models.PageAction.TYPES.NAVIGATE
+            };
+            expect((0, _getSubmissionStatus.default)(_models.FormTypes.HUB, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toMatchObject({
+              page: undefined
+            });
+          });
+          it('should return the page on the action if it exists among the pages', function () {
+            var PAGE = PAGES[1].id;
+            var ACTION = {
+              type: _models.PageAction.TYPES.NAVIGATE,
+              page: PAGE
+            };
+            expect((0, _getSubmissionStatus.default)(_models.FormTypes.HUB, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toMatchObject({
+              page: PAGE
+            });
+          });
+          it('should return undefined if the page on the action does not exist among the pages', function () {
+            var PAGE = 'golf';
+            var ACTION = {
+              type: _models.PageAction.TYPES.NAVIGATE,
+              page: PAGE
+            };
+            expect((0, _getSubmissionStatus.default)(_models.FormTypes.HUB, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toMatchObject({
+              page: undefined
+            });
+          });
+        });
+        describe("when the action type is '".concat(_models.PageAction.TYPES.SAVE_AND_RETURN, "'"), function () {
+          Object.values(_models.FormTypes).forEach(function (formType) {
+            it("should return the current page if the form type is '".concat(formType, "'"), function () {
+              var ACTION = _models.PageAction.DEFAULTS.saveAndReturn;
+              expect((0, _getSubmissionStatus.default)(formType, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toMatchObject({
+                page: PAGES[0].id
+              });
+            });
+          });
+        });
+        describe("when the form type is '".concat(_models.FormTypes.HUB, "'"), function () {
+          var FORM_TYPE = _models.FormTypes.HUB;
+          it("should return '".concat(_models.FormPages.HUB, "' by default"), function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES)).toMatchObject({
+              page: _models.FormPages.HUB
+            });
+          });
+          it('should return action.page if specified', function () {
+            var ACTION = {
+              page: 'bob'
+            };
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, undefined, ACTION, FORM_DATA)).toMatchObject({
+              page: ACTION.page
+            });
+          });
+        });
+        describe("when the form type is '".concat(_models.FormTypes.CYA, "'"), function () {
+          var FORM_TYPE = _models.FormTypes.CYA;
+          var ACTION = _models.PageAction.DEFAULTS.saveAndContinue;
+          it('should return the first page by default', function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, ACTION, FORM_DATA)).toMatchObject({
+              page: PAGES[0].id
+            });
+          });
+          it('should return the second page when on the first page', function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[0].id, ACTION, FORM_DATA)).toMatchObject({
+              page: PAGES[1].id
+            });
+          });
+          it('should return the third page when on the second page', function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[1].id, ACTION, FORM_DATA)).toMatchObject({
+              page: PAGES[2].id
+            });
+          });
+          it("should return the fifth page when on the third page because the fourth page show_when is not met", function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[2].id, ACTION, FORM_DATA)).toMatchObject({
+              page: PAGES[4].id
+            });
+          });
+          it("should return the sixth page when on the fifth page because the sixth page show_when is met", function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[4].id, ACTION, FORM_DATA)).toMatchObject({
+              page: PAGES[5].id
+            });
+          });
+          it("should return '".concat(_models.FormPages.CYA, "' when on the sixth (last) page"), function () {
+            var lastPageIndex = PAGES.length - 1;
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[lastPageIndex].id, ACTION, FORM_DATA)).toMatchObject({
+              page: _models.FormPages.CYA
+            });
+          });
+        });
+        describe("when the form type is '".concat(_models.FormTypes.WIZARD, "'"), function () {
+          var FORM_TYPE = _models.FormTypes.WIZARD;
+          it('should return the first page by default', function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES)).toMatchObject({
+              page: PAGES[0].id
+            });
+          });
+          it('should return the second page when on the first page', function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[0].id)).toMatchObject({
+              page: PAGES[1].id
+            });
+          });
+          it('should return undefined when on the last page', function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[PAGES.length - 1].id)).toMatchObject({
+              page: undefined
+            });
+          });
+        });
+        describe("when the form type is '".concat(_models.FormTypes.FORM, "'"), function () {
+          var FORM_TYPE = _models.FormTypes.FORM;
+          it("should always return the first page if there any pages and the action is not '".concat(_models.PageAction.TYPES.SAVE_AND_RETURN, "'"), function () {
+            var ACTION = {
+              type: _models.PageAction.TYPES.SUBMIT,
+              nextPageId: 'bob'
+            };
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES)).toMatchObject({
+              page: PAGES[0].id
+            });
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[0].id)).toMatchObject({
+              page: PAGES[0].id
+            });
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[1].id)).toMatchObject({
+              page: PAGES[0].id
+            });
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, PAGES, PAGES[0].id, ACTION)).toMatchObject({
+              page: PAGES[0].id
+            });
+          });
+          it('should return undefined if there are no pages', function () {
+            expect((0, _getSubmissionStatus.default)(FORM_TYPE, [])).toMatchObject({
+              page: undefined
+            });
+          });
+        });
+      });
+    });
+  });
+});

package/dist/components/FormRenderer/helpers/index.js

@@ -13,6 +13,8 @@ var _getFormState = _interopRequireDefault(require("./getFormState"));
 
 var _getNextPageId = _interopRequireDefault(require("./getNextPageId"));
 
+var _getSubmissionStatus = _interopRequireDefault(require("./getSubmissionStatus"));
+
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 // Local imports
@@ -20,7 +22,8 @@ var helpers = {
   canActionProceed: _canActionProceed.default,
   canCYASubmit: _canCYASubmit.default,
   getFormState: _getFormState.default,
-  getNextPageId: _getNextPageId.default
+  getNextPageId: _getNextPageId.default,
+  getSubmissionStatus: _getSubmissionStatus.default
 };
 exports.helpers = helpers;
 var _default = helpers;

package/dist/components/PageActions/PageActions.stories.mdx

@@ -43,7 +43,7 @@ Renders the buttons at the bottom of a Page.
 <Canvas>
   <Story name="Custom action">
     {() => {
-      const ACTIONS = [{ type: 'navigate', url: '/profile', label: 'Go to profile' }];
+      const ACTIONS = [{ type: 'navigate', page: 'profile', label: 'Go to profile' }];
       const ON_ACTION = (action, patch, onError) => {
         console.log('action invoked', action, patch);
       };

package/dist/components/PageActions/PageActions.test.js

@@ -87,7 +87,7 @@ describe('components', function () {
             case 0:
               ACTIONS = [{
                 type: 'navigate',
-                url: '/alpha',
+                page: 'alpha',
                 label: 'Alpha'
               }];
               _render3 = (0, _react.render)( /*#__PURE__*/_react2.default.createElement(_PageActions.default, {
@@ -121,7 +121,7 @@ describe('components', function () {
             case 0:
               NAVIGATE = {
                 type: _models.PageAction.TYPES.NAVIGATE,
-                url: '/alpha',
+                page: 'alpha',
                 label: 'Alpha'
               };
               ACTIONS = [NAVIGATE, _models.PageAction.TYPES.SUBMIT];

package/dist/components/SummaryList/RowAction.js

@@ -56,7 +56,7 @@ RowAction.propTypes = {
   row: _propTypes.default.shape({
     action: _propTypes.default.shape({
       label: _propTypes.default.string.isRequired,
-      href: _propTypes.default.string,
+      page: _propTypes.default.string,
       aria_suffix: _propTypes.default.string,
       onAction: _propTypes.default.func
     })

package/dist/components/SummaryList/RowAction.test.js

@@ -23,10 +23,10 @@ describe('components', function () {
       expect(container.childNodes.length).toEqual(0);
     });
     it('should handle a row with an href in the action', function () {
-      var HREF = 'http://alpha.homeoffice.gov.uk';
+      var PAGE = 'alpha';
       var ROW = {
         action: {
-          href: HREF,
+          page: PAGE,
           label: 'Change'
         }
       };
@@ -38,14 +38,14 @@ describe('components', function () {
 
       var link = container.childNodes[0];
       expect(link.tagName).toEqual('A');
-      expect(link.getAttribute('href')).toEqual(HREF);
+      expect(link.getAttribute('href')).toEqual("/".concat(PAGE));
       expect(link.textContent).toEqual(ROW.action.label);
     });
     it('should render an aria_suffix appropriately', function () {
-      var HREF = 'http://alpha.homeoffice.gov.uk';
+      var PAGE = 'alpha';
       var ROW = {
         action: {
-          href: HREF,
+          page: PAGE,
           label: 'Change',
           aria_suffix: 'the thing'
         }
@@ -58,7 +58,7 @@ describe('components', function () {
 
       var link = container.childNodes[0];
       expect(link.tagName).toEqual('A');
-      expect(link.getAttribute('href')).toEqual(HREF);
+      expect(link.getAttribute('href')).toEqual("/".concat(PAGE));
       expect(link.textContent).toEqual("".concat(ROW.action.label, " ").concat(ROW.action.aria_suffix));
     });
     it('should handle a row with an onAction in the action', function () {

package/dist/components/SummaryList/SummaryList.js

@@ -15,7 +15,7 @@ var _RowAction = _interopRequireDefault(require("./RowAction"));
 
 require("./SummaryList.scss");
 
-var _excluded = ["rows", "classBlock", "classModifiers", "className"];
+var _excluded = ["rows", "noChangeAction", "classBlock", "classModifiers", "className"];
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
@@ -30,6 +30,7 @@ exports.DEFAULT_CLASS = DEFAULT_CLASS;
 
 var SummaryList = function SummaryList(_ref) {
   var rows = _ref.rows,
+      noChangeAction = _ref.noChangeAction,
       classBlock = _ref.classBlock,
       classModifiers = _ref.classModifiers,
       className = _ref.className,
@@ -47,7 +48,7 @@ var SummaryList = function SummaryList(_ref) {
       className: classes('key')
     }, row.key), /*#__PURE__*/_react.default.createElement("dd", {
       className: classes('value')
-    }, row.value), /*#__PURE__*/_react.default.createElement("dd", {
+    }, row.value), !noChangeAction && /*#__PURE__*/_react.default.createElement("dd", {
       className: classes('actions')
     }, row.action && /*#__PURE__*/_react.default.createElement(_RowAction.default, {
       row: row
@@ -62,7 +63,7 @@ SummaryList.propTypes = {
     key: _propTypes.default.string.isRequired,
     value: _propTypes.default.any,
     action: _propTypes.default.shape({
-      href: _propTypes.default.string,
+      page: _propTypes.default.string,
       label: _propTypes.default.string,
       aria_suffix: _propTypes.default.string,
       onAction: _propTypes.default.func
@@ -70,10 +71,12 @@ SummaryList.propTypes = {
   })).isRequired,
   classBlock: _propTypes.default.string,
   classModifiers: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.arrayOf(_propTypes.default.string)]),
-  className: _propTypes.default.string
+  className: _propTypes.default.string,
+  noChangeAction: _propTypes.default.bool
 };
 SummaryList.defaultProps = {
-  classBlock: DEFAULT_CLASS
+  classBlock: DEFAULT_CLASS,
+  noChangeAction: false
 };
 var _default = SummaryList;
 exports.default = _default;

package/dist/components/SummaryList/SummaryList.scss

@@ -1,5 +1,5 @@
-@import "node_modules/govuk-frontend/govuk/_base";
-@import "node_modules/govuk-frontend/govuk/components/summary-list/_summary-list";
+@import 'node_modules/govuk-frontend/govuk/_base';
+@import 'node_modules/govuk-frontend/govuk/components/summary-list/_summary-list';
 
 .govuk-summary-list__actions {
   .govuk-link {
@@ -7,3 +7,11 @@
     cursor: pointer;
   }
 }
+
+.caseview__key {
+  font-weight: normal;
+}
+
+.caseview__value {
+  @include govuk-typography-weight-bold;
+}

package/dist/components/SummaryList/SummaryList.stories.mdx

@@ -1,36 +1,90 @@
 <!-- Global imports -->
+
 import { ArgsTable, Canvas, Meta, Story } from '@storybook/addon-docs';
 import { Details, Heading } from '@ukhomeoffice/cop-react-components';
 
 <!-- Local imports -->
+
 import Utils from '../../utils';
 import SummaryList from './SummaryList';
 
-<Meta title="Components/Summary list" id="D-SummaryList" component={ SummaryList } />
+<Meta
+  title='Components/Summary list'
+  id='D-SummaryList'
+  component={SummaryList}
+/>
 
-<Heading size="xl" caption="Components">Summary list</Heading>
+<Heading size='xl' caption='Components'>
+  Summary list
+</Heading>
 
 Renders a list of key-value pairs, most commonly on the **Check your answers** screen, and optionally
 display action links.
 
 <Canvas withToolbar>
-  <Story name="Default">
+  <Story name='Default'>
     {() => {
       const onAction = (row) => {
         console.log('action clicked', row);
       };
       const ROWS = [
-        { pageId: 'p1', fieldId: 'forename', key: 'Forename(s)', value: 'John' },
+        {
+          pageId: 'p1',
+          fieldId: 'forename',
+          key: 'Forename(s)',
+          value: 'John',
+        },
         { pageId: 'p1', fieldId: 'surname', key: 'Surname', value: 'Smith' },
-        { pageId: 'p2', fieldId: 'dob', key: 'Date of birth', value: '29/08/1993', action: { label: 'Change', onAction } },
+        {
+          pageId: 'p2',
+          fieldId: 'dob',
+          key: 'Date of birth',
+          value: '29/08/1993',
+          action: { label: 'Change', onAction },
+        },
       ];
-      return (
-        <SummaryList rows={ROWS} />
-      );
+      return <SummaryList rows={ROWS} />;
     }}
   </Story>
 </Canvas>
 
-<Details summary="Properties" className="no-indent">
-  <ArgsTable of={ SummaryList } />
+<Details summary='Properties' className='no-indent'>
+  <ArgsTable of={SummaryList} />
 </Details>
+
+## Variant
+
+### Read-only style
+
+<Canvas>
+  <Story name='Read-only style'>
+    {() => {
+      const onAction = (row) => {
+        console.log('action clicked', row);
+      };
+      const ROWS = [
+        {
+          pageId: 'p1',
+          fieldId: 'forename',
+          key: 'Forename(s)',
+          value: 'John',
+        },
+        { pageId: 'p1', fieldId: 'surname', key: 'Surname', value: 'Smith' },
+        {
+          pageId: 'p2',
+          fieldId: 'dob',
+          key: 'Date of birth',
+          value: '29/08/1993',
+          action: { label: 'Change', onAction },
+        },
+      ];
+      return (
+        <SummaryList
+          classModifiers='no-border'
+          rows={ROWS}
+          noChangeAction={true}
+        />
+      );
+    }}
+  </Story>
+</Canvas>