IssueTracker

Agree

Also it causes slider to be unusable while onValuChangeFinished != null

#comment1 could you provide all of your dependencies for your project that you saw the issue in? It might have been fixed by a foundational module that Material3 depends on.

Ran into the same issue, the onValuChangeFinished lambda is being passed as a remember key so recreates the state if that lambda reference changes (as seen in https://issuetracker.google.com/issues/322269951#comment5). You can work around it by remembering the lambda itself.

val context = LocalContext.current
var sliderPosition by remember { mutableStateOf(0f) }
val onValueChangeFinished: () -> Unit = remember {
  { Toast.makeText(context, sliderPosition.toString(), Toast.LENGTH_SHORT).show() }
}
Text(text = sliderPosition.toString())

Slider(
    value = sliderPosition, 
    onValueChange = { sliderPosition = it }, 
    onValueChangeFinished = onValueChangeFinished
)

Feb 19 is a U.S. holiday, there are 12+ impacted so far, Kevin, could you please take a look on tues once you are back at work? ty!

Yup, taking a look. Thanks Connie!

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

• androidx.compose.material3:material3:1.2.1
• androidx.compose.material3:material3-android:1.2.1
• androidx.compose.material3:material3-desktop:1.2.1
• androidx.compose.material3:material3:1.3.0-alpha02
• androidx.compose.material3:material3-android:1.3.0-alpha02
• androidx.compose.material3:material3-desktop:1.3.0-alpha02

Yeah it kinda sucks. They seem to be doing some architectural changes to TextFields for 1.5, so fingers crossed for that.

For anyone still experiencing this, the workaround I wrote in #comment4 is working pretty well in two production apps I have on the store, without any issues. You may need to adjust the code a bit for some use-cases, but it generally works out-of-the box, at least until a proper fix is released.

Hi! The new BasicTextField2 in Compose 1.6 alphas seems really promising. For now it seems like the issue of staying on top of the keyboard while typing is fixed; however, on a long text, selecting the text in the area where the keyboard is about to show doesn't scroll the text field to keep the cursor in view.

Once that gets fixed, this issue will be fully resolved and can be effectively closed!

Any updates or ETAs on that?

Any new developments on this issue? I'm using BasicTextField to implement a rich edit window (it can attach images, etc.), but it won't keep the text above the keyboard if I add Modifier.verticalScrollable() :(

which I think is still pretty important

Looks promising! Does that mean this is finally fixed? And when can we expect this to be released so we can test it?

It will be fixed in BasicTextField2 in the next 1.7.0 alpha release, alpha02. We are not planning to fix it in the current BasicTextField.

Great news finally!

This is a bit unrelated, but will the name of the BasicTextField2 API be always like this? Or will it eventually replace BasicTextField?

Because with the 2 it sounds like this "secondary" thing, not like the primary API that everyone should be using.

Other than that, thanks for the amazing work!

Nope. That and other questions answered here: https://www.droidcon.com/2023/10/06/reimagining-text-fields-in-compose-2/

Great to hear! Thanks for sharing the talk, and props to you guys for this amazing work!

#comment26 You should probably file a separate issue for that, as it seems to be unrelated to this one.

Okay I just played with the new BasicTextField... and it seems like this has been only half-fixed.

Inputting anything from the keyboard brings the cursor into focus, which is great. Adding new lines keeps the cursor in view, which is great.

However, tapping the text field (which opens the IME) in an area that will eventually be under the keyboard, does not scroll the cursor to stay in view.

My wacky workaround at the very beginning of this issue does that correctly, and not only for the keyboard, but for any size changes (my app has, in addition to system keyboard, an internal keyboard, and it works the exact same way). If the cursor is in view, I keep it in view no matter what. If the cursor has been scrolled out-of-view, then it's okay to bring it in view only once there's input from the user.

Is this behavior intended? I'm not 100% sure, but I believe that Android's EditText was keeping the cursor in view once the IME is shown.

I can make a video demonstration with all of these (New BasicTextField, legacy BasicTextField, my workaround, and EditText)

#comment28 If you're talking about the cursor not scrolling to stay on top of keyboard when it opens first, I agree, this is relevant here. I just noticed it myself as well.

In terms of the keyboard not showing at all the second time you click the text field, I think that's a different bug, and should be reported as its own issue with all the context of where and how that happens to you.

Also note that BasicTextField2 from 1.6.x releases is outdated, so make sure to use the new BasicTextField in 1.7.0 betas.

To comment #29 I believe you need to handle ime insets for that to work.

#comment31 Yes, I did apply the imePadding() Modifier on the BasicTextField, and it doesn't work even with it.

This isn't even IME-specific; any change in the TextField's size should try to keep the cursor in the viewport (if it was already visible before the size change).

imePadding does exactly just that: add padding, which in turn shrinks the size of the BasicTextField from the bottom. Currently, the new BasicTextField does not react to this at all, thus causing the cursor to stay under the keyboard.

My workaround (from #comment4) does this correctly, making sure that the cursor stays in view on size changes (only if the cursor was already in view, if not, do nothing because something else may have caused the size change).

It is still, however, a workaround (a wacky one to be clear, I'm not even sure I understand my own code, but it works flawlessly in production), and there's nothing than a proper upstream fix for everybody, that would be more performant and just less wacky.

#comment33 Again, the BasicTextField2 that's in Compose 1.6.x is outdated.

If you want to try the latest API, update to Compose 1.7.0-beta03 (which is the latest beta as of now), and use BasicTextField (which now uses the new implementation). This issue is mostly fixed in 1.7.

bringIntoViewRequester is irrelevant here, because we're talking about the internal scroll of the text field (especially in the case when it takes full screen space). It's not about scrolling some LazyList to keep the field in view, it's about scrolling the field itself to keep the cursor in view.

Again, if you update to 1.7.0 and still face the issue of the keyboard not opening the second time, that's a different issue, please report it separately.

#comment35 It's... partially fixed. I guess you still have to do Modifier.imePadding(), but the cursor will still come into view... once you type something.

However, since I still have to use the older BasicTextField sometimes because I need AnnotatedString, and because the current fix doesn't satisfy me, I just use a workaround for now, using BringIntoViewRequester. It's much simpler, cleaner and easier to understand than the abomination I wrote in #comment4, and is generally more flexible.

To keep the cursor in view on text changes (like the current fix), you need to add a bringIntoViewRequester to your BasicTextField, and then get use the TextLayoutResult from onTextLayout to get the bounding of the current cursor position, and call request bringIntoView using that Rect.

To complete the fix, you'll wanna store the TextLayoutResult in a state, and add a LaunchedEffect on TextFieldValue.selection/TextFieldState.selection changes, which does the exact same thing as before (calling bringIntoView with cursor rect).

Do the same on size changes (in Modifier.onPlaced), and you're fully set. The cursor will always be brought to view on size/selection/text changes, while still allowing users to scroll it away when none of these change, works well on desktop, and you can of course choose which triggers to keep.

You can also add some padding to the Rect you pass to bringIntoView, for example to keep some space between the line where the cursor is and the IME/bottom/top of screen.

I generally use this alongside a verticalScroll modifier to apply some internal non-clipping padding, but it should work fine without it, at least on the new BasicTextField.