IssueTracker

This is high priority for us. The "Priority" field on buganizer is misleading, it has a very specific internal meaning and interaction with other systems so we don't use it for actual prioritization for compose text issues. Unfortunately, the way we track actual priority is not visible externally.

Does this work as expected for you with Views? I was able to reproduce this bug with views as well – an EditText inside a ScrollView will not cause the view to scroll to keep the EditText in view if the keyboard covers it.

We've been taking a closer look at this bug and considered alternatives to fixing it, which require larger changes to the Compose keyboard controller and BringIntoView APIs. Unfortunately since Compose 1.2 is already in beta, the API is effectively frozen. We could explore more hacky ways to fix it in 1.2, but considering this behaviour matches what happens with Views, we decided it was better to fix properly with more time as opposed to fix it with a hack, increasing the tech debt and potentially creating other bugs. As a workaround, either don't call setDecorFitsSystemWindows(false) for now, or if you do, use ViewCompat.setWindowInsetsAnimationCallback to listen for when the keyboard stops animating and bring your text field into view using the BringIntoViewRequester API.

comment #5

I'm trying the workaround, but the ViewCompat.setWindowInsetsAnimationCallback is already set on the Compose side.

Compose code

Is it okay to overwrite this?

I realized that overwriting setWindowInsetsAnimationCallback disables ime animation on all screens when using navigation-compose.

Below code is the workaround I tried.

var text by remember { mutableStateOf("") }

val view = LocalView.current
val bringIntoViewRequester = remember { BringIntoViewRequester() }
val coroutineScope = rememberCoroutineScope()

// Workaround: Call it first to overwrite the setWindowInsetsAnimationCallback set by Compose.
WindowInsets.ime

DisposableEffect(view) {
    ViewCompat.setWindowInsetsAnimationCallback(
        view,
        object : WindowInsetsAnimationCompat.Callback(DISPATCH_MODE_STOP) {
            override fun onProgress(
                insets: WindowInsetsCompat,
                runningAnimations: MutableList<WindowInsetsAnimationCompat>
            ): WindowInsetsCompat {
                return WindowInsetsCompat.CONSUMED
            }

            override fun onEnd(animation: WindowInsetsAnimationCompat) {
                val isImeVisible =
                    WindowInsetsCompat.toWindowInsetsCompat(view.rootWindowInsets)
                        .isVisible(WindowInsetsCompat.Type.ime())
                if (isImeVisible) {
                    coroutineScope.launch {
                        bringIntoViewRequester.bringIntoView()
                    }
                }
            }
        }
    )
    onDispose {
        ViewCompat.setWindowInsetsAnimationCallback(view, null)
    }
}

Column(
    modifier = Modifier
        .fillMaxSize()
        .imePadding()
        .verticalScroll(rememberScrollState())
) {

    // ...
    
    TextField(
        value = text,
        onValueChange = { text = it },
        modifier = Modifier
            .bringIntoViewRequester(bringIntoViewRequester)
    )
}

It may be enough to just disable the setWindowInsetsAnimationCallback set by Compose.

val view = LocalView.current

// Workaround: Call it first to overwrite the setWindowInsetsAnimationCallback set by Compose.
WindowInsets.ime

LaunchedEffect(Unit) {
    ViewCompat.setWindowInsetsAnimationCallback(view, null)
}

comment8 this is not possible since you override the whole listener set by Compose. Either your listener works or the other, but never both.

It would be awesome to have an opt for an own listener or an API for the missing properties (e.g. animation fraction)