fix: update useDarkMode to not reset with OS value on every refresh

[refeals]

Fixes #512

This PR does two things:

1. Update useLocalStorage save initial value

The way things are now, useLocalStorage does not correctly represents the value in the localStorage. It returns a default value even if the key doesn't exist there yet and does not create it, meaning it returns a value that is not necessarily the truth (should be undefined. This causes a problem in useDarkMode (and probably other hooks).

The PR updates the hook to actually save the key/value in localStorage when the hook is called, preserving the hook's initial funcionality.

2. Update useDarkMode to not reset with OS value on every refresh

Removes the useIsomorphicLayoutEffect because it's unnecessary. The value isDarkMode will always be correctly set.

[changeset-bot]

⚠️ No Changeset found

Latest commit: ce8efc4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

[crucialfelix]

This PR fixes it for me — was always reverting to the OS default on every page load.

[refeals]

@juliencrn can you take a look?

[blacklee123]

please merge this pr

[rickyzhangca]

neeeeed

[Rigidity]

Keep running into this issue repeatedly

[abhijith-rubrik]

Can someone review and merge this

[refeals]

Unfortunately @juliencrn is the only one how can approve and merge this :/

[jgaik]

setValue already sets stored value, isnt it?
maybe we can remove the eslint now and fill the deps as the logic changed?