Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Use TextInput instead of CodeInput in FormView #729

Merged
merged 8 commits into from
Jul 17, 2020
Merged

Conversation

henrytao-me
Copy link
Contributor

@henrytao-me henrytao-me commented Jul 16, 2020

Part of #727

@henrytao-me henrytao-me requested a review from a team as a code owner July 16, 2020 16:57

<item name="colorPrimary">#2B4380</item>
<item name="colorPrimaryDark">#26374A</item>
<item name="colorControlActivated">#26374A</item>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This will help with the cursor color for Android.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@amazingphilippe
Copy link
Contributor

Looks good! Can we align the input text to the left?

padding="s"
maxLength={8}
fontSize={26}
borderWidth={0}
Copy link
Contributor

Choose a reason for hiding this comment

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

I was playing around and if I can suggest these attributes I think it would be cool!

autoCapitalize="characters"
fontFamily="Menlo"
letterSpacing={5}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added! LGTM

@timarney
Copy link
Member

Tested on iOS device - I'm seeing white text on a white background.

@timarney timarney merged commit 03d5a78 into master Jul 17, 2020
@timarney timarney deleted the issue/727/code-input branch July 17, 2020 00:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants