Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(38)

Issue 7648028: More distinguishable message is spoken when accessibility is enabled/disabled on login screen. (Closed)

Created:
9 years, 4 months ago by hashimoto
Modified:
9 years, 4 months ago
Reviewers:
dmazzoni
CC:
chromium-reviews, Zachary Kuznia
Visibility:
Public.

Description

More distinguishable message is spoken when accessibility is enabled/disabled on login screen. BUG=chromium-os:11950 TEST=Press Ctrl+Alt+Z on login screen and listen to what is spoken. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96902

Patch Set 1 #

Patch Set 2 : Fix along review comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M chrome/app/generated_resources.grd View 1 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
hashimoto
Could you review this? Since I am not a native English speaker, please suggest me ...
9 years, 4 months ago (2011-08-15 09:55:55 UTC) #1
dmazzoni
LGTM I like switching to "enabling" and "disabling", but since you're going to be working ...
9 years, 4 months ago (2011-08-16 04:52:47 UTC) #2
hashimoto
Thank you for your kind suggestion, now the message looks more distinguishable and meaningful. I ...
9 years, 4 months ago (2011-08-16 07:04:37 UTC) #3
dmazzoni
9 years, 4 months ago (2011-08-16 15:05:00 UTC) #4
On Tue, Aug 16, 2011 at 12:04 AM, <hashimoto@chromium.org> wrote:

> Thank you for your kind suggestion, now the message looks more
> distinguishable
> and meaningful.
> I will commit this new patch.
>
> About the word "accessibility", I agree with you that the word is too
> ambiguous
> and words like "spoken feedback" and "magnification" are better choices.
> Beside making a category in the settings, is there a plan to replace the
> word
> "accessibility" in the entire chrome source tree with other more precise
> words?
> Should I avoid using the ambiguous word "accessibility" in new codes?


I think we should still use the word Accessibility, because that's the right
general term used in software development to indicate special functionality
provided for people who might not use the standard user interface in any
way. However, we should try to be more specific when possible - instead of
EnableAccessibility, it'd be better to have a method like
EnableChromeVoxForAccessibility or EnableAccessibleSpokenFeedback.

- Dominic

Powered by Google App Engine
This is Rietveld 408576698