diff options
author | Tyler Gunn <tgunn@google.com> | 2016-05-10 08:39:54 -0700 |
---|---|---|
committer | Tyler Gunn <tgunn@google.com> | 2016-05-10 08:39:54 -0700 |
commit | ebef6b94037a2a028635a5371f3d8e0f97c7d206 (patch) | |
tree | fe55e8c6ae1b5065f9bb06d5a97fca51f225743f /InCallUI/res | |
parent | 8b25d786674f95034dfcdfff09ea16f8e3869311 (diff) |
Cleanup of video pause button.
The "pause video" button had a few issues that needed to be cleaned up:
1. The accessibility text for the button was always "pause video".
I've changed this so that it says, "Turn on camera" or "Turn off camera"
to better indicate what pushing the button will do.
2. When the user accepts an incoming request to initiate a 1-way video
call, the button state would not be correct (the user's camera is
disabled, but the button would not reflect that).
Bug: 27944419
Bug: 28550284
Change-Id: I2594971ec21b5714ca2d750894a93b758f7a80e5
Diffstat (limited to 'InCallUI/res')
-rw-r--r-- | InCallUI/res/layout/call_button_fragment.xml | 2 | ||||
-rw-r--r-- | InCallUI/res/values/strings.xml | 6 |
2 files changed, 5 insertions, 3 deletions
diff --git a/InCallUI/res/layout/call_button_fragment.xml b/InCallUI/res/layout/call_button_fragment.xml index 6dbfbf73a..802e3de62 100644 --- a/InCallUI/res/layout/call_button_fragment.xml +++ b/InCallUI/res/layout/call_button_fragment.xml @@ -126,7 +126,7 @@ <ToggleButton android:id="@+id/pauseVideoButton" style="@style/InCallCompoundButton" android:background="@drawable/btn_compound_video_off" - android:contentDescription="@string/onscreenPauseVideoText" + android:contentDescription="@string/onscreenTurnOffCameraText" android:visibility="gone" /> <!-- "Change to audio call" for video calls. --> diff --git a/InCallUI/res/values/strings.xml b/InCallUI/res/values/strings.xml index 57a1b5389..7701cdb2d 100644 --- a/InCallUI/res/values/strings.xml +++ b/InCallUI/res/values/strings.xml @@ -313,8 +313,10 @@ <!-- Text for the onscreen "Switch camera" button. When clicked, this switches the user's camera for video calling between the front-facing camera and the back-facing camera. --> <string name="onscreenSwitchCameraText">Switch camera</string> - <!-- Text for the onscreen "Pause video" button. --> - <string name="onscreenPauseVideoText">Pause video</string> + <!-- Text for the onscreen "turn on camera" button. --> + <string name="onscreenTurnOnCameraText">Turn on camera</string> + <!-- Text for the onscreen "turn off camera" button. --> + <string name="onscreenTurnOffCameraText">Turn off camera</string> <!-- Text for the onscreen overflow button, to see additional actions which can be done. --> <string name="onscreenOverflowText">More options</string> |