diff --git a/CHANGELOG.md b/CHANGELOG.md index 44231ff4eed..f1d98e41ebc 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,6 +5,7 @@ ### Fixes - Backfill release, environment, distribution, tags, and app version/build—and use the matching replay-on-error sample rate—for `ApplicationExitInfo` ANR and native crash events captured before SDK initialization, without reusing options cached by a later app update ([#5762](https://github.com/getsentry/sentry-java/pull/5762)) +- `SentryTagModifierNode.isImportantForBounds` now matches the default behavior and returns `true` ([#5789](https://github.com/getsentry/sentry-java/pull/5789)) ## 8.49.0 diff --git a/sentry-compose/src/androidMain/kotlin/io/sentry/compose/SentryModifier.kt b/sentry-compose/src/androidMain/kotlin/io/sentry/compose/SentryModifier.kt index 787c66b3b0b..3c8fb48c35a 100644 --- a/sentry-compose/src/androidMain/kotlin/io/sentry/compose/SentryModifier.kt +++ b/sentry-compose/src/androidMain/kotlin/io/sentry/compose/SentryModifier.kt @@ -55,13 +55,11 @@ public object SentryModifier { } // SemanticsModifierNode.isImportantForBounds() was added as an abstract method in - // compose-ui 1.11. Classes compiled against earlier versions lack this method in + // compose-ui 1.11. Classes compiled against earlier versions lack this method in // their bytecode, which causes AbstractMethodError when the accessibility tree is - // traversed on 1.11+ runtimes. We can't use the `override` keyword here because - // the method doesn't exist in the compile-time dependency (compose-ui 1.6.x), but - // the JVM satisfies the abstract-method requirement at runtime via signature - // matching. SentryTagModifierNode only stores a semantic tag and has no visual - // effect on layout, so it is not important for bounds. - @Suppress("unused") fun isImportantForBounds(): Boolean = false + // traversed on 1.11+ runtimes. + // Returning true to match the default behavior + // https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/node/SemanticsModifierNode.kt;l=69-83;drc=bd7809b4bc9205721c2f1bc681694dd348885849 + @Suppress("unused") fun isImportantForBounds(): Boolean = true } }