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

use localStorage for snippets cache #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ciki
Copy link

@Ciki Ciki commented May 2, 2015

No description provided.

@Ciki Ciki changed the title use localStorage for snippets cache [close #24] use localStorage for snippets cache May 2, 2015
@Ciki
Copy link
Author

Ciki commented May 2, 2015

However, I'm not sure now this is the right approach as it does not handle situations when localStorage is full (5MB)

@Tomas2D
Copy link

Tomas2D commented Jun 5, 2016

Very useful! +1

@becygroup
Copy link

@Ciki I think you can you something like localStorage.removeItem(snippetsOrHash); in loadSnippets function

@zipper
Copy link
Contributor

zipper commented Dec 7, 2016

It might be useful to make methods loadSnippets and saveSnippets available from outside this extension, so you can use them in another custom extensions. Eg. $.nette.ext('history').loadSnippets(...) as is in your forked repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants