Skip to content

Fix TextInput maxLength documentation - #2856

Open
huytdps13400 wants to merge 1 commit into
necolas:masterfrom
huytdps13400:fix/2855-text-input-max-length-doc-type
Open

huytdps13400 wants to merge 1 commit into
necolas:masterfrom
huytdps13400:fix/2855-text-input-max-length-doc-type

Conversation

@huytdps13400

Copy link
Copy Markdown

Summary

  • correct the documented TextInput maxLength prop type from string to number
  • align the generated documentation with the existing public Flow type and HTML contract

Fixes #2855.

Verification

  • npm run build -w react-native-web-docs: passed; generated TextInput page renders maxLength as ?number
  • npm run test: Flow, Prettier, ESLint, 46 DOM suites / 712 tests passed, 3 Node suites / 6 tests passed
  • git diff --check: passed

@codesandbox-ci

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit a3ba3f6:

Sandbox Source
react-native-web-examples Configuration

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.

TextInput maxLength documentation uses the wrong type

1 participant