Skip to content
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

2022 HTTP Chapter #3128

Merged
merged 10 commits into from
Sep 21, 2022
Merged

2022 HTTP Chapter #3128

merged 10 commits into from
Sep 21, 2022

Conversation

paivaspol
Copy link
Contributor

@paivaspol paivaspol commented Sep 15, 2022

@paivaspol paivaspol self-assigned this Sep 15, 2022
@tunetheweb tunetheweb added the writing Related to wording and content label Sep 15, 2022
@tunetheweb tunetheweb added this to the 2022 Content Writing milestone Sep 15, 2022
@github-actions
Copy link
Contributor

Images automagically compressed by Calibre's image-actions

Compression reduced images by 25.1%, saving 39.65 KB.

Filename Before After Improvement Visual comparison
src/static/images/2022/http/alt-svc-example.png 36.55 KB 21.78 KB -40.4% View diff
src/static/images/2022/http/connections-comparison-per-page.png 19.27 KB 15.08 KB -21.7% View diff
src/static/images/2022/http/h2-push-usage.png 15.88 KB 13.36 KB -15.8% View diff
src/static/images/2022/http/h3-support-by-cdn-use.png 19.88 KB 15.42 KB -22.4% View diff
src/static/images/2022/http/h3-support-per-request.png 16.81 KB 13.99 KB -16.8% View diff
src/static/images/2022/http/http2-adoption-per-cdn.png 17.09 KB 13.22 KB -22.6% View diff
src/static/images/2022/http/http2-adoption-per-request.png 16.41 KB 12.63 KB -23.0% View diff
src/static/images/2022/http/http2-adoption-per-site.png 16.17 KB 12.92 KB -20.1% View diff

1553 images did not require optimisation.

@paivaspol paivaspol marked this pull request as ready for review September 16, 2022 04:19
@paivaspol paivaspol mentioned this pull request Sep 16, 2022
6 tasks
@paivaspol paivaspol linked an issue Sep 16, 2022 that may be closed by this pull request
6 tasks
@rviscomi
Copy link
Member

What's the status of this PR, any blockers?

@paivaspol
Copy link
Contributor Author

Nope. It’s ready for review and merge.

@tunetheweb
Copy link
Member

@paivaspol I've edited the chapter and pushed to this branch, and updated the staging link above.

The main changes were:

  • Tidied up figures
  • Reworded HTTP/2 push alternatives section, including adding a new call out figure.
  • Removed HTTP/3 CDN adoption graph as data is suspect as discussed on chat.

Let me know if you're good to merge.

@tunetheweb
Copy link
Member

Oh and you can see the edits I made here: 657550f

@tunetheweb tunetheweb merged commit c814e62 into main Sep 21, 2022
@tunetheweb tunetheweb deleted the http2022-markdown-and-figures branch September 21, 2022 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
writing Related to wording and content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HTTP 2022
3 participants