netlify[bot]
commented
3 months ago Deploy Preview for ons-design-system-preview ready!
| Name | Link |
|---|---|
| Latest commit | 52e740120e975ed2539c2eec87497e178f60ff5e |
| Latest deploy log | https://app.netlify.com/sites/ons-design-system-preview/deploys/669624552b1cfc0008f3b0ec |
| Deploy Preview | https://deploy-preview-3256--ons-design-system-preview.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
What is the context of this PR?
This fixes: #3028
I have created a new variant called summary for the list component. EQ currently uses the summary component to display a list of companies in their surveys. The summary component is not designed to display a simple list of string.
I have added a variant to the list component to align with the style of the summary component. The list component is the correct component to use when listing a list of names or strings.
No unit testing needed to be added as the current tests to cover for single or multiple variants additions.
How to review this PR
Pull this pr and confirm that the summary variant in the lists example displays and looks identical in style to the simple summary component
Checklist
This needs to be completed by the person raising the PR.