view.requestFocus(FOCUS_LEFT, rect)/ FOCUS_UP/DOWN/RIGHT, will skip a item. [388590015]

Fixed

Bug

Status Update

No update yet.

Description

jr...@google.com

created issue #1

Jan 8, 2025 10:56PM

Jetpack Compose version: Latest Jetpack Compose component used: AndroidView, Focus Android Studio Build: uild #AI-233.14808.21.2331.11574862 Kotlin version: Latest

Steps to Reproduce or Code Sample to Reproduce:

See attached apk for repro demo.

Correct behavior: Using arrow key to focus on button1 first. And then press keyboard key D. This will trigger button2.requestFocus(FOCUS_RIGHT, button1Rect). It will work like press the arrow key right.
Incorrect behavior: Using arrow key to focus on button3 first. And then press keyboard key A. This will trigger button2.requestFocus(FOCUS_LEFT, button3Rect). Now the focus will jump over button 2 and focus on button 1 instead.

Also see attached image diagram to demonstrate what the minimum requirement to trigger the issues

hellogoogle3compose.apk

12 MB

Download

unnamed (1).png

51 KB

View

Download

Comments

mo...@google.com <mo...@google.com> #2Jan 9, 2025 05:24PM

Status: Won't Fix (Intended Behavior)

Project: platform/frameworks/support
Branch: androidx-main
Author: Marcello Galhardo <mgalhardo@google.com>
Link: https://android-review.googlesource.com/3375573

Set compilationName for Android Test on JVM in savedstate

Expand for full commit details

Set `compilationName` for Android Test on JVM in `savedstate` 
 
* Fixes "Expect actual declaration for (...) `androidHostTest`" 
 
Bug: 381063087 
Test: ViewModelScenarioTest 
Change-Id: Idb7bd5062a5dc408a83524cbb3ce737058cca14f

Files:

M lifecycle/lifecycle-viewmodel-testing/build.gradle

Hash: 512b3f0b2fd331fa346f686f9f473462f02d4174
Date: Tue Nov 26 14:12:55 2024

jr...@google.com <jr...@google.com> #3Jan 9, 2025 11:57PM

Status: Assigned (reopened)

Project: platform/frameworks/support
Branch: androidx-main
Author: Marcello Galhardo <mgalhardo@google.com>
Link: https://android-review.googlesource.com/3375898

Set robolectric in lifecycle-viewmodel-testing

Expand for full commit details

Set `robolectric` in `lifecycle-viewmodel-testing` 
 
Bug: 381063087 
Test: ViewModelScenarioTest 
Change-Id: Ia5466af402a10291c10e45662b6b5d9a76748a34

Files:

M lifecycle/lifecycle-viewmodel-testing/build.gradle

Hash: 52a8c8438f3522eced0edfc243419179796cd9e1
Date: Tue Nov 26 11:46:10 2024

ap...@google.com <ap...@google.com> #4Jan 11, 2025 02:53AM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-main
Author: Marcello Galhardo <mgalhardo@google.com>
Link: https://android-review.googlesource.com/3375899

Add RobolectricTest to lifecycle-viewmodel-testing

Expand for full commit details

Add `RobolectricTest` to `lifecycle-viewmodel-testing` 
 
Bug: 381063087 
Test: ViewModelScenarioTest 
Change-Id: Ia80fcf0ccc49f232247de97a3046f445854f5203

Files:

A lifecycle/lifecycle-viewmodel-testing/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/testing/RobolectricTest.android.kt
A lifecycle/lifecycle-viewmodel-testing/src/androidUnitTest/kotlin/androidx/lifecycle/viewmodel/testing/RobolectricTest.android.kt
M lifecycle/lifecycle-viewmodel-testing/src/commonTest/kotlin/androidx/lifecycle/viewmodel/testing/DefaultCreationExtrasTest.kt
A lifecycle/lifecycle-viewmodel-testing/src/commonTest/kotlin/androidx/lifecycle/viewmodel/testing/RobolectricTest.kt
M lifecycle/lifecycle-viewmodel-testing/src/commonTest/kotlin/androidx/lifecycle/viewmodel/testing/ViewModelScenarioTest.kt
A lifecycle/lifecycle-viewmodel-testing/src/nonAndroidTest/kotlin/androidx/lifecycle/viewmodel/testing/RobolectricTest.nonAndroid.kt

Hash: 70cfbc46068bdf9899d15020498a7e49641b3a53
Date: Tue Nov 26 11:53:48 2024

na...@google.com <na...@google.com> #5Jan 29, 2025 06:12PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Marcello Galhardo <mgalhardo@google.com>
Link: https://android-review.googlesource.com/3375896

Move ViewModelScenarioTest.kt to commonTest

Expand for full commit details

Move `ViewModelScenarioTest.kt` to `commonTest` 
 
Bug: 381063087 
Test: ViewModelScenarioTest 
Change-Id: I33c5c0722229adbc841ba63e728c6d772e76be14

Files:

D lifecycle/lifecycle-viewmodel-testing/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/testing/ViewModelScenarioInstrumentedTest.kt
M lifecycle/lifecycle-viewmodel-testing/src/commonTest/kotlin/androidx/lifecycle/viewmodel/testing/ViewModelScenarioTest.kt

Hash: 958948fdd1f1d62bdc73f9440ae21235b9b51410
Date: Tue Nov 26 11:30:08 2024

Issue 388590015

Description

Issue summary

Comments

mo...@google.com <mo...@google.com> #2Jan 9, 2025 05:24PM

jr...@google.com <jr...@google.com> #3Jan 9, 2025 11:57PM

ap...@google.com <ap...@google.com> #4Jan 11, 2025 02:53AM

na...@google.com <na...@google.com> #5Jan 29, 2025 06:12PM

Add comment

Issue metadata