Remove nested ScrollView in the PrivateSpaceEducation screen.

It is not needed for the scrolling to work -- GlifLayout from the SUW
library generates a ScrollView. Additional ScrollView causes Talkback to
interpret it as a separate item and read out contents of its all
children. See b/379256039.

Video before: https://drive.google.com/file/d/1H3k5AxlIdgAA15PHUQ9tVq5AechziA4Z/view?usp=sharing&resourcekey=0-ozeQsQSS5jubL0pZ5FAZHA
Video after: https://drive.google.com/file/d/1W_ODgQhbQS6Xf_57A4TcYeJbqMJxYF16/view?usp=sharing&resourcekey=0-zQ0_p3qlIxpBL3rB5SC4vQ

In the before video, talkback is highlighting and (the green frames) and reading out the whole "How it works" section in addition to each element separately. After the fix it only highlights elements separately.

Bug: 379256039
Test: Manually testesd and presubmit.
Flag: EXEMPT bugfix
Change-Id: Idf6772b8772c54c1907d3a12617fcd03451af41b
This commit is contained in:
Kasia Krejszeff
2025-01-07 01:44:12 +00:00
parent 891ea3d18a
commit e0036fc8b2

View File

@@ -24,10 +24,6 @@
app:sucHeaderText="@string/private_space_setup_title"
app:sudDescriptionText="@string/private_space_hide_apps_summary"
android:icon="@drawable/ic_private_space_icon">
<ScrollView
android:id="@+id/main_clear_scrollview"
android:layout_width="match_parent"
android:layout_height="match_parent">
<LinearLayout
android:id="@+id/sud_layout_icon_container"
style="@style/SudContentFrame"
@@ -124,5 +120,4 @@
android:text="@string/private_space_learn_more_text"/>
</RelativeLayout>
</LinearLayout>
</ScrollView>
</com.google.android.setupdesign.GlifLayout>