-
Notifications
You must be signed in to change notification settings - Fork 22.6k
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
Issue/12834 Fix Content Negotiation Blurry Text In Diagram #12856
Conversation
There was a problem hiding this 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.
5f816f6
to
a30a9d9
Compare
18a25b1
to
f6c360f
Compare
f6c360f
to
6ea714a
Compare
Thanks for the catch! Did some fiddling to clean up git history, but now it should be one commit for the correct image. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was my mistake, I skipped reading the image protocol. Updated with a compressed image that should passed tests. |
Thanks a lot! Looks good to me. Congratulations upon your first merged commit here! Welcome aboard! 🎉 |
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