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

Commit 65d2e8c5 authored by Kai Li's avatar Kai Li Committed by Android (Google) Code Review
Browse files

Merge "Fix infinite draw loop" into main

parents d162e52a b630daac
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -90,7 +90,7 @@ fun Chip(action: ActionViewModel, modifier: Modifier = Modifier) {
                )
                .padding(start = 12.dp, end = 16.dp, top = 4.dp, bottom = 4.dp),
    ) {
        val painter = rememberDrawablePainter(action.icon.drawable)
        val painter = rememberDrawablePainter(action.icon.large)
        Image(
            painter = painter,
            contentDescription = stringResource(id = R.string.ambient_cue_icon_content_description),
+3 −5
Original line number Diff line number Diff line
@@ -43,7 +43,6 @@ import androidx.compose.foundation.shape.CircleShape
import androidx.compose.foundation.shape.RoundedCornerShape
import androidx.compose.material3.FilledIconButton
import androidx.compose.material3.Icon
import androidx.compose.material3.IconButtonColors
import androidx.compose.material3.IconButtonDefaults
import androidx.compose.material3.LocalMinimumInteractiveComponentSize
import androidx.compose.material3.MaterialTheme
@@ -70,8 +69,8 @@ import androidx.compose.ui.graphics.graphicsLayer
import androidx.compose.ui.layout.onGloballyPositioned
import androidx.compose.ui.platform.LocalConfiguration
import androidx.compose.ui.platform.LocalDensity
import androidx.compose.ui.res.stringResource
import androidx.compose.ui.res.painterResource
import androidx.compose.ui.res.stringResource
import androidx.compose.ui.text.font.FontWeight
import androidx.compose.ui.text.style.TextOverflow
import androidx.compose.ui.unit.Dp
@@ -79,7 +78,6 @@ import androidx.compose.ui.unit.IntSize
import androidx.compose.ui.unit.dp
import androidx.compose.ui.util.fastForEachIndexed
import androidx.compose.ui.util.lerp
import com.android.compose.PlatformIconButton
import com.android.compose.ui.graphics.painter.rememberDrawablePainter
import com.android.systemui.ambientcue.ui.compose.modifier.animatedActionBorder
import com.android.systemui.ambientcue.ui.utils.FilterUtils
@@ -276,7 +274,7 @@ fun NavBarPill(
                                            .padding(start = 6.dp, end = 6.dp),
                                ) {
                                    Image(
                                        painter = rememberDrawablePainter(action.icon.drawable),
                                        painter = rememberDrawablePainter(action.icon.small),
                                        contentDescription =
                                            stringResource(
                                                id = R.string.ambient_cue_icon_content_description
@@ -318,7 +316,7 @@ fun NavBarPill(
                            } else {
                                // Smaller app icons
                                Image(
                                    painter = rememberDrawablePainter(action.icon.drawable),
                                    painter = rememberDrawablePainter(action.icon.small),
                                    contentDescription = action.label,
                                    modifier =
                                        Modifier.then(
+1 −1
Original line number Diff line number Diff line
@@ -343,7 +343,7 @@ private fun CloseButton(onCloseClick: () -> Unit, modifier: Modifier = Modifier)
@Composable
private fun Icon(action: ActionViewModel, backgroundColor: Color, modifier: Modifier = Modifier) {
    Image(
        painter = rememberDrawablePainter(action.icon.drawable),
        painter = rememberDrawablePainter(action.icon.small),
        contentDescription = stringResource(id = R.string.ambient_cue_icon_content_description),
        modifier =
            modifier
+11 −5
Original line number Diff line number Diff line
@@ -87,11 +87,17 @@ class AmbientCueInteractorTest(flags: FlagsParameterization) : SysuiTestCase() {
                    ActionModel(
                        icon =
                            IconModel(
                                small =
                                    applicationContext.resources.getDrawable(
                                        R.drawable.ic_content_paste_spark,
                                        applicationContext.theme,
                                    ),
                                "test.icon",
                                large =
                                    applicationContext.resources.getDrawable(
                                        R.drawable.ic_content_paste_spark,
                                        applicationContext.theme,
                                    ),
                                iconId = "test.icon",
                            ),
                        label = "Sunday Morning",
                        attribution = null,
+3 −3
Original line number Diff line number Diff line
@@ -42,7 +42,7 @@ class FilterUtilsTest : SysuiTestCase() {
    fun setUp() {
        calendarAction1 =
            ActionViewModel(
                icon = IconViewModel(mock(), "calendar_icon", 0),
                icon = IconViewModel(mock(), mock(), "calendar_icon", 0),
                label = "Sunday Morning",
                attribution = null,
                onClick = {},
@@ -51,7 +51,7 @@ class FilterUtilsTest : SysuiTestCase() {
            )
        calendarAction2 =
            ActionViewModel(
                icon = IconViewModel(mock(), "calendar_icon", 0),
                icon = IconViewModel(mock(), mock(), "calendar_icon", 0),
                label = "Sunday Evening",
                attribution = null,
                onClick = {},
@@ -60,7 +60,7 @@ class FilterUtilsTest : SysuiTestCase() {
            )
        mapsAction =
            ActionViewModel(
                icon = IconViewModel(mock(), "map_icon", 0),
                icon = IconViewModel(mock(), mock(), "map_icon", 0),
                label = "Philz Coffee San Carlos",
                onClick = {},
                onLongClick = {},
Loading