diff options
| author | Andrew Lee <anwlee@google.com> | 2015-10-28 12:08:40 -0700 |
|---|---|---|
| committer | Andrew Lee <anwlee@google.com> | 2015-10-28 12:24:05 -0700 |
| commit | 71b73c57c458269f952de9448c810345c965757b (patch) | |
| tree | 9a1b471ed7274c1d647f0be6dc2d0fe19e209b99 | |
| parent | dc07e74a17aceb566b5a78a8b553ac437102b91a (diff) | |
Tweak accessibility in for call blocking settings.
+ Adjust talkback focus to allow the user to page through the cards
for the settings and import separately, even though they are
structurally part of the same header component, visually they
reside in different "cards".
+ Change the "hide setting" to be toggled by tapping on the text
description area.
Bug: 25163592
Change-Id: I0d1fcd28dba058d707bfe5099cdf912aed14f728
| -rw-r--r-- | res/layout/blocked_number_header.xml | 13 | ||||
| -rw-r--r-- | src/com/android/dialer/filterednumber/BlockedNumbersFragment.java | 7 |
2 files changed, 16 insertions, 4 deletions
diff --git a/res/layout/blocked_number_header.xml b/res/layout/blocked_number_header.xml index ce1d4feab..29a05bd26 100644 --- a/res/layout/blocked_number_header.xml +++ b/res/layout/blocked_number_header.xml @@ -17,7 +17,8 @@ xmlns:card_view="http://schemas.android.com/apk/res-auto" android:layout_width="match_parent" android:layout_height="wrap_content" - android:orientation="vertical"> + android:orientation="vertical" + android:focusable="false"> <LinearLayout android:id="@+id/blocked_numbers_disabled_for_emergency" android:layout_width="match_parent" @@ -25,6 +26,7 @@ android:orientation="vertical" android:layout_marginBottom="8dp" android:padding="16dp" + android:focusable="true" android:visibility="gone"> <TextView @@ -58,7 +60,8 @@ android:gravity="center_vertical" android:padding="16dp" android:paddingEnd="8dp" - android:background="@android:color/white"> + android:background="@android:color/white" + android:focusable="true"> <TextView android:layout_width="0dp" @@ -70,7 +73,8 @@ <Switch android:id="@+id/hide_blocked_calls_switch" android:layout_width="wrap_content" android:layout_height="wrap_content" - android:layout_marginLeft="40dp" /> + android:layout_marginLeft="40dp" + android:focusable="false" /> </LinearLayout> @@ -85,7 +89,8 @@ android:layout_width="match_parent" android:layout_height="wrap_content" android:orientation="vertical" - android:background="@android:color/white"> + android:background="@android:color/white" + android:focusable="true"> <TextView android:id="@+id/textView" android:layout_width="wrap_content" diff --git a/src/com/android/dialer/filterednumber/BlockedNumbersFragment.java b/src/com/android/dialer/filterednumber/BlockedNumbersFragment.java index 7788cbe1b..4ec65b52d 100644 --- a/src/com/android/dialer/filterednumber/BlockedNumbersFragment.java +++ b/src/com/android/dialer/filterednumber/BlockedNumbersFragment.java @@ -79,6 +79,13 @@ public class BlockedNumbersFragment extends ListFragment FilteredNumbersUtil.setShouldHideBlockedCalls(getActivity(), isChecked); } }); + getActivity().findViewById(R.id.hide_blocked_calls_setting).setOnClickListener( + new View.OnClickListener() { + @Override + public void onClick(final View view) { + mHideSettingSwitch.toggle(); + } + }); mImportSettings = getActivity().findViewById(R.id.import_settings); mBlockedNumbersDisabledForEmergency = |