Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixed keyboard appearing in SelectionContainer in iOS #1448

Merged

Conversation

mazunin-v-jb
Copy link

@mazunin-v-jb mazunin-v-jb commented Jul 14, 2024

Fixed the keyboard appearing in SelectionContainer in iOS

Fixes: JetBrains/compose-multiplatform#4986
https://youtrack.jetbrains.com/issue/CMP-1554/iOS-Selection-Container-shows-keyboard-while-selecting-text
JetBrains/compose-multiplatform#5088

Testing

Manual. Open Test App, go Components -> Selection, double tap any selectable text, on-screen keyboard shouldn't appear.

Release Notes

Fixes - iOS

  • Fixed the keyboard appearing when selecting from SelectionContainer

Google CLA

You need to sign the Google Contributor’s License Agreement at https://cla.developers.google.com/.
This is needed since we synchronise most of the code with Google’s AOSP repository. Signing this agreement allows us to synchronise code from your Pull Requests as well.

@mazunin-v-jb mazunin-v-jb self-assigned this Jul 14, 2024
Copy link
Member

@MatkovIvan MatkovIvan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it will also fix JetBrains/compose-multiplatform#5088

@mazunin-v-jb
Copy link
Author

you right, I forgot to add that

@mazunin-v-jb
Copy link
Author

It would be great to add some uikit instrumented tests for this. However, its current implementation does not support imitation of double / triple taps (which is crucial for testing this), so it is decided to merge it without auto tests.

@mazunin-v-jb mazunin-v-jb merged commit ec6b8fb into jb-main Jul 22, 2024
5 of 6 checks passed
@mazunin-v-jb mazunin-v-jb deleted the v.mazunin/fix-keybord-appearing-in-selection-container branch July 22, 2024 13:15
@chrisjenx
Copy link

I will test alpha02 in a bit and let you know if it fixes #5088

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(iOS) Selection Container shows keyboard while selecting text
4 participants