-
Notifications
You must be signed in to change notification settings - Fork 823
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
Add importance: 'low' to precaching requests #1855
Comments
Ditto for the revalidation request used in the |
@addyosmani do we know yet how priority will get exposed in |
The example from the spec for making requests is
i.e. it gets passed in as either That would imply that it's exposed as |
CC: @khempenius, who expressed some interest in this. |
I approve this feature 👍 https://twitter.com/DenisTRUFFAUT/status/1133372511280869376 |
I think this feature is needed to make precaching a truly progressive feature of any PWA. Right now, and under certain circumstances (e.g. poor mobile data connection and a few megs of assets to precache), precaching initially worsens the user experience and must be compensated for -- e.g. delayed service worker registration & loading screen upon service worker registration that is closed upon activation. It look forward to the days of smarter (it's already pretty good) precaching :-) |
I've posted this elsewhere, but just to reflect the most recent status on this issue: Priority Hints went through the Origin Trial process in Chrome, and did not progress past that in their current incarnation. So we're currently blocked on implemented anything in Workbox until at least one browser ships this feature. Folks can check https://chromestatus.com/feature/5273474901737472 and ⭐️ https://bugs.chromium.org/p/chromium/issues/detail?id=821464 for updates. |
Priority Hints is currently being experimented with in Chrome, and may come to other browsers in the future.
We can make use of them to mark the
fetch()
requests used for precaching as being low-priority, to reduce the chance that they will consume bandwidth needed for requests made from thewindow
context.CC: @addyosmani
The text was updated successfully, but these errors were encountered: