Semantics not being merged in dev15 [161979921]

Fixed

Bug

Status Update

No update yet.

Description

ja...@google.com

created issue #1

Jul 23, 2020 03:15PM

In dev15, some tests are broken in Jetnews as a result of semantics not being merged anymore.

For example Box(Modifier.clickable) { Text("This text does not appear on the tree") }

The workaround is to useUnmergedTree = true

@Test
fun app_opensArticle() {
onRoot(useUnmergedTree = true).printToLog("jetnews")
onAllNodes(hasSubstring("Manuel Vivo"), useUnmergedTree = true)[0].performClick()
onAllNodes(hasSubstring("3 min read"), useUnmergedTree = true)[0].assertIsDisplayed()
}

Filip believes the regression happened in aosp/1360099

Comments

pa...@google.com <pa...@google.com> #2Jul 23, 2020 03:43PM

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

ae...@google.com <ae...@google.com> Jul 23, 2020 08:42PM

Accepted by ae...@google.com.

ae...@google.com <ae...@google.com> #3Jul 27, 2020 08:34PM

I'm working on it this week. Note that the culprit is actually the big substantive patch I landed on Jul 3, http://aosp/1354246

ae...@google.com <ae...@google.com> #4Jul 29, 2020 07:19AM

I started a WIP patch to allow non-unique keys: http://aosp/1375563 . It doesn't fix any bug so far because everything on the semantics client side (accessibility, tests) is just taking the first key/value pair, reproducing the old behavior. Semantics tests pass, next I'll check nothing else is broken and then make it work as expected with duplicate TestTags and Text semantics especially.

ap...@google.com <ap...@google.com> #5Aug 11, 2020 11:23PM

Marked as fixed.

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

commit 1531c13353d4caa1aa6a8f41a58adf67d911531f
Author: Alexandre Elias <aelias@google.com>
Date: Fri Aug 07 20:03:55 2020

Implement per-property merge policy.

This changes the current merging behavior into the default
implementation of an open function called "merge()", and adds different
implementations for the following:
- Text and Label: concatenates strings with comma in between (same as
TalkBack's native merging behavior). This behavior will probably
evolve further in future patches to merge text layout for accessibility.
- TestTag and Hidden: does not merge at all.
- Popup and Dialog: throws an exception.

Note that the latter two behaviors require carefully distinguishing
between collapsing and merging so that, for example, collapsed TestTags
are not dropped as well.

Also switch ListItem to have mergeAllDescendants (should've had that
before anyway), which has more multiple-Text stress tests than
Button/Toggleable. As an example of how TalkBack behaves with a two-line
ListItem, see

https://screenshot.googleplex.com/06KZaX6VfsG

Fixes: 161979921
Test: depthFirstLabelConcat, clickableTest_clickOnChildText
Relnote: "Added mergePolicy lambda to SemanticsPropertyKey. This can be
used to define a custom policy for mergeAllDescendants semantics
merging. The default policy is to use the parent value if already
present, otherwise the child value."

Change-Id: Iaf6c4cc327017ee492f4d8334c8df5167d33df58

M ui/ui-core/api/current.txt
M ui/ui-core/api/public_plus_experimental_current.txt
M ui/ui-core/api/restricted_current.txt
M ui/ui-core/src/androidAndroidTest/kotlin/androidx/compose/ui/semantics/SemanticsTests.kt
M ui/ui-core/src/commonMain/kotlin/androidx/compose/ui/semantics/SemanticsConfiguration.kt
M ui/ui-core/src/commonMain/kotlin/androidx/compose/ui/semantics/SemanticsNode.kt
M ui/ui-core/src/commonMain/kotlin/androidx/compose/ui/semantics/SemanticsProperties.kt
M ui/ui-core/src/commonMain/kotlin/androidx/compose/ui/semantics/SemanticsWrapper.kt
M ui/ui-foundation/src/androidAndroidTest/kotlin/androidx/compose/foundation/ClickableTest.kt
M ui/ui-material/integration-tests/material-demos/src/main/java/androidx/compose/material/demos/ButtonDemo.kt
M ui/ui-material/src/androidAndroidTest/kotlin/androidx/compose/material/TabTest.kt
M ui/ui-material/src/commonMain/kotlin/androidx/compose/material/ListItem.kt

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

Issue 161979921