Refactor internal components into `internal` folder

[joshblack]

Refactor internal components into `internal` folder

[joshblack]

Hey @lesliecdubs! No plans in particular, just was something that came up in PR reviews last week. Inbox triage could be a great place for it 👍

[ollie-iterators]

_VisuallyHidden.tsx is still on the page https://github.com/primer/react/tree/main/src, so it should not be checked off, but _TextInputInnerVisualSlot should be.

[mperrotti]

We shouldn't have two - we should just have one.

@joshblack - does your VisuallyHidden do anything that the original doesn't?

[ollie-iterators]

It looks like the same thing also may be happening with _TextInputInnerAction.tsx

[lesliecdubs]

Thanks for filing @joshblack! Should this issue go into our Primer teams backlog "inbox" for triage by the team, or did you have other plans for it?

[green6erry]

It appears there are two VisuallyHidden components. It looks like the main authors were @mperrotti on the one not in internal, and @joshblack for the one already in internal. The easiest way forward would be to keep both and they are used in multiple places. Is someone able to weigh in how to proceed?

[joshblack]

@mperrotti I think they have a difference in API and styles and we just haven't taken the time to merge them yet. I think the difference at that point ends up being that you would toggle rendering VisuallyHidden instead of passing in isVisible, if I remember right!