Skip to content

Commit d6ab960

Browse files
Samiya CaurDevtools-frontend LUCI CQ
authored andcommitted
Add aria-activedescendant to listbox in select folder dialog
Bug: 406073540 Change-Id: I8feb7739d06752fb75235f29a6dc70a1a144677c Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6450199 Reviewed-by: Ergün Erdoğmuş <[email protected]> Auto-Submit: Samiya Caur <[email protected]> Commit-Queue: Ergün Erdoğmuş <[email protected]>
1 parent c18fac6 commit d6ab960

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

front_end/panels/ai_assistance/SelectWorkspaceDialog.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,10 +120,13 @@ export class SelectWorkspaceDialog extends UI.Widget.VBox {
120120
` : nothing}
121121
</div>
122122
${hasFolders ? html`
123-
<ul role="listbox" aria-label=${lockedString(UIStringsNotTranslate.selectFolder)} tabindex="0">
123+
<ul role="listbox" aria-label=${lockedString(UIStringsNotTranslate.selectFolder)} tabindex="0"
124+
aria-activedescendant=${input.folders.length > 0 ? `option-${input.selectedIndex}` : ''}>
124125
${input.folders.map((folder, index) => {
126+
const optionId = `option-${index}`;
125127
return html`
126128
<li
129+
id=${optionId}
127130
@click=${() => input.onProjectSelected(index)}
128131
class=${index === input.selectedIndex ? 'selected' : ''}
129132
aria-selected=${index === input.selectedIndex ? 'true' : 'false'}

0 commit comments

Comments
 (0)