Store toolbar state in localStorage

[5e-Cleric]

Both visibility and zoom state should be saved in localstorage with the id of the brew, to guarrantee a better UX, if a user does not want to have the toolbar there all the time, it should stay hidden between reloads, and same goes for the zoom level.

[Gazook89]

How many different brew ids should be in there?

[calculuschild]

Why not just a universal setting instead of per brew?

[5e-Cleric]

I was thinking you may want to see a different zoom and page layout depending on the brew, but perhaps you're right, one universal setting is good enough.

[Gazook89]

How about this: save the current setting in localStorage, not as brew specific. But if a new (not just /new but just a different) brew is opened, it is reset to the defaults.

So this means that if you hit reload on the page, the view settings are preserved. But if you decide to view/edit/create a new brew, it goes back to the default.

[calculuschild]

How about this: save the current setting in localStorage, not as brew specific. But if a new (not just /new but just a different) brew is opened, it is reset to the defaults.

So this means that if you hit reload on the page, the view settings are preserved. But if you decide to view/edit/create a new brew, it goes back to the default.

This restriction sounds like it defeats the purpose of the PR if it disappears every time you change pages. How often are users just hitting reload on the same page without visiting another brew in between?

[5e-Cleric]

This restriction sounds like it defeats the purpose of the PR if it disappears every time you change pages.

I agree