replace Session Storage and Local Storage drivers with Web Storage driver

[thecotne]

Describe the feature

session and locale storage is really same kind of storage so it does not make sense to have two separate drivers for them

see

• https://developer.mozilla.org/en-US/docs/Web/API/Storage
• https://developer.mozilla.org/en-US/docs/Web/API/Web_Storage_API

it's also one interface in typescript for both objects

• https://github.com/microsoft/TypeScript/blob/main/src/lib/dom.generated.d.ts#L21520
• https://github.com/microsoft/TypeScript/blob/main/src/lib/dom.generated.d.ts#L25709
• https://github.com/microsoft/TypeScript/blob/main/src/lib/dom.generated.d.ts#L25758

Additional information

• Would you be willing to help implement this feature?

[pi0]

You are right. PR welcome to introduce a single new driver. We can reexport it as local/session storage for ease of use and bw compatibility.

[thecotne]

@pi0 i am confused #530 does not seem to be related to this issue

#530 is some minor refactoring reusing one driver in another

this issue is feature request for new driver

[pi0]

The term "web" storage is less known and might be confusing in unstorage drivers contrxt since we list browser specific drivers in a page (+indexdb and cookie in future) ++ unstorage interface itself is loosly based on same interface.

With refactor, localstorage driver is basically same but only a more known name.

If you have any ideas for another name that is more clear we can do it in v2.

[thecotne]

idk what would be more clear to you. i am just confused why you said i should open PR and then did not review it...

if you did not like term "Web Storage" why not say it? or should i have picked up a hint from you starting your comment with "You are right. PR welcome" as "i don't have time for this"?

should we open issue in MDN repo to inform them that term "Web Storage" is confusing to web developers?

[pi0]

Thanks again for your time ❤️ , sorry the review just got delayed and some changes stalled in there, otherwise, I would have commented or applied to refactor on yours... The main idea of refactoring them to one impl was the main point for me to ease maintenance.

Web Storage abstraction term in the context of MDN docs and standards is totally fine but as an unstorage driver name could be confusing IMO + the fact that abstracted driver is not usable, only available Web Storage instances (localStorage or sessionStorage) are.