diff options
author | Andrew Lee <anwlee@google.com> | 2015-01-13 13:19:50 -0800 |
---|---|---|
committer | Andrew Lee <anwlee@google.com> | 2015-01-13 22:31:42 -0800 |
commit | 87f98d75fa021f00911fad13773d8972e7ebeb55 (patch) | |
tree | fda92f48e4b799945505c9e178b5fd8f7bb9a97d /InCallUI/res/drawable | |
parent | e736798d5d8bd9de0df3df5e91a4b652e120efa3 (diff) |
Improve call button accessibility.
+ Convert Audio Button into a ToggleButton, and update its content
description depending on whether its acting as a toggle for
speakerphone, or if it's acting as a menu button to select from
more than two audio output modes.
+ Set custom style on ToggleButtons with null background, to override
the default ToggleButton styling which was causing shadows.
- Delete CallToggleButton, which did some hacky things with the
content description and accessibility.
- Delete helper method which abused TYPE_ANNOUNCEMENTS. Rely on
default TalkBack behavior for ToggleButtons to read on click.
- Delete unused accessibility strings. The states these were formerly
read out for will now be handled by a combination of button labels
and default TalkBack behavior.
The UI (without TalkBack) behaves the same after these changes.
The behavior with TalkBack after these changes is that the audio
(when toggling for speakerphone), mute, and hold buttons will
generally read "speaker/mute/hold button (not) checked" when
explored by touch depending on their state. They will read
"speaker/mute/hold" when clicked.
If more than two audio modes are supported, the audio button will
read "<mode> button checked", where <mode> can be speaker, bluetooth,
handset, or earpiece. When clicked, it will open the audio pop up
menu and read that there are # options displayed on the screen. This
is combines the ToggleButton and PopupMenu Talkback behavior, since
this is a custom widget.
Bug: 18783204
Change-Id: I27363c697bb018b5f87ad2e27678f8db6d93b2e7
Diffstat (limited to 'InCallUI/res/drawable')
-rw-r--r-- | InCallUI/res/drawable/btn_compound_audio.xml | 9 | ||||
-rw-r--r-- | InCallUI/res/drawable/btn_compound_background.xml | 2 |
2 files changed, 1 insertions, 10 deletions
diff --git a/InCallUI/res/drawable/btn_compound_audio.xml b/InCallUI/res/drawable/btn_compound_audio.xml index 222ff1b41..25a64a6ab 100644 --- a/InCallUI/res/drawable/btn_compound_audio.xml +++ b/InCallUI/res/drawable/btn_compound_audio.xml @@ -90,13 +90,4 @@ android:tint="@color/selectable_icon_tint" /> </item> - <!-- Generic "audio mode" icon. --> - <!-- TODO: is this actually needed? --> - <!-- - <item android:id="@+id/soundItem"> - <bitmap android:src="@drawable/ic_sound_holo_dark" - android:gravity="center" /> - </item> - --> - </layer-list> diff --git a/InCallUI/res/drawable/btn_compound_background.xml b/InCallUI/res/drawable/btn_compound_background.xml index 36aab0df7..20e2a3056 100644 --- a/InCallUI/res/drawable/btn_compound_background.xml +++ b/InCallUI/res/drawable/btn_compound_background.xml @@ -32,4 +32,4 @@ <item android:drawable="@drawable/btn_unselected" /> </selector> </item> -</ripple>
\ No newline at end of file +</ripple> |