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

Commit feb01cda authored by Michael Mikhail's avatar Michael Mikhail
Browse files

[a11y] update recommendation content discription for new layout 

Content description for card should be the header and the media cover
items have the same content description of the old version.

Bug: 267295529
Test: Checked talkback and clicked on all ui components. Video
is in bug link.

Change-Id: If8d780220e01d10327f5f58ea6246e28ac8fb51a
parent f7b51341

packages/SystemUI/res/layout/media_recommendation_view.xml

+4 −2
Original line number Diff line number Diff line
@@ -47,7 +47,8 @@ 
        android:singleLine="true"

        android:textSize="12sp"

        android:gravity="top"

        android:layout_gravity="bottom"/>

        android:layout_gravity="bottom"

        android:importantForAccessibility="no"/>



    <!-- Album name -->

    <TextView
@@ -61,7 +62,8 @@ 
        android:singleLine="true"

        android:textSize="11sp"

        android:gravity="center_vertical"

        android:layout_gravity="bottom"/>

        android:layout_gravity="bottom"

        android:importantForAccessibility="no"/>



    <!-- Seek Bar -->

    <SeekBar

packages/SystemUI/src/com/android/systemui/media/controls/ui/MediaControlPanel.java

+8 −3
Original line number Diff line number Diff line
@@ -734,9 +734,14 @@ public class MediaControlPanel { 
            contentDescription =

                    mRecommendationViewHolder.getGutsViewHolder().getGutsText().getText();

        } else if (data != null) {

            if (mFeatureFlags.isEnabled(Flags.MEDIA_RECOMMENDATION_CARD_UPDATE)) {

                contentDescription = mContext.getString(

                        R.string.controls_media_smartspace_rec_header);

            } else {

                contentDescription = mContext.getString(

                        R.string.controls_media_smartspace_rec_description,

                        data.getAppName(mContext));

            }

        } else {

            contentDescription = null;

        }