Summary: | AX: "video element controller" is an overly verbose default description for the playback controls; how about just "playback" | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | James Craig <jcraig> | ||||||||||
Component: | Accessibility | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | aboxhall, apinheiro, cfleizach, dmazzoni, eric.carlson, feature-media-reviews, jcraig, jdiggs, jer.noble, webkit-bug-importer, webkit.review.bot | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Attachments: |
|
Description
James Craig
2013-03-28 17:41:58 PDT
Took a look at the source. There is an "audio element controller" role description, too. I plan to change these to "video playback" and "audio playback" to decrease the default verbosity in assistive technologies like VoiceOver. This will trigger some Loc string updates for about 40 languages. Created attachment 195826 [details]
patch
Created attachment 195827 [details]
patch
Created attachment 195830 [details]
patch. previously missed a changelog.
Comment on attachment 195830 [details] patch. previously missed a changelog. View in context: https://bugs.webkit.org/attachment.cgi?id=195830&action=review > Source/WebCore/ChangeLog:8 > + No new tests (OOPS!). Please remove the no new tests line, because you're updating existing tests Created attachment 195990 [details]
patch. removed references to "no new tests" because I updated existing test coverage.
Comment on attachment 195990 [details] patch. removed references to "no new tests" because I updated existing test coverage. Clearing flags on attachment: 195990 Committed r147343: <http://trac.webkit.org/changeset/147343> All reviewed patches have been landed. Closing bug. |