Skip to content

Commit 8ee5ae5

Browse files
committed
VPR-104 fix(a11y): move heading outside q-list to fix aria-required-children
The h3 "Domains" heading was a direct child of q-list (role="list"), violating WCAG 1.3.1 — list elements must only contain listitem children. Move heading above q-list so the role hierarchy is valid.
1 parent cd25404 commit 8ee5ae5

1 file changed

Lines changed: 24 additions & 26 deletions

File tree

VueApp/src/CTS/pages/ManageDomains.vue

Lines changed: 24 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -127,32 +127,30 @@ getDomains()
127127
</q-form>
128128

129129
<div class="row">
130-
<q-list class="col col-12 col-md-10 col-lg-6">
131-
<q-item-label
132-
header
133-
class="text-primary"
134-
><h3>Domains</h3></q-item-label
135-
>
136-
<q-item
137-
v-for="d in domains"
138-
:key="d.domainId"
139-
>
140-
<q-item-section
141-
top
142-
side
130+
<div class="col col-12 col-md-10 col-lg-6">
131+
<h3 class="text-primary q-px-md">Domains</h3>
132+
<q-list>
133+
<q-item
134+
v-for="d in domains"
135+
:key="d.domainId"
143136
>
144-
<q-btn
145-
icon="edit"
146-
dense
147-
flat
148-
class="secondary"
149-
:aria-label="`Edit domain: ${d.name}`"
150-
@click="domain = d"
151-
></q-btn>
152-
</q-item-section>
153-
<q-item-section top>{{ d.order }}. {{ d.name }}</q-item-section>
154-
<q-item-section>{{ d.description }}</q-item-section>
155-
</q-item>
156-
</q-list>
137+
<q-item-section
138+
top
139+
side
140+
>
141+
<q-btn
142+
icon="edit"
143+
dense
144+
flat
145+
class="secondary"
146+
:aria-label="`Edit domain: ${d.name}`"
147+
@click="domain = d"
148+
></q-btn>
149+
</q-item-section>
150+
<q-item-section top>{{ d.order }}. {{ d.name }}</q-item-section>
151+
<q-item-section>{{ d.description }}</q-item-section>
152+
</q-item>
153+
</q-list>
154+
</div>
157155
</div>
158156
</template>

0 commit comments

Comments
 (0)