Skip to content
This repository was archived by the owner on Mar 10, 2023. It is now read-only.

Update explainers to reflect latest discussions #14

Closed
addyosmani opened this issue Nov 6, 2017 · 2 comments
Closed

Update explainers to reflect latest discussions #14

addyosmani opened this issue Nov 6, 2017 · 2 comments

Comments

@addyosmani
Copy link
Collaborator

Trying to summarize a few discussions before filing PRs:

  1. We are now going to use the importance attribute instead of group. Related to Converge to a single group attribute, with values indicating before and after #6
  2. We will use the following importance buckets: critical, high, medium, low, unimportant Group names bikeshed #7
  3. We do want a JS equivalent for what we're doing here, but have not concretely explored this yet. Support SW install usage case via new property on Request interface #11 touches on this. We can say "There is a desire to explore priority hints importance in terms of fetch.". Perhaps we should just reference that issue or another to encourage continuing that discussion.
  4. V0 will not explore a resource throttling algorithm, which came up in Questions following initial implementation #10. Does that sound reasonable? We've had some input on this topic but unsure if enough to make a call here.
@yoavweiss
Copy link
Contributor

Thanks for summarizing the different discussions! This all makes sense to me.
Regarding 4), UAs could apply throttling on low priority third parties in H2 regardless of explicit hints. It's also something we can explore using SW.

@addyosmani
Copy link
Collaborator Author

Thanks for summarizing the different discussions!

Cheers!

4), UAs could apply throttling on low priority third parties in H2 regardless of explicit hints. It's also something we can explore using SW.

This is a useful reminder and something good to be aware of.

addyosmani added a commit to addyosmani/priority-hints that referenced this issue Nov 6, 2017
addyosmani added a commit to addyosmani/priority-hints that referenced this issue Nov 6, 2017
yoavweiss added a commit that referenced this issue Nov 6, 2017
Update attribute name from group -> importance (#14)
yoavweiss added a commit that referenced this issue Nov 6, 2017
Add `importance` groups from #14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants