New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add semantic properties to DialogWindow, Popup and Dialog #698
Add semantic properties to DialogWindow, Popup and Dialog #698
Conversation
compose/ui/ui-test-junit4/src/desktopTest/kotlin/androidx/compose/ui/test/FiltersTest.kt
Outdated
Show resolved
Hide resolved
compose/ui/ui-test-junit4/src/desktopTest/kotlin/androidx/compose/ui/test/FiltersTest.kt
Show resolved
Hide resolved
@@ -51,7 +51,7 @@ class TextActionsTest { | |||
TestTextField("hello") | |||
} | |||
|
|||
with(rule.onNodeWithTag("tag")){ | |||
with(rule.onNodeWithTag("tag")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like only whitespace changed in this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I fixed it here and in a bunch of other test files.
3d4bd82
to
5fd71e6
Compare
ed8278e
to
09cd413
Compare
5fd71e6
to
8b8cc84
Compare
09cd413
to
311a2e5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we reuse
Line 108 in 55f3f01
private val semanticsModifier = EmptySemanticsElement |
?
UPD: Will be fixed in separate PR
90c1950
to
4862e61
Compare
311a2e5
to
4769460
Compare
4769460
to
e732e9a
Compare
Proposed Changes
dialog()
semantic property toDialogWindow
.dialog()
semantic property toDialog
popup()
semantic property toPopup
.Testing
Test: Added (and enabled) unit tests for this.