-
Notifications
You must be signed in to change notification settings - Fork 9
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
Function calls in vignettes could link to the man page #68
Comments
Regex is probably going to be very tricky here. You could try |
Actually, I think that if we render the vignettes using Quarto, we get links automatically: https://marginaleffects.com/articles/comparisons.html |
Oh that's cool |
What I wrote above is incorrect: Links are not added automatically in quarto. I opened a PR to resolve this: |
The PR was merged but this issue is still open. I think this is because it still does not work everywhere, does it? (At least it does not for me, even with Relatedly, it would be good to hyperlink the topics in the "See Also" sections of the manual pages. References to functions in the same package could be hyperlinked within the altdoc site. References to other packages could resolve to rdrr.io (which pkgdown uses as well). |
So I realized after doing a bunch of work that Quarto can autolink by adding a line to the preamble. Leaving this open for now because I need to remove the cruft from that merged PR. Good call on the See Also. I'll look into it when I find time (probably after UseR!) |
@zeileis I think you might be able to get function links everywhere by adding this to the bottom of your format:
html:
code-link: true Your |
Hmm, the And, yes, I like the look of the betareg page, too. But it is altdoc/quarto which makes this easy. I didn't have to do a lot to get something really nice. |
Thanks. Reopening, and will check later. |
@zeileis I'm not sure this is going to fix the "See Also" problem, but I think the lack of internal links is likely due to the absence of a http://marginaleffects.com/pkgdown.yml Has this content: urls:
reference: https://marginaleffects.com/man
article: https://marginaleffects.com/vignettes Note that this YAML file must be live on the internet when |
Thanks, Vincent, I just told you in person but just to keep Etienne in the loop as well: Putting |
OK, thanks. Not sure what As for the NEWS, I thought that it might be a .md vs. .qmd issue but the result was the same no matter which suffix I used (unless I overlooked something). |
pkgdown
does this.Probably requires some regex magic.
The text was updated successfully, but these errors were encountered: