Deprecate onImeActionPerformed [179071523]

Fixed

Feature Request

Status Update

No update yet.

Description

ra...@google.com

created issue #1

Feb 1, 2021 10:27PM

OnImeActionPerformed is being replaced by KeyboardActions. Deprecate and remove onImeActionPerformed

Comments

ra...@google.com <ra...@google.com> Feb 3, 2021 07:22PM

Accepted by ra...@google.com.

ap...@google.com <ap...@google.com> #2Feb 3, 2021 09:34PM

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

ra...@google.com <ra...@google.com> Feb 3, 2021 10:13PM

Marked as fixed.

so...@google.com <so...@google.com> Feb 4, 2021 10:58AM

Status: Assigned (reopened)

so...@google.com <so...@google.com> #3Feb 4, 2021 12:29PM

Hey Ralston,

I've just noticed today that onImeActionPerformed has not been removed from the CoreTextField, I assume this is needed to deprecate it from public API. Yet, the comment reference this closed ticket

    textStyle: TextStyle = TextStyle.Default,
    // TODO(b/179071523): Deprecate and remove onImeActionPerformed.
    onImeActionPerformed: (ImeAction) -> Unit = {},

If you're tracking this removal somewhere else, please ignore this and re-mark this bug as fixed :) Otherwise you might want to keep the bug open until you remove that parameter.

ra...@google.com <ra...@google.com> #4Feb 4, 2021 05:37PM

Accepted by ra...@google.com.

Thanks for pointing this out Anastasia. I was thinking of doing this as part of

b/178633932

.
This bug was only meant to deprecate onImeActionPerformed.

But the description of the bug and the code TODO says deprecate and remove.

Let's keep this bug open till I remove onImeActionPerformed.

Message last modified on Feb 4, 2021 05:38PM

ap...@google.com <ap...@google.com> #5Feb 5, 2021 12:36AM

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

commit 4934ede8ef1985a875d383ef15adeddbfcba48fe
Author: Ralston Da Silva <ralu@google.com>
Date: Thu Feb 04 13:29:39 2021

Remove onImePerformed from CoreTextField

onImeActionPerformed has been replaced by keyboardActions.
This CL removes this parameter now that all the deprecated
functions using it were removed as part of aosp/1575163.

Bug: 179071523
Test: N/A
Change-Id: Id4853dd719779ff393d6b1a74b69da2c5b3ee620

M compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/text/CoreTextField.kt

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

ae...@google.com <ae...@google.com> Feb 9, 2021 05:35AM