RememberPullRefreshState does not recompose with new offsets [263159832]

Fixed

Bug

Status Update

No update yet.

Description

[Deleted User]

created issue #1

Dec 20, 2022 10:21AM

Jetpack Compose version: 1.3.1 Jetpack Compose component used: androidx.compose.material:material

Android Studio Build: Build #AI-222.4345.14.2221.9321504, built on November 22, 2022

Kotlin version: 1.7.20

Steps to Reproduce or Code Sample to Reproduce:

Start with this example
Add a statusbar padding to the refreshingOffset & refreshTreshold (see code below)
See that the statusbar offsets are not handled.

It seems that insets are first laid out without any insets, the next composition gets the real insets. However as the pull to refresh state is being remembered the offset can never be updated. This causes the pull to refresh indicator to be below the statusbar in our case.

A workaround for now is getting the insets from the activity window insets, however this should not be the proper workaround.

Example code for adding statusbar padding:

    val state = rememberPullRefreshState(
            refreshing = viewState.showPullToRefreshLoader,
            onRefresh = viewModel::onPullToRefreshTriggered,
            refreshingOffset = PullRefreshDefaults.RefreshingOffset + WindowInsets.statusBars.asPaddingValues().calculateTopPadding(),
            refreshThreshold = PullRefreshDefaults.RefreshThreshold + WindowInsets.statusBars.asPaddingValues().calculateTopPadding()
           )

It seems to me that the pull to refresh state also needs to recompose on new offsets.

Comments

lp...@google.com <lp...@google.com> Dec 21, 2022 03:05PM

Reassigned to lp...@google.com.

ap...@google.com <ap...@google.com> #2Dec 21, 2022 05:33PM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-main

commit 9082f62682f853ad5251a1c79dde9eccba7abdd9
Author: Max Alfonso-Ying <maxying@google.com>
Date: Thu Apr 18 00:34:40 2024

[M2 text field] Apply background to decoration box

...instead of to the BasicTextField, so changing the
backgroundColor will properly change the decoration
box's background color.

Fixes:

b/307694651

Test: added unit tests
Relnote: "Fix backgroundColor not applying to
TextFieldDecorationBox and OutlinedTextFieldDecorationBox.
Decoration boxes now accept a `shape` parameter."
Change-Id: I371c26718597cb36ac537e9412ce476532afb40d

M compose/material/material/api/current.txt
M compose/material/material/api/restricted_current.txt
M compose/material/material/integration-tests/material-demos/src/main/java/androidx/compose/material/demos/TextFieldDecorationBoxDemos.kt
M compose/material/material/src/androidInstrumentedTest/kotlin/androidx/compose/material/textfield/TextFieldDecorationBoxTest.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/OutlinedTextField.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextField.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextFieldDefaults.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextFieldImpl.kt

https://android-review.googlesource.com/3046833

na...@google.com <na...@google.com> #3Jan 11, 2023 08:11PM

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

androidx.compose.material:material:1.4.0-alpha04

Issue 263159832

Description

Issue summary

Comments

lp...@google.com <lp...@google.com> Dec 21, 2022 03:05PM

ap...@google.com <ap...@google.com> #2Dec 21, 2022 05:33PM

na...@google.com <na...@google.com> #3Jan 11, 2023 08:11PM

Add comment

Issue metadata