[Focus] In AndroidView requestFocus for an composable outside of the AndroidView will be treat it as move to next(similar to tab). [376142752]

Fixed

Bug

Status Update

No update yet.

Description

jr...@google.com

created issue #1

Oct 29, 2024 01:06AM

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:
1. See cl/690811317 MainActivity.kt
2. Create several buttons under AndroidView. And several buttons outside of AndroidView in compose
3. Add an onKeyEvent on AndroidView that request focus for the buttons in compose.
3. Press the key that set in the onKeyEvent. The focus will move to the next AndroidView button instead of buttons incompose.

Stack trace (if applicable):

Comments

be...@google.com <be...@google.com> #2Oct 29, 2024 01:43PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit c9e90d1e5671b7e6a8880aecbe32bb13f4b37560
Author: Siyamed Sinir <siyamed@google.com>
Date: Thu Oct 31 11:30:30 2019

Remove font related TODOs

- Ability to read font weight from the font

b/143700963

- Support for font collections

b/122476423

- Support for variable fonts

b/143703328

- support for multiple font family

b/127483182

- BaselineShift.lerp

b/143699225

Test: ./gradlew ui:ui-text:test
Test: ./gradlew ui:ui-text:connectedAndroidTest

Bug: 141645236
Bug: 143700963
Bug: 122476423
Bug: 143703328
Bug: 127483182
Bug: 143699225
Change-Id: I4ff001fd1e0fff4cd3aa0c64efae929057e8d5de

M ui/ui-text/api/0.1.0-dev03.txt
M ui/ui-text/api/current.txt
M ui/ui-text/api/public_plus_experimental_0.1.0-dev03.txt
M ui/ui-text/api/public_plus_experimental_current.txt
M ui/ui-text/api/restricted_0.1.0-dev03.txt
M ui/ui-text/api/restricted_current.txt
M ui/ui-text/src/main/java/androidx/ui/text/font/Font.kt
M ui/ui-text/src/main/java/androidx/ui/text/platform/AndroidParagraphHelper.kt
M ui/ui-text/src/main/java/androidx/ui/text/platform/TypefaceAdapter.kt
M ui/ui-text/src/main/java/androidx/ui/text/style/BaselineShift.kt
M ui/ui-text/src/test/java/androidx/ui/text/font/FontFamilyTest.kt
M ui/ui-text/src/test/java/androidx/ui/text/font/FontTest.kt

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

https://goto.google.com/android-sha1/c9e90d1e5671b7e6a8880aecbe32bb13f4b37560

ra...@google.com <ra...@google.com> #3Oct 29, 2024 11:24PM

Reassigned to mo...@google.com.

from dupped ticket

"CSS has font-variation-settings attribute
Also check if there are commonly used settings that we can add to the API.

Note:
Tracking ticket based on

https://docs.google.com/document/d/1MzK83egP-Jghc2ZW8IcIcJkud_vMM7jgDClCv4CkO_c/edit#bookmark=id.nyju92hr2ygw"

mo...@google.com <mo...@google.com> Nov 7, 2024 11:28PM

Accepted by mo...@google.com.

ap...@google.com <ap...@google.com> #4Dec 11, 2024 04:48PM

Marked as fixed.

bulk comment, fyi: we have added icebox to all the tickets because it was suggested by blintz in order to prevent auto-close tickets. we want to keep our Compose tickets to remain opened.

see

b/123001228#comment6

pr...@google.com <pr...@google.com> #5Jan 15, 2025 06:31PM

discussion from aosp/1672420

variable font settings can be included during font creation (for Font() functions.)

there are two parts of the story:

This font is weight X
I want to render the text in weight Y (regardless of the font weight)

ap...@google.com <ap...@google.com> #6Apr 2, 2025 11:44PM

important for Material.

Issue 376142752