Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remember autoShow state in locale storage (if available) #283

Closed

Conversation

frankdekker
Copy link
Contributor

The autoShow state wasn't remember between page refreshes.

When MailDev interface loads the autoShow state will be retrieved from local storage.
When autoShow state changes, the state will be persisted in local storage.

@codecov-io
Copy link

Codecov Report

Merging #283 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #283   +/-   ##
=======================================
  Coverage   71.65%   71.65%           
=======================================
  Files           9        9           
  Lines         515      515           
  Branches      107      107           
=======================================
  Hits          369      369           
  Misses        146      146

@matthijsthoolen
Copy link

@oktapodia any chance you can take a look at this pull request?

@djfarrelly
Copy link
Member

djfarrelly commented Mar 29, 2020

Thanks for your contribution @frankdekker! With your other contribution in #284, I decided to use this idea and expand it to both app settings with #291 :)

@djfarrelly djfarrelly closed this Mar 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants