Skip to content

Commit

Permalink
COMMAND PALLET: Shortcut keys are not read by screenreader (fixes #3839)
Browse files Browse the repository at this point in the history
  • Loading branch information
bpasero committed Mar 8, 2016
1 parent 4d9bab0 commit f1948d6
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/vs/workbench/parts/quickopen/browser/commandsHandler.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,10 @@ class BaseCommandEntry extends QuickOpenEntryGroup {
}

public getAriaLabel(): string {
if (this.key) {
return nls.localize('entryAriaLabelWithKey', "{0}, shortcut {1}, commands", this.getLabel(), this.key);
}

return nls.localize('entryAriaLabel', "{0}, commands", this.getLabel());
}

Expand Down

2 comments on commit f1948d6

@zersiax
Copy link

@zersiax zersiax commented on f1948d6 Mar 8, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, will test later :)

@bpasero
Copy link
Member Author

@bpasero bpasero commented on f1948d6 Mar 8, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Please sign in to comment.