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

Commit bc35e4b9 authored by Evan Severson's avatar Evan Severson
Browse files

[Task manager] Set the content description for the number button 

This helps accessibility users understand the interface better.

Test: Manual
Fixes: 319543761
Flag: EXEMPT bugfix
Change-Id: I6ecfd29df04cb7119d3a7520ba6bbae67f014889
parent c2f77d14

packages/SystemUI/compose/features/src/com/android/systemui/qs/footer/ui/compose/FooterActions.kt

+6 −1
Original line number Diff line number Diff line
@@ -253,6 +253,7 @@ private fun RowScope.ForegroundServicesButton( 
    } else {

        NumberButton(

            model.foregroundServicesCount,

            contentDescription = model.text,

            showNewDot = model.hasNewChanges,

            onClick = model.onClick,

        )
@@ -281,6 +282,7 @@ fun IconButton(model: FooterActionsButtonViewModel, modifier: Modifier = Modifie 
@Composable

private fun NumberButton(

    number: Int,

    contentDescription: String,

    showNewDot: Boolean,

    onClick: (Expandable) -> Unit,

    modifier: Modifier = Modifier,
@@ -311,7 +313,10 @@ private fun NumberButton( 
            ) {

                Text(

                    number.toString(),

                    modifier = Modifier.align(Alignment.Center),

                    modifier =

                        Modifier.align(Alignment.Center).semantics {

                            this.contentDescription = contentDescription

                        },

                    style = MaterialTheme.typography.bodyLarge,

                    color = colorAttr(R.attr.onShadeInactiveVariant),

                    // TODO(b/242040009): This should only use a standard text style instead and