Merge "Fix no response after Talkback users double-tap on Learn more in Accessibility pages" into tm-dev
This commit is contained in:
@@ -25,9 +25,7 @@ import androidx.preference.PreferenceViewHolder;
|
||||
|
||||
import com.android.settingslib.widget.FooterPreference;
|
||||
|
||||
/**
|
||||
* A custom preference acting as footer of a page. Disables the movement method by default.
|
||||
*/
|
||||
/** A custom preference acting as footer of a page. Disables the movement method by default. */
|
||||
public final class AccessibilityFooterPreference extends FooterPreference {
|
||||
|
||||
private boolean mLinkEnabled;
|
||||
@@ -46,12 +44,16 @@ public final class AccessibilityFooterPreference extends FooterPreference {
|
||||
|
||||
final TextView title = holder.itemView.findViewById(android.R.id.title);
|
||||
if (mLinkEnabled) {
|
||||
// When a TextView has a movement method, it will set the view to clickable. This makes
|
||||
// View.onTouchEvent always return true and consumes the touch event, essentially
|
||||
// nullifying any return values of MovementMethod.onTouchEvent.
|
||||
// When a TextView has a movement method, it will set the view to focusable and
|
||||
// clickable. This makes View.onTouchEvent always return true and consumes the touch
|
||||
// event, essentially nullifying any return values of MovementMethod.onTouchEvent.
|
||||
// To still allow propagating touch events to the parent when this view doesn't have
|
||||
// links, we only set the movement method here if the text contains links.
|
||||
title.setMovementMethod(LinkMovementMethod.getInstance());
|
||||
|
||||
// Groups of related title and link content by making the container focusable,
|
||||
// then make all the children inside not focusable.
|
||||
title.setFocusable(false);
|
||||
} else {
|
||||
title.setMovementMethod(/* movement= */ null);
|
||||
}
|
||||
|
@@ -122,5 +122,8 @@ public class AccessibilityFooterPreferenceController extends BasePreferenceContr
|
||||
} else {
|
||||
footerPreference.setLinkEnabled(false);
|
||||
}
|
||||
|
||||
// Grouping subcomponents to make more accessible.
|
||||
footerPreference.setSelectable(false);
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user