Moving theme switch component out of root

[lsbyerley]

I've been hacking around with this stack and find myself wanting to slim down the root component. For instance, I'd want to move the theme switch function to a Header component. From a remix perspective, is it bad practice to add the theme switch to a header component that is imported as a client component in the root (or an index file)? It is performing server actions by updating user preferences so I guess I may still be in the nextjs mindset of client/server components in thinking that it it would be incorrect.

[lsbyerley]

I've since turned this into a full stack component in my codebase.. if there's an interest I can submit a PR into this stack

[lsbyerley]

#737