Remove references to AppCache #356
Conversation
| </IfModule> | ||
|
|
||
| AddType text/cache-manifest .manifest | ||
| AddType text/cache-manifest .appcache |
There was a problem hiding this comment.
We should drop this entire file. it doesn't get used
There was a problem hiding this comment.
True, I suppose it's leftover from a time this was once served by Apache. Deleted.
argilo
force-pushed
the
remove-app-cache
branch
from
fd2e9d9 to
784110a
Compare
|
|
||
| // if for some reason, applicationCache is not working, load the app after a 3s timeout | ||
| // load the app after a 3s timeout | ||
| var initTimer = setTimeout(trackerInit, 3000); |
There was a problem hiding this comment.
Just wondering - what if we just load straight away without a timeout?
There was a problem hiding this comment.
In that case, the splash screen won't be seen. Do you want to remove it?
There was a problem hiding this comment.
Oh, I guess we lose our sponsorship logos.... @darksidelemm there's probably little reason to wait 3 seconds at startup, but we'd a better way of doing sponsorship stuff... happy to keep it for the time being but something maybe we should think about?
There was a problem hiding this comment.
If you decide to keep the splash screen, a useful improvement would be to let the rest of the app initialize behind it. At present, things like map tiles and telemetry don't even begin loading until the splash screen disappears.
2 participants
AppCache is deprecated and support has been removed from browsers. AppCache functionality has been removed from sondehub-tracker as well, so we may as well clean up the last few references to it.