Skip to content

Commit 4fc9e55

Browse files
authored
Merge pull request #2256 from canonical/WD-31831-fix-a-11-y-tests-for-form-gen-on-c-com
Fix a 11 y tests for the form generator template.
2 parents 2a51f35 + 588cbff commit 4fc9e55

File tree

3 files changed

+10
-10
lines changed

3 files changed

+10
-10
lines changed

.pa11yci

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,6 @@
77
"--no-sandbox"
88
]
99
},
10-
"ignore": [
11-
"WCAG2AA.Principle1.Guideline1_3.1_3_1.H71.NoLegend",
12-
"WCAG2AA.Principle1.Guideline1_3.1_3_1.H44.NotFormControl",
13-
"WCAG2AA.Principle4.Guideline4_1.4_1_2.H91.Textarea.Name",
14-
"WCAG2AA.Principle4.Guideline4_1.4_1_2.H91.InputText.Name",
15-
"WCAG2AA.Principle1.Guideline1_1.1_1_1.H65",
16-
"WCAG2AA.Principle2.Guideline2_4.2_4_1.H64.1.NoTitle"
17-
],
1810
"hideElements": "iframe[style*='display: none'], [style*='visibility: hidden;']"
1911
},
2012
"urls": [

static/sass/styles.scss

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -743,6 +743,11 @@ $p-small-lh-diff: map-get($line-heights, default-text) -
743743
}
744744
}
745745

746+
.p-heading--4.legend-is-required::before {
747+
content: " * ";
748+
position: relative;
749+
}
750+
746751
.u-extra-space {
747752
margin-bottom: 4rem;
748753
}

templates/shared/forms/form-fields.html

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,13 @@
2222
<hr class="p-rule is-fixed-width" />
2323
<fieldset class="p-fieldset-section{% if fieldset.inputType == 'checkbox-visibility' %} js-toggle-checkbox-visibility{% elif fieldset.isRequired and fieldset.inputType == 'checkbox' %} js-required-checkbox{% elif fieldset.inputType == 'radio' %} js-remove-radio-names{% endif %} {% if fieldset.inputType == 'checkbox' or fieldset.inputType == 'checkbox-visibility' %}js-remove-checkbox-names{% endif %}"
2424
aria-labelledby="{{ fieldset_id }}-legend">
25+
<legend class="u-off-screen">
26+
{{ fieldset.title }}{% if fieldset.isRequired %} (required) {% endif %}
27+
</legend>
2528
<div class="row--50-50 {% if fieldset.noCommentsFromLead != True %}js-formfield{% endif %}">
2629
<div class="col">
27-
<legend class="p-heading--4 js-formfield-title {% if fieldset.isRequired %}is-required{% endif %}"
28-
id="{{ fieldset_id }}-legend">{{ fieldset.title }}</legend>
30+
<p class="p-heading--4 js-formfield-title {% if fieldset.isRequired %}legend-is-required{% endif %}"
31+
id="{{ fieldset_id }}-legend">{{ fieldset.title }}</p>
2932
</div>
3033
<div class="col">
3134
<ul class="p-list">

0 commit comments

Comments
 (0)