Skip to content

Commit cd4e057

Browse files
committed
Make the group name field show up as required to screenreaders
Fixes CNVS-20968 Test plan: - When viewing a group set, click "+ Group" - With the screenreader turned on, focus the name field - Ensure that your screenreader notes that the field is required Change-Id: Ic9892a634515bd40bf26b26bfd35e6586fc0221c Reviewed-on: https://gerrit.instructure.com/56670 Tested-by: Jenkins Reviewed-by: Matthew Wheeler <mwheeler@instructure.com> QA-Review: Steven Shepherd <sshepherd@instructure.com> Product-Review: Alex Boyd <aboyd@instructure.com>
1 parent b019dc0 commit cd4e057

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

app/views/jst/groups/manage/groupEdit.handlebars

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<div class="control-group">
22
<label class="control-label" for="group_name">{{#t "group_name"}}Group Name{{/t}}</label>
33
<div class="controls">
4-
<input type="text" id="group_name" name='name' value="{{name}}"
4+
<input type="text" id="group_name" name='name' value="{{name}}" aria-required="true"
55
placeholder="{{#t "group_name"}}Group Name{{/t}}">
66
</div>
77
</div>

0 commit comments

Comments
 (0)