Skip to content

Commit 4985b46

Browse files
musalevogtn
andauthored
Add ac11y support for the people-picker to narrate selected people (#1439)
* Add ac11y support for the p-picker to narrate selected people * Update the aria-orientation to vertical Co-authored-by: Nicolas Vogt <[email protected]>
1 parent ebfca02 commit 4985b46

File tree

2 files changed

+20
-5
lines changed

2 files changed

+20
-5
lines changed

packages/mgt-components/src/components/mgt-people-picker/mgt-people-picker.scss

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,10 @@ mgt-people-picker .root {
5353
font-weight: normal;
5454
overflow: hidden;
5555

56+
&__options {
57+
display: contents;
58+
}
59+
5660
&__person-wrapper {
5761
display: flex;
5862
margin: 0 5px 6px 0;

packages/mgt-components/src/components/mgt-people-picker/mgt-people-picker.ts

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -620,10 +620,19 @@ export class MgtPeoplePicker extends MgtTemplatedComponent {
620620
return null;
621621
}
622622
return html`
623-
${selectedPeople.slice(0, selectedPeople.length).map(
624-
person =>
625-
html`
626-
<div class="selected-list__person-wrapper">
623+
<div
624+
tabindex="0"
625+
aria-label="selected-people"
626+
aria-orientation="vertical"
627+
role="listbox"
628+
class="selected-list__options">${selectedPeople.slice(0, selectedPeople.length).map(
629+
person =>
630+
html`
631+
<div
632+
role="option"
633+
tabindex="0"
634+
aria-label=${person.displayName}
635+
class="selected-list__person-wrapper">
627636
${
628637
this.renderTemplate(
629638
'selected-person',
@@ -635,6 +644,8 @@ export class MgtPeoplePicker extends MgtTemplatedComponent {
635644
<div class="selected-list__person-wrapper__overflow">
636645
<div class="selected-list__person-wrapper__overflow__gradient"></div>
637646
<div
647+
tabindex="0"
648+
aria-label="close-icon"
638649
class="selected-list__person-wrapper__overflow__close-icon"
639650
@click="${e => this.removePerson(person, e)}"
640651
>
@@ -643,7 +654,7 @@ export class MgtPeoplePicker extends MgtTemplatedComponent {
643654
</div>
644655
</div>
645656
`
646-
)}
657+
)}</div>
647658
`;
648659
}
649660
/**

0 commit comments

Comments
 (0)