Fixed
Status Update
Comments
ap...@google.com <ap...@google.com> #2
Thanks Alex,
I agree with you; These selection controls should have default 48x48 size at least.
TouchDelegate
support in compose is still yet to be explored (
ap...@google.com <ap...@google.com> #3
I tentatively marked this blocked on
ra...@google.com <ra...@google.com>
ap...@google.com <ap...@google.com> #4
Project: platform/frameworks/support
Branch: androidx-main
commit 171901e7b5dd472f6bdc5352691662a6b4162363
Author: George Mount <mount@google.com>
Date: Tue Aug 10 18:52:23 2021
Give Buttons a minimum touch target size
Fixes: 188906426
Fixes: 188675934
Bug: 171509422
Bug: 128421932
Surface was clipping its semantics and pointer input touch
bounds. Clip regions are now allowing touch events to extend
beyond their clip to accommodate minimum touch target.
Relnote: "Allow clip to extend touch target bounds beyond
the clip region for minimum touch target purposes."
Test: new test for Button
Test: manually checked with accessibility scanner
Change-Id: I43e10218f7a20b5a8190ea838cef8eb8440928d1
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/ButtonTest.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/draw/GraphicsLayerModifierTest.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/input/pointer/RestrictedSizeTest.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/node/DelegatingLayoutNodeWrapper.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/node/InnerPlaceable.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/node/LayoutNodeWrapper.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/node/PointerInputDelegatingWrapper.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/semantics/SemanticsWrapper.kt
https://android-review.googlesource.com/1792408
Branch: androidx-main
commit 171901e7b5dd472f6bdc5352691662a6b4162363
Author: George Mount <mount@google.com>
Date: Tue Aug 10 18:52:23 2021
Give Buttons a minimum touch target size
Fixes: 188906426
Fixes: 188675934
Bug: 171509422
Bug: 128421932
Surface was clipping its semantics and pointer input touch
bounds. Clip regions are now allowing touch events to extend
beyond their clip to accommodate minimum touch target.
Relnote: "Allow clip to extend touch target bounds beyond
the clip region for minimum touch target purposes."
Test: new test for Button
Test: manually checked with accessibility scanner
Change-Id: I43e10218f7a20b5a8190ea838cef8eb8440928d1
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/ButtonTest.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/draw/GraphicsLayerModifierTest.kt
M compose/ui/ui/src/androidAndroidTest/kotlin/androidx/compose/ui/input/pointer/RestrictedSizeTest.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/node/DelegatingLayoutNodeWrapper.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/node/InnerPlaceable.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/node/LayoutNodeWrapper.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/node/PointerInputDelegatingWrapper.kt
M compose/ui/ui/src/commonMain/kotlin/androidx/compose/ui/semantics/SemanticsWrapper.kt
Description
No description yet.