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

Issue/12834 Fix Content Negotiation Blurry Text In Diagram #12856

Conversation

hh-shiung
Copy link
Contributor

Summary

Redrew diagram for Content Negotiation to fix blurry text with previous image.

Motivation

The image clarity should be an improvement over the old image. Bad images strains eyeballs.

Supporting details

Related issues

Fixes #12834

Metadata

This PR

  • Fixes a typo, bug, or other error

@hh-shiung hh-shiung requested a review from a team as a code owner February 9, 2022 11:51
@hh-shiung hh-shiung requested review from teoli2003 and removed request for a team February 9, 2022 11:51
@github-actions github-actions bot added the Content:HTTP HTTP docs label Feb 9, 2022
Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new image looks good, but you haved replaced httpnego.png instead of httpnegoserver.png. If you fix this, we are good to go.

@hh-shiung hh-shiung force-pushed the Issue/12834-fix-content-negotiation-blurry-text-in-diagram branch from 5f816f6 to a30a9d9 Compare February 9, 2022 12:32
@hh-shiung hh-shiung requested review from a team as code owners February 9, 2022 12:32
@hh-shiung hh-shiung requested review from sideshowbarker and removed request for a team February 9, 2022 12:32
@github-actions github-actions bot added Content:WebAPI Web API docs Content:HTTP HTTP docs and removed Content:HTTP HTTP docs labels Feb 9, 2022
@hh-shiung hh-shiung force-pushed the Issue/12834-fix-content-negotiation-blurry-text-in-diagram branch from 18a25b1 to f6c360f Compare February 9, 2022 12:41
@github-actions github-actions bot removed the Content:WebAPI Web API docs label Feb 9, 2022
@hh-shiung hh-shiung force-pushed the Issue/12834-fix-content-negotiation-blurry-text-in-diagram branch from f6c360f to 6ea714a Compare February 9, 2022 12:43
@hh-shiung hh-shiung requested review from teoli2003 and removed request for a team February 9, 2022 12:45
@hh-shiung
Copy link
Contributor Author

Thanks for the catch!

Did some fiddling to clean up git history, but now it should be one commit for the correct image.

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to the test, you need to compress the image more:

image

(I always forget about this!)

@hh-shiung
Copy link
Contributor Author

It was my mistake, I skipped reading the image protocol.

Updated with a compressed image that should passed tests.

@hh-shiung hh-shiung requested a review from teoli2003 February 9, 2022 12:59
@teoli2003 teoli2003 merged commit 1f19a76 into mdn:main Feb 9, 2022
@teoli2003
Copy link
Contributor

Thanks a lot!

Looks good to me.

Congratulations upon your first merged commit here! Welcome aboard! 🎉

@hh-shiung hh-shiung deleted the Issue/12834-fix-content-negotiation-blurry-text-in-diagram branch February 9, 2022 15:48
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Content:HTTP HTTP docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with "Content negotiation": (Blurry text in diagrams)
2 participants