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

Backport #9559 and #9484 for 3.11.x #9609

Merged
merged 10 commits into from
Aug 26, 2022

Conversation

jeromelaban
Copy link
Member

GitHub Issue (If applicable): #9430

jeromelaban and others added 6 commits August 25, 2022 15:25
…with UITextField

This change is needed to make the SinglelineTextBoxView.SelectedTextRange conditional, and avoid changing the value provided to UIKit. This breaks the use of `TextBoxSelectionChanged`, but can be restored by using the `FeatureConfiguration.TextBox.IOS16EnableSelectionSupport` feature flag if the issue ends up being fixed in UIKit. See #9430 for more details.
…ashes

Pro: It does not break app compatibility, selection works

Con: This does not work _currently_ with the dynamic registrar [1] which is the default when building for iOS simulators. OTOH there's a workaround for that [2].

[1] Unless building with the iOS SDK for Xcode 14 (currently only previews) once xamarin/xamarin-macios#15712 is merged

[2] Adding `--registrar=static` to the simulator builds will fix this, sadly build times will be longer (so better use [1] asap)
@gitpod-io
Copy link

gitpod-io bot commented Aug 25, 2022

@jeromelaban jeromelaban changed the base branch from release/stable/3.3 to release/stable/3.11 August 25, 2022 19:33
This step is needed by the snapshot tests, which are running npm at the beginning.
@jeromelaban jeromelaban merged commit 55dbd69 into release/stable/3.11 Aug 26, 2022
@jeromelaban jeromelaban deleted the dev/jela/ios16-textbox-bp-3x branch August 26, 2022 03:24
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