Fix TextInput prop types & remove system props

[jfuchs]

This PR should give us a more accurate type for the props of TextInput. Previously the props included:

type TextInputInternalProps = {
// etc
} & ComponentProps<typeof Wrapper> &
ComponentProps<typeof Input>

But that left some props as a messy intersection of types, like onChange (#1377). We can resolve that by explicitly omitting any fields from Input's props that are explicitly destructured from TextInput's props.

Closes#1377

See also #1163

Merge checklist

• Added/updated tests
• Added/updated documentation
• Tested in Chrome
• Tested in Firefox
• Tested in Safari
• Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

[changeset-bot]

🦋 Changeset detected

Latest commit: e6959bd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@primer/components	Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[jfuchs]

I've got some tests to fix, but wanted to put up the PR to get feedback on the approach

[github-actions]

size-limit report 📦

Path	Size
dist/browser.esm.js	48.59 KB (-0.23% 🔽)
dist/browser.umd.js	48.85 KB (-0.21% 🔽)

[colebemis]

👍