-
Notifications
You must be signed in to change notification settings - Fork 89
Add API for iOS IME Options #2108
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
Conversation
...e/ui/ui-text/src/uikitMain/kotlin/androidx/compose/ui/text/input/PlatformImeOptions.uikit.kt
Outdated
Show resolved
Hide resolved
f6df067
to
e7c2b0b
Compare
compose/ui/ui-text/src/commonMain/kotlin/androidx/compose/ui/text/input/ImeOptions.kt
Outdated
Show resolved
Hide resolved
d0ebd82
to
0eb23ac
Compare
...i-text/src/uikitMain/kotlin/androidx/compose/ui/text/input/PlatformImeOptions.skiko.uikit.kt
Outdated
Show resolved
Hide resolved
...e/ui/ui-text/src/uikitMain/kotlin/androidx/compose/ui/text/input/PlatformImeOptions.uikit.kt
Outdated
Show resolved
Hide resolved
...i-text/src/uikitMain/kotlin/androidx/compose/ui/text/input/PlatformImeOptions.skiko.uikit.kt
Outdated
Show resolved
Hide resolved
...i-text/src/uikitMain/kotlin/androidx/compose/ui/text/input/PlatformImeOptions.skiko.uikit.kt
Outdated
Show resolved
Hide resolved
cecf88b
to
0eb23ac
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.
Please follow API checks doc and re-gen API dump
@@ -1380,7 +1380,7 @@ public final class androidx/compose/ui/text/input/PasswordVisualTransformation : | |||
public fun hashCode ()I | |||
} | |||
|
|||
public final class androidx/compose/ui/text/input/PlatformImeOptions { | |||
public class androidx/compose/ui/text/input/PlatformImeOptions { |
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.
cc @igordmn for awareness: changing actual
skiko/non-android variant of an empty (not yet adopted) class to open
.
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.
I would prefer adding uikit-only open variant of the PlatformImeOptions
, keeping final
for all other platforms.
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's not compilation target specific. It's about how non-android variants handle that, so I see no reason to introduce the diff here.
Keep only skiko actual variant was an explicit ask from me above
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.
@svastven , could you please create a separate ticket for covering this functionality with tests - at least check the fact that we're propagating required values to the UITestInput.
bd7b1a3
to
a7270dc
Compare
a7270dc
to
db3e5e8
Compare
This PR introduces iOS-specific IME options support for text input components. This enables users to access and configure native UIKit text input traits from Compose code.
Fixes - https://youtrack.jetbrains.com/issue/CMP-7882
Testing
iOS Platform IME Options section was added to the demo application into the iOS-specific features section
This should be tested by QA
Release Notes
Features - iOS
PlatformImeOptions