IssueTracker

Jetpack Compose release version: 1.0.0-beta04
Android Studio Build: #AI-203.7148.57.2031.7242491

Steps to Reproduce:

1. Add focusModifier() before onFocusChanged {}
2. onFocusChanged is not called even if the element reaches focus
3. The required order is not documented in either of the modifiers

Sample project that demonstrates the bug can be found at https://github.com/hvrauhal/compose-focusmodifier-bug

In the following example BoxWithFocusModifiersInFaultyOrder is the not working version:

@Composable
private fun FocusDemo() {
    Column {
        Text("Use arrows to move focus over the following items:")
        BoxWithFocusModifiersInGoodOrder()
        BoxWithFocusModifiersInFaultyOrder()
        BoxWithFocusModifiersInGoodOrder()
        BoxWithFocusModifiersInFaultyOrder()
    }
}

@Composable
private fun BoxWithFocusModifiersInGoodOrder() {
    var color by remember { mutableStateOf(Color.Yellow) }
    Box(modifier = Modifier
        .onFocusChanged {
            color = if (it.isFocused) Color.Red else Color.Blue
        }
        .focusModifier()
        .background(color)
    ) {
        Text (text = "onFocusChanged is invoked", style = TextStyle(Color.White))
    }
}

@Composable
private fun BoxWithFocusModifiersInFaultyOrder() {
    var color by remember { mutableStateOf(Color.Yellow) }
    Box(modifier = Modifier
        .focusModifier()
        .onFocusChanged {
            color = if (it.isFocused) Color.Red else Color.Blue
        }
        .background(color)
    ) {
        Text("onFocusChanged is NOT invoked", style = TextStyle(Color.White))
    }
}