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

Commit 14f5eca4 authored by burakov's avatar burakov
Browse files

[bc25] Let `OverlayShade` wrap its content, for easier responsiveness. 

This will allow each of the caller composables to determine the width
depending on the content, which is useful since NSSL already determines
the notification width itself.

Test: Existing unit tests build and pass.
Bug: 338033836
Flag: com.android.systemui.dual_shade
Change-Id: I75732f785c6f73a912d677cb7578df0443dbba90
parent e7b166db

packages/SystemUI/compose/features/src/com/android/systemui/shade/ui/composable/OverlayShade.kt

+1 −9
Original line number Diff line number Diff line
@@ -24,7 +24,6 @@ import androidx.compose.foundation.layout.Row 
import androidx.compose.foundation.layout.Spacer

import androidx.compose.foundation.layout.fillMaxSize

import androidx.compose.foundation.layout.padding

import androidx.compose.foundation.layout.width

import androidx.compose.foundation.shape.RoundedCornerShape

import androidx.compose.material3.MaterialTheme

import androidx.compose.runtime.Composable
@@ -95,12 +94,7 @@ private fun SceneScope.Panel( 
    modifier: Modifier = Modifier,

    content: @Composable () -> Unit,

) {

    Box(

        modifier =

            modifier

                .width(OverlayShade.Dimensions.PanelWidth)

                .clip(OverlayShade.Shapes.RoundedCornerPanel)

    ) {

    Box(modifier = modifier.clip(OverlayShade.Shapes.RoundedCornerPanel)) {

        Spacer(

            modifier =

                Modifier.element(OverlayShade.Elements.PanelBackground)
@@ -133,8 +127,6 @@ object OverlayShade { 
    object Dimensions {

        val ScrimContentPadding = 16.dp

        val PanelCornerRadius = 46.dp

        // TODO(b/338033836): This width should not be fixed.

        val PanelWidth = 390.dp

    }



    object Shapes {