Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d8fb5cb8 authored by Ioana Alexandru's avatar Ioana Alexandru
Browse files

Ensure informative text in guts is read out by TalkBack

Make the non-clickable TextViews in our various notification guts as
importantForAccessibility to ensure that TalkBack reads them out when
appropriate.

Fix: 419739937
Test: manual, using TalkBack and opening the different types of guts
Flag: android.app.notifications_redesign_templates
Flag: com.android.systemui.permission_helper_ui_rich_ongoing
Change-Id: Ibcef0a8a4b7eacde1bf86281bd13b0c742afea4e
parent 357db9df
Loading
Loading
Loading
Loading
+2 −0
Original line number Diff line number Diff line
@@ -198,6 +198,7 @@
            android:visibility="gone"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:importantForAccessibility="yes"
            style="@*android:style/TextAppearance.DeviceDefault.Notification" />

        <!-- Non configurable multichannel text. appears instead of @+id/interruptiveness_settings-->
@@ -207,6 +208,7 @@
            android:visibility="gone"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:importantForAccessibility="yes"
            style="@*android:style/TextAppearance.DeviceDefault.Notification" />

        <LinearLayout
+3 −0
Original line number Diff line number Diff line
@@ -138,6 +138,7 @@
            android:visibility="gone"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:importantForAccessibility="yes"
            style="@*android:style/TextAppearance.DeviceDefault.Notification" />

        <!-- Non configurable app/channel text. appears instead of @+id/interruptiveness_settings-->
@@ -147,6 +148,7 @@
            android:visibility="gone"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:importantForAccessibility="yes"
            style="@*android:style/TextAppearance.DeviceDefault.Notification" />

        <!-- Non configurable multichannel text. appears instead of @+id/interruptiveness_settings-->
@@ -156,6 +158,7 @@
            android:visibility="gone"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:importantForAccessibility="yes"
            style="@*android:style/TextAppearance.DeviceDefault.Notification" />

        <LinearLayout
+1 −0
Original line number Diff line number Diff line
@@ -119,6 +119,7 @@
                    android:layout_width="match_parent"
                    android:layout_height="wrap_content"
                    android:textDirection="locale"
                    android:importantForAccessibility="yes"
                    style="@style/TextAppearance.NotificationImportanceChannelGroup" />
            </LinearLayout>
        </com.android.systemui.statusbar.notification.row.ButtonLinearLayout>
+2 −0
Original line number Diff line number Diff line
@@ -327,6 +327,7 @@ asked for it -->
                android:layout_height="wrap_content"
                android:layout_alignParentStart="true"
                android:gravity="start|center_vertical"
                android:importantForAccessibility="yes"
                style="@style/TextAppearance.NotificationImportanceButton"/>
            <TextView
                android:id="@+id/promoted_explain"
@@ -336,6 +337,7 @@ asked for it -->
                android:layout_below="@id/promoted_explain_title"
                android:gravity="start|center_vertical"
                android:maxWidth="200dp"
                android:importantForAccessibility="yes"
                style="@style/TextAppearance.NotificationImportanceDetail"/>
            <TextView
                android:id="@+id/promoted_demote"