-
Notifications
You must be signed in to change notification settings - Fork 61
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
make resources offline available #112
Comments
cc @gr2m :) |
We're generating ours like this: https://github.com/pouchdb/pouchdb/blob/master/docs/manifest.appcache if it's any help. |
Thanks @NickColley for the input :) |
I'm not sure if appcache is the best solution to tackle the font issue. I'd separate the issue from making the docs available offline. Could you check with @smashingmag how they did it? I know Vitaly put quite some effort to find a best solution to make their page fast, and I think they ended up with a solution for the fonts that loads them async, and caches them in localStorage |
@gr2m this sounds awesome! will do. Thanks! |
The font is now stored in the localStorage thanks to @verpixelt! :) We still want the documentation to be offline available. Any suggestions @gr2m ? |
Booop https://github.com/pouchdb/pouchdb/blob/master/docs/manifest.appcache Also keep in mind with appcache it will always serve from it's cache first then update itself, so we did the following to tell the user there's an update: https://github.com/pouchdb/pouchdb/blob/master/docs/_layouts/default.html#L202 |
I'd say let's tackle the offline thing for next hood.ie. It should be straight forward to do with appcache-nanny, including the update notification |
Appcache-Nanny was also my thought. |
Because of font issues (#101 and #108) and also because we want to make the website and especially documentation offline available, I'd love to add an app manifest. As we know, the app cache is a douchebag, I'd suggest we use appcache-nanny. https://www.npmjs.org/package/appcache-nanny
Your suggestions?
cc @verpixelt
The text was updated successfully, but these errors were encountered: