IssueTracker

Thanks for your feedback! We have just landed a CL that enables to configure a FLAG_SECURE on a dialog. By default it will inherit it from window launching it, so all compose popups from now on should inherit the flag. The flag can be also configured for AlertDialog. This will be available in the next release.

I am tremendously grateful that you added fixes for this and https://issuetracker.google.com/issues/143778148!

For testing purposes, by "the next release", do you mean alpha05? I don't know when your internal cutoff is for what makes it into a particular release.

Thanks again!

Sorry, yes you are right, alpha05 is the release that will have this change included. Thanks!

Yes, it is available in the latest compose released yesterday

Will there ever be support for Android 11 and below?

Stretch effect overscroll will be supported only for android 12 and above. For versions prior to that, the original glow edge overscroll will be shown.

This is done by design, so that overscroll in every app will feel the same, leaving users with the feel of consistency.

I hope that makes sense.

This looks broken indeed. I believe this should not happen if you are using latest snapshot where we properly support stretch effect. Could you share more information about this sample's setup? Compose version, code with snippet (is it LazyColumn, Modifier.verticalScroll or your own Modifier.scrollable) and any other relevant info.

Thanks.

I used LazyColumn in compose_version = '1.1.0-alpha03' . I just created a brand new project to test this, still the same issue. I am attaching android version,build number... if that helps.

//code snippet

@Composable
fun ScreenMain() {
    LazyColumn(
        modifier = Modifier.fillMaxSize(),
        content = {

            items(30) {
                RowItem(number = it)
            }
        })
}

@Composable
fun RowItem(number: Int) {
    Box(
        modifier = Modifier
            .fillMaxWidth()
            .height(50.dp)
            .padding(all = 8.dp)
            .border(width = 1.dp, color = Color.Red),
        contentAlignment = Alignment.Center,
    ) {
        Text(text = "Row $number", color = Color.Black)
    }
}

Thanks for additional info.

I tried this example in the emulator, pixel 3 and some various Samsungs, and I cannot reproduce the issue for some reason.

There's a version check that prohibits any kind of overscroll for S devices on 1.1.0-alpha03.

Could you please try a real phone with android S if you have some? Or a fresh emulator image?

Alternatively, you can turn off overscroll for now and wait for alpha04 where stretch overscroll for S devices will be fully supported.

Thanks for the reply. I reinstalled Android S Images and then created a new virtual device. Issue seems to be resolved now. Probably issue was with the S preview.

Oh, that makes sense as well, as some S previews had incomplete support for stretch in the RenderNode, so it might have been broken there.

I'm glad that everything is working well for you now!

Is the default material color correct for both light mode and dark mode? I see 0xff666666 hardcoded here: https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:compose/foundation/foundation/src/androidMain/kotlin/androidx/compose/foundation/gestures/OverScrollConfiguration.kt;l=38?q=OverScrollConfiguration%20lang:kotlin&ss=androidx%2Fplatform%2Fframeworks%2Fsupport:compose%2F

But IIRC this color is themed using android:colorEdgeEffect, for example here: https://cs.android.com/android/platform/superproject/+/master:frameworks/base/core/res/res/values/themes_material.xml;l=775?q=themes_material.xml&ss=android%2Fplatform%2Fsuperproject:frameworks%2Fbase

Might need to update MaterialTheme.kt and the material-components-android-compose-theme-adapter project too if this is the case.

Yup, you are right.

The default color is design system agnostic. Since we're waiting for API feedback - we cannot wire up foundation glow api with material just yet. It will be done as soon as we will graduate OverScrollConfiguration from experimental.

you can do it manually for now in your app.