Fixed
Status Update
Comments
tc...@google.com <tc...@google.com> #2
This bug/feature is targeting an iteration that has already passed. Can you please update the status or iteration? Thanks!
ra...@google.com <ra...@google.com> #3
Sorry my bad, I actually created this after the build was cut. Thanks for pointing that out. Updated the iteration
ap...@google.com <ap...@google.com> #4
Project: platform/frameworks/support
Branch: androidx-main
commit 576510cc7d0d663fbb9f187d6ca8ef990f6499a2
Author: Ralston Da Silva <ralu@google.com>
Date: Thu Jan 07 14:13:29 2021
Mark FocusRequester.createRefs() as experimental API
We need to improve this API further. Refer to the linked bug for details.
Since this is not a priority for now I'm marking this API as experimental.
Bug: 177000821
Test: ./gradlew compose:ui:ui:connectedCheck -P android.testInstrumentationRunnerArguments.class=androidx.compose.ui.focus.FocusRequesterTest
RelNote: FocusRequester.createRefs is now marked as experimental as it might change.
Change-Id: I2d898d56ed0ac33f5a08253509cfd811ee0e8a5d
M compose/foundation/foundation/src/androidAndroidTest/kotlin/androidx/compose/foundation/FocusableTest.kt
M compose/foundation/foundation/src/androidAndroidTest/kotlin/androidx/compose/foundation/text/CoreTextFieldSoftKeyboardTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/textfield/TextFieldTest.kt
M compose/ui/ui/api/current.txt
M compose/ui/ui/api/public_plus_experimental_current.txt
M compose/ui/ui/api/restricted_current.txt
M compose/ui/ui/integration-tests/ui-demos/src/main/java/androidx/compose/ui/demos/focus/CustomFocusOrderDemo.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/focus/CustomFocusTraversalTest.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/focus/FocusChangedTest.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/focus/FocusRequesterTest.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/focus/FocusRequester.kt
https://android-review.googlesource.com/1540714
Branch: androidx-main
commit 576510cc7d0d663fbb9f187d6ca8ef990f6499a2
Author: Ralston Da Silva <ralu@google.com>
Date: Thu Jan 07 14:13:29 2021
Mark FocusRequester.createRefs() as experimental API
We need to improve this API further. Refer to the linked bug for details.
Since this is not a priority for now I'm marking this API as experimental.
Bug: 177000821
Test: ./gradlew compose:ui:ui:connectedCheck -P android.testInstrumentationRunnerArguments.class=androidx.compose.ui.focus.FocusRequesterTest
RelNote: FocusRequester.createRefs is now marked as experimental as it might change.
Change-Id: I2d898d56ed0ac33f5a08253509cfd811ee0e8a5d
M compose/foundation/foundation/src/androidAndroidTest/kotlin/androidx/compose/foundation/FocusableTest.kt
M compose/foundation/foundation/src/androidAndroidTest/kotlin/androidx/compose/foundation/text/CoreTextFieldSoftKeyboardTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/textfield/TextFieldTest.kt
M compose/ui/ui/api/current.txt
M compose/ui/ui/api/public_plus_experimental_current.txt
M compose/ui/ui/api/restricted_current.txt
M compose/ui/ui/integration-tests/ui-demos/src/main/java/androidx/compose/ui/demos/focus/CustomFocusOrderDemo.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/focus/CustomFocusTraversalTest.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/focus/FocusChangedTest.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/focus/FocusRequesterTest.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/focus/FocusRequester.kt
Description
val (focusRequester1, focusRequester2) = FocusRequester.FocusRequesterFactory
val (focusRequester1, focusRequester2) = FocusRequester.createRefs()
Things that we should improve on:
1. Both of these appear next to each other in autocomplete, and we would like users to use createRefs() instead of FocusRequesterFactory.
2. Issues relating to remembering the FocusRequester. (Not an issue for the functionality of FocusRequester, but if this pattern is used in other APIs that need remembered objects, this could be an issue)