Caching assets in localStorage definitely isn't a "best practice," and seems like something that should only be done when necessary.

One of the best uses for localStorage though, is caching client-side templates. If you're building an app that's got a ton of different models and templates for each one, it makes your application a lot more efficient to lazy-load them as needed and cache them in localStorage for later use with versioning and cache invalidation.