New issue
Advanced search Search tips

Issue 827418 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 816355



Sign in to add a comment

CCA: Change record button label to differentiate "start recording" and "stop recording"

Project Member Reported by shenghao@chromium.org, Mar 30 2018

Issue description

This is a release blocking issue asked from the accessibility reviewer.

Currently, the recording shutter button label is always "Record video". In accessibility mode, it reads "Record video, button, press search plus space to activate" for both start recording and stop recording cases.

We should change the label to "Start recording" and "Stop recording", respectively.

Yuli, can you prioritize on this? After this is fixed, please request the translation of strings.

 
Blocking: 816355
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 3 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/apps/camera/+/c7b184cb6552fdcfe8fd10693884d7269edeb027

commit c7b184cb6552fdcfe8fd10693884d7269edeb027
Author: Yuli Huang <yuli@chromium.org>
Date: Tue Apr 03 01:22:28 2018

Fix video recording's app/tooltip/error messages.

BUG= chromium:827418 
TEST=Tested manually on elm.

Change-Id: I16531b982c88b80650140ca339748c0b88c38033
Reviewed-on: https://chromium-review.googlesource.com/988152
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>
Tested-by: yuli <yuli@chromium.org>

[modify] https://crrev.com/c7b184cb6552fdcfe8fd10693884d7269edeb027/src/js/views/camera.js
[modify] https://crrev.com/c7b184cb6552fdcfe8fd10693884d7269edeb027/src/_locales/en/messages.json

Comment 3 by y...@chromium.org, Apr 3 2018

Status: Fixed (was: Assigned)
Please verify the fix after the canary is updated. Thanks.
I can see the labels "Start recording" and "Stop recording" as tooltip messages.
But, in accessibility Chromevox spoken feedback mode, it doesn't read out when clicked on 'Record video' button. It actually doesn't read out for any of the button press in camera app.
After turning on accessibility mode, you have to use TAB to switch focus to different buttons. When the capture button in recording mode is focused, the tooltip of it will be read out.
Status: Verified (was: Fixed)
Verified on M67 (10575.32.0, 67.0.3396.41)
Project Member

Comment 7 by bugdroid1@chromium.org, Jan 15

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c7b184cb6552fdcfe8fd10693884d7269edeb027

commit c7b184cb6552fdcfe8fd10693884d7269edeb027
Author: Yuli Huang <yuli@chromium.org>
Date: Tue Apr 03 01:22:28 2018

Fix video recording's app/tooltip/error messages.

BUG= chromium:827418 
TEST=Tested manually on elm.

Change-Id: I16531b982c88b80650140ca339748c0b88c38033
Reviewed-on: https://chromium-review.googlesource.com/988152
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>
Tested-by: yuli <yuli@chromium.org>

[modify] https://crrev.com/c7b184cb6552fdcfe8fd10693884d7269edeb027/src/_locales/en/messages.json
[modify] https://crrev.com/c7b184cb6552fdcfe8fd10693884d7269edeb027/src/js/views/camera.js

Sign in to add a comment