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

Add secureTextEntry to the OTPInput's TextInput component #381

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

ChrisMattew
Copy link
Collaborator

Short description

Added the secureTextEntry property to the TextInput of the custom OTPInput component to obscure the keyboard in screenshots and screen recordings, in case the component is used in a context where such protection is enabled (e.g. using react-native-screenshot-prevent lib).

List of changes proposed in this pull request

  • Added secureTextEntry prop

@ChrisMattew ChrisMattew requested review from dmnplb and a team as code owners January 17, 2025 14:49
@ChrisMattew ChrisMattew self-assigned this Jan 17, 2025
Copy link
Member

@shadowsheep1 shadowsheep1 left a comment

Choose a reason for hiding this comment

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

LGTM

@shadowsheep1 shadowsheep1 merged commit fc2f962 into main Jan 17, 2025
6 checks passed
@shadowsheep1 shadowsheep1 deleted the IOPID-2609-otp-input-secure-text branch January 17, 2025 15:11
shadowsheep1 pushed a commit that referenced this pull request Jan 20, 2025
## Short description
Added the `secureTextEntry` property to the `TextInput` of the custom
`OTPInput` component to obscure the keyboard in screenshots and screen
recordings, in case the component is used in a context where such
protection is enabled (e.g. using `react-native-screenshot-prevent`
lib).

## List of changes proposed in this pull request
- Added `secureTextEntry` prop
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