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

Jump to bottom

x/pkgsite: add redirect link to Sourcegraph for method caller / uses of a particular type #39703

Open
joshuabezaleel opened this issue Jun 19, 2020 · 2 comments
Open

x/pkgsite: add redirect link to Sourcegraph for method caller / uses of a particular type #39703

joshuabezaleel opened this issue Jun 19, 2020 · 2 comments
Labels
help wanted NeedsFix The path to resolution is known, but the work has not been done. pkgsite Suggested Issues that may be good for new contributors looking for work to do.
Milestone
pkgsite/unplanned

Comments

@joshuabezaleel
Copy link
Contributor

joshuabezaleel commented Jun 19, 2020 •
edited

What is the URL of the page with the issue?

Every package detail page. I first noticed it when accessed https://godoc.org/golang.org/x/tools/go/ssa#FieldAddr

What is your user agent?

Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.108 Safari/537.36

Screenshot

image

What did you do?

Hover to the right of the type/function.

What did you expect to see?

See a "Uses" link that list the method callers for a function and the one who use the type for a type which will then redirects to Sourcegraph for that particular function/type. For example, function Common() will redirect to this link.

image

image

What did you see instead?

I saw both the pilcrow (¶) for the type/function link and the Uses redirect link at the old godoc.org, but only saw pilcrow for every package at the new pkg.go.dev.
@gopherbot gopherbot added this to the Unreleased milestone Jun 19, 2020
@joshuabezaleel
Copy link
Contributor Author

joshuabezaleel commented Jun 19, 2020 •
edited

I am not that experienced to Go and pretty new to the codebase but if no one is interested I think I can give this issue a shot 🙂 cc: @julieqiu

@julieqiu
Copy link
Member

julieqiu commented Jun 22, 2020 •
edited

@joshuabezaleel - feel free to send a CL!

Some information to help you get started with this issue:

  1. Check out CONTRIBUTING.md.
  2. The documentation HTML is defined in internal/fetch/dochtml/template.go.

@julieqiu julieqiu added pkgsite NeedsFix The path to resolution is known, but the work has not been done. help wanted Suggested Issues that may be good for new contributors looking for work to do. labels Jun 22, 2020
@julieqiu julieqiu removed the website label Sep 22, 2020
@julieqiu julieqiu removed the website label Sep 22, 2020
@joshuabezaleel joshuabezaleel mentioned this issue Nov 8, 2020
Open
@joshuabezaleel joshuabezaleel mentioned this issue Jan 22, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted NeedsFix The path to resolution is known, but the work has not been done. pkgsite Suggested Issues that may be good for new contributors looking for work to do.
Projects
None yet
Milestone
pkgsite/unplanned
Development

No branches or pull requests
3 participants
@julieqiu @joshuabezaleel @gopherbot