Rework useStorageValue to more simple and robust variant

[xobotyi]

New hook has less options, now it is impossible to make hook isolated, all hooks listening same key are synchronised.

Also hook return signature changed to object instead of array, as it provides more obvious API.

close #959
fix #451

[codecov]

Codecov Report

Merging #960 (b5b776e) into master (532cc41) will decrease coverage by 0.38%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##            master     #960      +/-   ##
===========================================
- Coverage   100.00%   99.61%   -0.39%     
===========================================
  Files           61       61              
  Lines         1065     1039      -26     
  Branches       184      165      -19     
===========================================
- Hits          1065     1035      -30     
- Misses           0        2       +2     
- Partials         0        2       +2     

Impacted Files	Coverage Δ
src/useLocalStorageValue/useLocalStorageValue.ts	77.77% <100.00%> (-22.23%)	⬇️
...c/useSessionStorageValue/useSessionStorageValue.ts	77.77% <100.00%> (-22.23%)	⬇️
src/useStorageValue/useStorageValue.ts	100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[xobotyi]

@ArttuOll could you please finish this one?

I think only docs left for now, the code itself and tests are already done.

[ArttuOll]

Sure! I'll take a look over the next few days.

[xobotyi]

@JoeDuncko @ArttuOll please review PR as a whole.

[ArttuOll]

The hook seems to work as expected.

One thing I noticed was that the code example does not show the usage of the hook itself; it just shows the rendered HTML. I could not figure out how to make it show the hook though.

I commented on a couple spots where the exhaustive-deps lint rule seemed to be unnecessarily disabled, which I personally dislike a lot, but it does not really affect the functionality of the hook here.

[xobotyi]

There is barely possible example of useStorageValue hook, also it is not exported at all.

As for useSessionStorage/LocalStorage - they're pretty functional, based on inputs, showing that hooks are synced across the page and, in case of LocalStoage - across tabs

[ArttuOll]

There is barely possible example of useStorageValue hook, also it is not exported at all.

As for useSessionStorage/LocalStorage - they're pretty functional, based on inputs, showing that hooks are synced across the page and, in case of LocalStoage - across tabs

Sorry, I think I was a bit unclear. I meant that the examples of useLocalStorageValue and useSessionStorageValue do not show the usage of the hook in their code examples (when you press the "show code" button in Storybook). The examples are good, but you cannot see the usage of the hook in the example code in Storybook, which I think would be helpful.

[xobotyi]

Have no idea how to make storybook to better handle code preview =(
Also have an idea of migrating to docosaurus since storybook caused lots of issues recently, but it is another discussion =)

[ArttuOll]

We have other examples also where this is a problem. I was thinking of going over the docs some time and doing little improvements like this. I'll worry about this then.

[xobotyi]

🎉 This PR is included in version 17.0.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀