Merge branch 'main' into fix-flakey-macro-tests

ONSdigital · Nov 6, 2024 · aea36f3 · aea36f3
2 parents c1c96f1 + 2bdd1f1
commit aea36f3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/src/components/summary/_macro-options.md b/src/components/summary/_macro-options.md
@@ -53,12 +53,12 @@
 
 ## SummaryAction
 
-| Name               | Type   | Required | Description                                                                                 |
-| ------------------ | ------ | -------- | ------------------------------------------------------------------------------------------- |
-| text               | string | true     | Text for the action link                                                                    |
-| url                | string | true     | The URL for the HTML `href` attribute of the link used to change the value of the row item  |
-| attributes         | object | false    | HTML attributes (for example, data attributes) to add to the action link                    |
-| visuallyHiddenText | string | false    | Visually hidden text in a span under the action link to add more context for screen readers |
+| Name               | Type   | Required | Description                                                                                                                      |
+| ------------------ | ------ | -------- | -------------------------------------------------------------------------------------------------------------------------------- |
+| text               | string | true     | Text for the action link                                                                                                         |
+| url                | string | true     | The URL for the HTML `href` attribute of the link used to change the value of the row item                                       |
+| attributes         | object | false    | HTML attributes (for example, data attributes) to add to the action link                                                         |
+| visuallyHiddenText | string | false    | Visually hidden text in a span under the action link to add more context for screen readers, , defaults the text in `text` param |
 
 ## SummaryLink
 

diff --git a/src/components/summary/_macro.njk b/src/components/summary/_macro.njk
@@ -112,10 +112,10 @@
                                                         class="ons-summary__button"
                                                         {% if action.attributes %}{% for attribute, value in (action.attributes.items() if action.attributes is mapping and action.attributes.items else action.attributes) %}{{ ' ' }}{{ attribute }}="{{ value }}"{% endfor %}{% endif %}
                                                     >
-                                                        <span class="ons-summary__button-text" aria-hidden="true">
-                                                            {{- action.text -}}
-                                                        </span>
-                                                        <span class="ons-u-vh">{{ action.visuallyHiddenText }}</span>
+                                                        <span class="ons-summary__button-text" aria-hidden="true">{{- action.text -}}</span>
+                                                        <span class="ons-u-vh"
+                                                            >{{ action.visuallyHiddenText | default (action.text) }}</span
+                                                        >
                                                     </a>
                                                 {% endfor %}
                                             </dd>