From a23c53c9a94025ab3f43007db5294b353f416e4b Mon Sep 17 00:00:00 2001 From: Chaohui Wang Date: Tue, 25 Jun 2024 13:26:12 +0800 Subject: [PATCH] Fix Talkback can focus on blank area By hiding radio button frame instead of just radio button. Fix: 347185801 Flag: EXEMPT bug fix Test: manual - on ApnSettings Change-Id: I7a2ba536f037a6cd014989bba1374d034169d90b --- src/com/android/settings/network/apn/ApnPreference.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/com/android/settings/network/apn/ApnPreference.java b/src/com/android/settings/network/apn/ApnPreference.java index 879fcb602f1..55258c139c6 100644 --- a/src/com/android/settings/network/apn/ApnPreference.java +++ b/src/com/android/settings/network/apn/ApnPreference.java @@ -85,10 +85,11 @@ public class ApnPreference extends Preference final RelativeLayout textArea = (RelativeLayout) view.findViewById(R.id.text_layout); textArea.setOnClickListener(this); + final View radioButtonFrame = view.itemView.requireViewById(R.id.apn_radio_button_frame); final RadioButton rb = view.itemView.requireViewById(R.id.apn_radiobutton); mRadioButton = rb; if (mDefaultSelectable) { - view.itemView.requireViewById(R.id.apn_radio_button_frame).setOnClickListener((v) -> { + radioButtonFrame.setOnClickListener((v) -> { rb.performClick(); }); rb.setOnCheckedChangeListener(this); @@ -96,9 +97,9 @@ public class ApnPreference extends Preference mProtectFromCheckedChange = true; rb.setChecked(mIsChecked); mProtectFromCheckedChange = false; - rb.setVisibility(View.VISIBLE); + radioButtonFrame.setVisibility(View.VISIBLE); } else { - rb.setVisibility(View.GONE); + radioButtonFrame.setVisibility(View.GONE); } }