-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Enable auto correct #359
Enable auto correct #359
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #359 +/- ##
=======================================
Coverage 87.89% 87.90%
=======================================
Files 64 64
Lines 10379 10382 +3
=======================================
+ Hits 9123 9126 +3
Misses 1256 1256 ☔ View full report in Codecov by Sentry. |
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.
Thank you for contributing! I personally disable the auto correction the moment I setup my devices so never noticed we're missing auto correction. But it would make sense to have a flag to control it from 'FleatherEditor' and 'FleatherField'.
Oh really? You must have amazing finger control. Updated PR now with a new |
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.
Thanks! I left a comment for you. PTAL.
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 accidentally deleted my previous review. Left you some comments please take a look.
BTW, the auto correction seems to be working fine on iOS too. Simulator.Screen.Recording.-.iPhone.15.-.2024-05-29.at.22.34.00.mov |
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.
Thanks, LGTM! @amantoux Do you wanna take a look too?
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.
LGTM
I thought it would be a major undertaking, but after some research it turns out toggling the autocorrect flag was enough. Is there a reason this was turned off or can it be enabled? Auto correct was one of the main reasons that made me still reach for Apple Notes at times.
Note that I'm using Google Keyboard in the attached demo and auto correct also worked when I tried with SwiftKey. I could not get it to work with the default iOS keyboard (not in English nor Swedish). Any thoughts on that? My guess it is some kind of locale setting either on my device or with the Fleather editor somehow.
RPReplay_Final1716982462.mp4