Chromium Code Reviews| Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js |
| diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js |
| index c556d59aed984ef9eb3677a760d940927a28bbf3..91914da7163c82813dce25d489aaa8f913d3f6ae 100644 |
| --- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js |
| +++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js |
| @@ -230,8 +230,7 @@ Output.ROLE_INFO_ = { |
| msgId: 'role_menubar', |
| }, |
| menuItem: { |
| - msgId: 'role_menuitem', |
| - earconId: 'BUTTON' |
|
dmazzoni
2016/06/14 21:46:32
Did you mean to get rid of the earcon for menuItem
David Tseng
2016/06/15 22:59:20
I did; not sure how you feel about it, but output
|
| + msgId: 'role_menuitem' |
| }, |
| menuItemCheckBox: { |
| msgId: 'role_menuitemcheckbox', |
| @@ -471,6 +470,10 @@ Output.RULES = { |
| '@describe_index($indexInParent, $parentChildCount) ' + |
| '$description' |
| }, |
| + menuItemCheckBox: { |
| + speak: '$if($checked, $earcon(CHECK_ON), $earcon(CHECK_OFF)) ' + |
| + '$name $role $checked $description' |
| + }, |
| menuListOption: { |
| speak: '$name @role_menuitem ' + |
| '@describe_index($indexInParent, $parentChildCount) $description' |