fix: Update TextInput ref forwarding

[ocoutts-usds]

Summary

Currently this component forwards refs in a way that does not work in new versions of react and I'm not sure if it ever worked in any version of react. Leaving the old way in case somebody is using it.

[brandonlenz]

A bit weird, but the existing pattern for making sure the components have a display name is done like so:

• react-uswds/src/components/forms/DatePicker/Day.tsx

  Line 27 in 4513e51

  const DayForwardRef: React.ForwardRefRenderFunction<

• react-uswds/src/components/forms/DatePicker/Day.tsx

  Line 113 in 4513e51

  export const Day = forwardRef(DayForwardRef)

I don't remember if there was a reason we did it that way, or if your approach is better, so I won't block on it

[ocoutts-usds]

I am happy to change but I find it a bit strange - this approach makes it so that the display name of the component (shows up in dev tools, error messages) will be DayForwardRef not Day which I find somewhat unideal.

[brandonlenz]

I agree with you. Let's leave it as is. Can always update easily if we find any problems or want to unify one way or the other

[kimallen]

Could you add some context in the Summary for what bug this addresses?

[brandonlenz]

Currently this component forwards refs in a way that does not work in new versions of react and I'm not sure if it ever worked in any version of react. Leaving the old way in case somebody is using it.

Yea this might have been a relic from much older versions of react, since I think this was likely one of the first components built for ReactUSWDS

[brandonlenz]

@all-contributors please add ocoutts-usds for bug, code

[allcontributors]

@brandonlenz

Could not find the user [ocoutts-usds](https://github.com/ocoutts-usds) on github.

[brandonlenz]

@all-contributors please add @ocoutts-usds for bug,code

[allcontributors]

@brandonlenz

I've put up a pull request to add @ocoutts-usds! 🎉