Scaffold with TimeText prevents content from being interacted with in Talkback mode [210721259]

Fixed

Bug

adexe s nau

Status Update

No update yet.

Description

va...@google.com

created issue #1

Dec 14, 2021 10:01PM

The standard setup of the Wear Scaffold is to include the TimeText() Composable in the timeText slot of the Scaffold.

When Talkback is enabled, this prevents the underlying content from being interactable entirely, I think because the TimeText() is an overlay that is placed on top of all content. I'm not sure of an easy workaround either, apart from removing the TimeText entirely.

Comments

jn...@google.com <jn...@google.com> Dec 31, 2021 11:17AM

Assigned to mk...@google.com.

ys...@google.com <ys...@google.com> #2Jan 14, 2022 10:14AM

The fix is done

ys...@google.com <ys...@google.com> #3Jan 14, 2022 10:16AM

Curious, is it intended that TimeText could have an interact-able composable? Like a Network symbol in leadingCurvedContent that shows network information when clicked?

ss...@google.com <ss...@google.com> #4Mar 23, 2022 03:02PM

CurvedText (used inside TimeText) had a semantic annotation so talkback could read the text to the user.
In the last patch (going out as alpha19) we removed that, since CurvedText is not a compose-ui node anymore, but we will look into adding it back again (with better bounds).

st...@google.com <st...@google.com> May 13, 2022 02:11PM

Reassigned to ss...@google.com.

ss...@google.com <ss...@google.com> Oct 3, 2022 10:55AM

Accepted by ss...@google.com.

ap...@google.com <ap...@google.com> #5Oct 4, 2022 12:07PM

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

commit 29737ba91ffe21e7b3132693094b5484c5cecbe1
Author: Sergio Sancho <ssancho@google.com>
Date: Mon Oct 03 14:18:01 2022

Create compose-ui nodes for curved text elements.

For each curvedText/basicCurvedText we create a compose-ui node that has
as content description the text of the (basic)curved text, and we size
and place it appropiatelly. This enables talkback to work properly with
curved text elements.
Minor change to the internal API for CurvedChild.initializeMeasure
Test: Manual, with and withouth talkback. + ./gradlew :wear:compose:compose-foundation:connectedCheck
Relnote: "curveText and basicCurvedText will now have a properly sized
and placed (but empty) compose-ui node associated with them, using the text as
content description, to properly work with talkback"
Bug: 210721259

Change-Id: I7af7ced923180839eee782fe2e45a1976c7e8ab4

M wear/compose/compose-foundation/samples/src/main/java/androidx/wear/compose/foundation/samples/CurvedWorldSample.kt
M wear/compose/compose-foundation/src/commonMain/kotlin/androidx/wear/compose/foundation/CurvedLayout.kt
M wear/compose/compose-foundation/src/commonMain/kotlin/androidx/wear/compose/foundation/CurvedContainer.kt
M wear/compose/compose-foundation/src/commonMain/kotlin/androidx/wear/compose/foundation/CurvedSize.kt
M wear/compose/compose-foundation/src/commonMain/kotlin/androidx/wear/compose/foundation/BasicCurvedText.kt
M wear/compose/compose-foundation/src/commonMain/kotlin/androidx/wear/compose/foundation/CurvedPadding.kt
M wear/compose/compose-foundation/src/androidAndroidTest/kotlin/androidx/wear/compose/foundation/SpyModifier.kt
M wear/compose/compose-foundation/src/commonMain/kotlin/androidx/wear/compose/foundation/CurvedComposable.kt
M wear/compose/integration-tests/demos/src/main/java/androidx/wear/compose/integration/demos/FoundationDemos.kt
M wear/compose/compose-foundation/src/commonMain/kotlin/androidx/wear/compose/foundation/CurvedModifier.kt

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

ss...@google.com <ss...@google.com> Oct 4, 2022 12:07PM

Marked as fixed.

na...@google.com <na...@google.com> #6Oct 24, 2022 01:38PM

The following release(s) address this bug:

androidx.wear.compose:compose-foundation:1.1.0-beta01