IssueTracker

Does this happen if you temporarily remove/disable the pull-refresh part?

We're facing the same issue with TopAppBar and TopAppBarDefaults.pinnedScrollBehavior. When scrolling quickly on a lazy layout, the app bar container color does not respond.

Repo: https://github.com/smelfungus/NestedScrollAppBarIssue/tree/master

Video: https://drive.google.com/file/d/1XmJENXecUYq0hAC2xGBvrr2k_XeeXvZe/view?usp=sharing

val scrollBehavior = TopAppBarDefaults.pinnedScrollBehavior()

Scaffold(
    modifier = Modifier
        .fillMaxSize()
        .nestedScroll(scrollBehavior.nestedScrollConnection),
    topBar = {
        TopAppBar(
            title = {
                Text(
                    modifier = Modifier.fillMaxWidth(),
                    text = "Hello!",
                    textAlign = TextAlign.Center,
                )
            },
            scrollBehavior = scrollBehavior,
        )
    },
) { innerPadding ->
    LazyColumn(
        modifier = Modifier.padding(innerPadding),
        contentPadding = PaddingValues(16.dp),
        verticalArrangement = Arrangement.spacedBy(8.dp),
    ) {
        items(100) {
            Text(
                text = "Item $it",
                modifier = Modifier
                    .fillMaxWidth()
                    .background(MaterialTheme.colorScheme.surfaceContainerHigh)
                    .padding(24.dp),
            )
        }
    }
}

I am facing the same bug and i think the root case for it is that the PinnedScrollBehavior doesn't override the onPostFling in its NestedScrollConnection implementation.

I made a crude simple custom version and added

override suspend fun onPostFling(consumed: Velocity, available: Velocity): Velocity {
    if (available.y > Velocity.Zero.y)
        state.contentOffset = 0f

    return super.onPostFling(consumed, available)
}

and then it works fine for me!

Project: platform/frameworks/support
Branch: androidx-main
Author: Shalom Gibly <sgibly@google.com>
Link:      https://android-review.googlesource.com/3375689

Fix the app-bars behaviors to detect end of scroll

Fix the app-bars behaviors to detect end of scroll 
 
Apply a fix to all the behaviors to detect the end of scroll in the post 
fling function. This will better ensure an app-bar color change when the 
content is scrolled all the way. 
 
Bug: 293665988 
Test: Manual 
Relnote: "Fix the top and bottom app-bar behaviors to reliably change 
color when content is scrolled all the way." 
 
Change-Id: Idc4e834695cbd7cb8099f7b63cf21d5b764f1c81

@marko I can't speak to the PullToRefresh feature as I've not worked on it, but to generally answer your question: I would say that is ultimately your call to make. Personally I would recommend using Material3, as it has a longer window of active development (feature requests, bug fixes, other dev time resources) while the goal is to sunset M2 support over time. However M2 is of course more feature complete at this time.

PullToRefresh is not currently on the M3 radar however you are welcome to file a feature request.

@se...@google.com maybe not so good to post the full mail here?

The following release(s) address this bug.It is possible this bug has only been partially addressed:

• androidx.compose.material3:material3:1.1.0-alpha04