Skip to content
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

reproduce and reminder about iOS SelectionContainer bug #867

Merged
merged 10 commits into from
Oct 12, 2023

Conversation

dima-avdeev-jb
Copy link

Made reproducer of a bug JetBrains/compose-multiplatform#3718
This bug should be fixed after merging newer version of androidx-main to jb-main branch.
Added TODO comment to check and close related issues after that.

@@ -24,5 +24,6 @@ val IosBugs = Screen.Selection(
KeyboardEmptyWhiteSpace,
KeyboardPasswordType,
UIKitRenderSync,
DispatchersMainDelayCheck
DispatchersMainDelayCheck,
SelectionContainer,
Copy link
Member

Choose a reason for hiding this comment

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

I guess we need to add such tests into common code. This list was made to call API from uikitMain source set, but now it's used not for this purpose

Copy link
Author

Choose a reason for hiding this comment

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

Done!
Moved to commonMain "BugReproducers" section.
Also moved all bug reproducers to separate package.

…ainer

# Conflicts:
#	compose/mpp/demo/src/uikitMain/kotlin/bugs/IosBugs.kt
@dima-avdeev-jb dima-avdeev-jb merged commit b4c9798 into jb-main Oct 12, 2023
2 of 3 checks passed
@dima-avdeev-jb dima-avdeev-jb deleted the dima.avdeev/COMPOSE-478-iOS-SelectionContainer branch October 12, 2023 08:53
mazunin-v-jb pushed a commit that referenced this pull request Dec 7, 2023
Made reproducer of a bug
JetBrains/compose-multiplatform#3718
This bug should be fixed after merging newer version of androidx-main to
jb-main branch.
Added TODO comment to check and close related issues after that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants