Fix React warnings

[somebody1234]

Pull Request Description

Fix some React warnings:

• Fix warning about nested buttons when close button is in tab button
  • The fix for now is to change the outer button to a link without a href. It's not ideal but we still want to use the CSS that comes with buttons, we just can't have nested button elements in HTML.
• Fix warning about setting state outside of a useEffect when deciding to show Local category when user is not enabled
• Fix passing invalid isInvalid prop to plain HTML element in ToS modal

Important Notes

None

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

• [x] The documentation has been updated, if necessary.
• [x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
• [x] All code follows the Scala, Java, TypeScript, and Rust style guides. In case you are using a language not listed above, follow the Rust style guide.
• [x] Unit tests have been written where possible.

[somebody1234]

@MrFlashAccount here's the wip btw - based on the debug logs it seems like the clip-path is getting set, and then it is immediately getting unset by a second ref containing null, in a component that somehow has a reference to the same background element

on a side note, i've removed loaderPosition from ariaComponents.Button since i imagine it was only useful for tab buttons - the default behavior is probably what's best for all actual buttons

[MrFlashAccount]

on a side note, i've removed loaderPosition from ariaComponents.Button since i imagine it was only useful for tab buttons - the default behavior is probably what's best for all actual buttons

This one is a good thing. I think we should probably keep it