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.

Sign up for GitHub

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

\href{}{} drops brackets #22

Open
leeper opened this issue Mar 17, 2016 · 2 comments
Open

\href{}{} drops brackets #22

leeper opened this issue Mar 17, 2016 · 2 comments

Comments

@leeper
Copy link

leeper commented Mar 17, 2016

I just started using this and noticed that tags of the form \href{URL}{TEXT} get converted to \href{URLTEXT} (brackets dropped). I'm not sure where this is happening in the code, though, so I don't know how to send a patch.
@yihui
Copy link
Owner

yihui commented Mar 17, 2016

It is already 2016 and you still have packages that are not using roxygen2 right from the beginning? :)

This is probably a bug, but it is not immediately clear to me how to fix it, and I do not have much time to investigate it, either. Sorry.

Also see https://github.com/yihui/Rd2roxygen/blob/master/R/Rd2roxygen-package.R#L14-L16

@leeper
Copy link
Author

leeper commented Mar 17, 2016

Currently resurrecting some old projects. :-/

Thanks, and no pressure, obviously.

@yihui yihui mentioned this issue Apr 15, 2016
Closed
@yihui yihui mentioned this issue May 29, 2017
Open
joshuaulrich added a commit to joshuaulrich/Rd2roxygen that referenced this issue Mar 21, 2024
@joshuaulrich
Fix \href{}{}
98757d0 
Fixes yihui#22.
joshuaulrich added a commit to joshuaulrich/Rd2roxygen that referenced this issue Mar 21, 2024
@joshuaulrich
Fix \href{}{}
ea5c07e 
Fixes yihui#22.
joshuaulrich added a commit that referenced this issue Apr 3, 2024
@joshuaulrich
Fix sections and href (#36)
e23f740 
* Add test Rd file
* Process all sections (#25)
  Only the first section was being processed. Any later sections were
  dropped silently.
* Fix \href{}{} (#22)
* Add \dots and \ldots to single argument macros
* Update news
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yihui @leeper