fix(useLocalStorage): state change on first mount

[adesurirey]

Description

Fixes a regression introduced in fdd1b23 where useLocalStorage updates it state during first mount leading to an unwanted re-render. To avoid that I replaced useLayoutEffect by useUpdateEffect and added a test that ensure we only read the storage once on first mount.

closes #1970

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as before)

Checklist

• Read the Contributing Guide
• Perform a code self-review
• Comment the code, particularly in hard-to-understand areas
• Add documentation
• Add hook's story at Storybook
• Cover changes with tests
• Ensure the test suite passes (yarn test)
• Provide 100% tests coverage
• Make sure code lints (yarn lint). Fix it with yarn lint:fix in case of failure.
• Make sure types are fine (yarn lint:types).

[defunctzombie]

@adesurirey anything holding back this fix?

[adesurirey]

@defunctzombie no just waiting to be reviewed and merged

/cc @xobotyi

[xobotyi]

@adesurirey i'm no more collaborator =)
I've strated my own project wher i can control anything except from prs merge 🙃

https://github.com/react-hookz/web

[adesurirey]

@xobotyi so react-use is definitely dead?

[xobotyi]

@adesurirey presumably - yes.

I've explained everything here: #1974