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

Adds workaround for caret not appearing on iOS 17 #314

Merged
merged 2 commits into from
Sep 24, 2023

Conversation

simonbs
Copy link
Owner

@simonbs simonbs commented Sep 24, 2023

This works around an issue where the caret does not appear when editing starts on iOS 17 until the user have tapped the text view again.

This is a regression in UITextInteraction and other developers of text editors are seeing the same issue.

I have filed this as feedback to Apple on July 15th, 2023 but have yet to hear back from them (FB12622609).

The issue seems to be caused by the UITextSelectionDisplayInteraction that is automatically installed by UITextInteraction not being active initially.

@simonbs simonbs self-assigned this Sep 24, 2023
@simonbs simonbs merged commit 4a565b5 into main Sep 24, 2023
2 checks passed
@simonbs simonbs deleted the bugfix/ios17-caret-not-appearing branch September 24, 2023 14:20
@simonbs
Copy link
Owner Author

simonbs commented Sep 24, 2023

This has made it through app review and is available in Runestone Text Editor 1.3.20: https://apps.apple.com/dk/app/runestone-text-editor/id1548193893

eliperkins added a commit to eliperkins/Runestone that referenced this pull request Nov 30, 2023
This will still mean that the bug addressed in simonbs#314 will persist, but will allow consumers bound to older Xcode versions to continue to update the rest of the codebase and its dependencies
simonbs pushed a commit that referenced this pull request Nov 30, 2023
* Enable backwards compatibility when compiling with Xcode 14

This will still mean that the bug addressed in #314 will persist, but will allow consumers bound to older Xcode versions to continue to update the rest of the codebase and its dependencies

* Add end-of-file new lines

* Safe-guard usage as well
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.

1 participant