-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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
x/pkgsite: net/http.Cookie appears to be misrendered #64615
Comments
Hm, I could not reproduce in Firefox with a new/empty profile. It seems like disabling the extension https://addons.mozilla.org/en-GB/firefox/addon/ublock-origin/ fixes the issue as well. Here are the console logs from loading the page in Firefox with the extension, resulting in the broken UI:
And without the extension, where the UI is fine:
|
Indeed the adblocker's "cosmetic filters" are hiding the Cookie heading. Not sure how that is happening, but clearly not a pkgsite bug :) |
For the sake of completeness, it turned out that one of the filter lists considered the heading a "cookie notice annoyance" :) Oh dear. Disabled. |
Previously: #59901 For those running into this in the future, the culprit was EasyList - Cookie Notices, which I've disabled now. It appears to be too aggressive, thinking that the Cookie header/section must be a browser cookie notice. |
What is the URL of the page with the issue?
https://pkg.go.dev/net/http#Cookie
What is your user agent?
Mozilla/5.0 (X11; Linux x86_64; rv:120.0) Gecko/20100101 Firefox/120.0
Screenshot
What did you do?
I tried to view the
Cookie
type in thenet/http
package, by clicking on its name in the Index section.That correctly points to https://pkg.go.dev/net/http#Cookie, but that does... Nothing. Manually scrolling to find the Cookie type shows that, for some reason, it got rendered adjacent to
func (ConnState) String
and without any anchor link or title.What did you expect to see?
The anchor link should work, and the type should be rendered with a title or section header. For example, see
CookieJar
:What did you see instead?
See the first screenshot above.
The text was updated successfully, but these errors were encountered: