-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Incorrect TextInput email field suggestion for "Hide my Email" (iOS) #46022
Comments
I'm noticing this as well. |
(pinging @cortinico just because they're the only RN contributor on this issue) Any thoughts from the RN team about this issue? Having two "Hide my email" suggestions over an email input field, and one of them inserting the string "Hide my email" into the email input when tapped, is confusing for users and leaves an app feeling unpolished for them. |
Have you opened an issue on Expo as well by any chance? |
Thanks for responding @cortinico. I hadn't, but I just posted the issue to expo's repo as well: expo/expo#32483. Correct me if I'm wrong, but since this issue is with a react-native component, ( |
Yes you're right, but our team is extremely small and doesn't have capacity to look into any issue that gets reported (we get a lot of them). As you're using Expo, having the issue opened also in Expo will help to exclude any problems which might be Expo specific |
Totally understand. Thanks for all the amazing work that you and the RN team do! Hope someone can take a look at this soon. |
@cortinico - this shouldn't be related to expo, we don't interfere with core apis from react-native like TextInput |
@troyshu - can you try updating to 0.76 and seeing if that resolves your issue? https://expo.dev/changelog/2024/10-24-sdk-52-beta - you can run |
I ran |
cc @cipolleschi if you have any insights on what could be causing this on iOS |
@troyshu is this happening also if you opt out from the New Architecture? Just to understand whether it has always been like that or not. I don't see anything in the codebase that suggests that we add the text twice. |
Thanks for taking a look at this @cipolleschi. Yes, the issue still happens when not using the new architecture (e.g. on Expo 51): see reproduction repo at this commit. The most recent commit of reproduction repo is on Expo 52 (new architecture). I saw this issue in the wild the other day in the Claude iOS app. Instantly knew they were using React Native haha RPReplay.Final.movEDIT: also still seeing the issue after disabling new architecture in Expo 52 with |
This is probably related to something native on the apple side. Flutter has the same issue. As can be seen here |
Description
When entering text into an email TextField (
textContentType="emailAddress"
), the "Hide my Email" suggestion behaves incorrectly on a plugged in iOS device (using Apple ID with iCloud+ and thus "Hide my Email" feature):"Hide my Email" is suggested twice. Selecting the first "Hide my email" incorrectly inserts the string "Hide my email" into the text field.
See attached screen recording.
Steps to reproduce
npm install
thennpm run ios:device
to run on deviceReact Native Version
0.76.1 (Expo 52)
Also occurred on React Native 0.74.5 and Expo 51: https://github.com/troyshu/expo-text-input-test/tree/979d9fc382353f4aa117cad625d00e7e1f05f3a9
Affected Platforms
Runtime - iOS
Output of
npx react-native info
Stacktrace or Logs
NA
Reproducer
https://github.com/troyshu/expo-text-input-test
Screenshots and Videos
Untitled.mp4
The text was updated successfully, but these errors were encountered: